Refactor update script to class and add unit test

1 files changed, 22 insertions, 0 deletions

diff --git a/apps/files_sharing/lib/updater.php b/apps/files_sharing/lib/updater.php
index 23ebc9fb811..950575ec26b 100644
--- a/apps/files_sharing/lib/updater.php
+++ b/apps/files_sharing/lib/updater.php
@@ -135,4 +135,26 @@ class Shared_Updater {
 		}
 	}
 
+	/**
+	 * clean up oc_share table from files which are no longer exists
+	 *
+	 * This fixes issues from updates from files_sharing < 0.3.5.6 (ownCloud 4.5)
+	 * It will just be called during the update of the app
+	 */
+	static public function fixBrokenSharesOnAppUpdate() {
+		// delete all shares where the original file no longer exists
+		$findShares = \OC_DB::prepare('SELECT `*PREFIX*share`.`id` ' .
+			'FROM `*PREFIX*share` LEFT JOIN `*PREFIX*filecache` ON `file_source` = `*PREFIX*filecache`.`fileid` ' .
+			'WHERE `*PREFIX*filecache`.`fileid` IS NULL AND `*PREFIX*share`.`item_type` IN (\'file\', \'folder\')');
+		$sharesFound = $findShares->execute(array())->fetchAll();
+
+		// delete those shares from the oc_share table
+		if (is_array($sharesFound) && !empty($sharesFound)) {
+			$removeShares = \OC_DB::prepare('DELETE FROM `*PREFIX*share` WHERE `id` = ? ');
+			foreach ($sharesFound as $share) {
+				$result = $removeShares->execute(array($share['id']));
+			}
+		}
+	}
+
 }