fix(systemtags): objecttype index registration

Signed-off-by: skjnldsv <[email protected]>
nextcloud · Jan 15, 2025 · ad6f4a7 · ad6f4a7
1 parent 47eedf9
commit ad6f4a7
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 11 deletions.
diff --git a/apps/systemtags/lib/Migration/Version31000Date20241018063111.php b/apps/systemtags/lib/Migration/Version31000Date20241018063111.php
@@ -20,24 +20,15 @@
 use OCP\Migration\SimpleMigrationStep;
 
 /**
- * Add objecttype index to systemtag_object_mapping
+ * Add etag column to systemtag
  */
 #[AddColumn(table: 'systemtag', name: 'etag', type: ColumnType::STRING, description: 'Adding etag for systemtag table to prevent conflicts')]
-#[AddIndex(table: 'systemtag_object_mapping', type: IndexType::INDEX, description: 'Adding objecttype index to systemtag_object_mapping')]
 class Version31000Date20241018063111 extends SimpleMigrationStep {
 
 	public function changeSchema(IOutput $output, Closure $schemaClosure, array $options): ?ISchemaWrapper {
 		/** @var ISchemaWrapper $schema */
 		$schema = $schemaClosure();
 
-		if ($schema->hasTable('systemtag_object_mapping')) {
-			$table = $schema->getTable('systemtag_object_mapping');
-
-			if (!$table->hasIndex('systag_objecttype')) {
-				$table->addIndex(['objecttype'], 'systag_objecttype');
-			}
-		}
-
 		if ($schema->hasTable('systemtag')) {
 			$table = $schema->getTable('systemtag');
 

diff --git a/apps/systemtags/lib/Migration/Version31000Date20241114171300.php b/apps/systemtags/lib/Migration/Version31000Date20241114171300.php
@@ -18,7 +18,7 @@
 use OCP\Migration\SimpleMigrationStep;
 
 /**
- * Add objecttype index to systemtag_object_mapping
+ * Add color column to systemtag
  */
 #[AddColumn(table: 'systemtag', name: 'color', type: ColumnType::STRING, description: 'Adding color for systemtag table')]
 class Version31000Date20241114171300 extends SimpleMigrationStep {

diff --git a/core/Application.php b/core/Application.php
@@ -235,6 +235,12 @@ public function __construct() {
 				'systag_by_objectid',
 				['objectid']
 			);
+
+			$event->addMissingIndex(
+				'systemtag_object_mapping',
+				'systag_objecttype',
+				['objecttype']
+			);
 		});
 
 		$eventDispatcher->addListener(AddMissingPrimaryKeyEvent::class, function (AddMissingPrimaryKeyEvent $event) {

diff --git a/core/Migrations/Version13000Date20170718121200.php b/core/Migrations/Version13000Date20170718121200.php
@@ -732,6 +732,8 @@ public function changeSchema(IOutput $output, \Closure $schemaClosure, array $op
 			$table->addIndex(['systemtagid', 'objecttype'], 'systag_by_tagid');
 			// systag_by_objectid was added later and might be missing in older deployments
 			$table->addIndex(['objectid'], 'systag_by_objectid');
+			// systag_objecttype was added later and might be missing in older deployments
+			$table->addIndex(['objecttype'], 'systag_objecttype');
 		}
 
 		if (!$schema->hasTable('systemtag_group')) {