fix: make OC\Core\Migrations\Version30000Date20240717111406 safer when adding columns in taskprocessing_tasks

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>

Author: julien-nc

core/Migrations/Version30000Date20240717111406.php

@@ -33,24 +33,29 @@ class Version30000Date20240717111406 extends SimpleMigrationStep {
 	public function changeSchema(IOutput $output, Closure $schemaClosure, array $options): ?ISchemaWrapper {
 		/** @var ISchemaWrapper $schema */
 		$schema = $schemaClosure();
+		$schemaChanged = false;
 
 		if ($schema->hasTable('taskprocessing_tasks')) {
 			$table = $schema->getTable('taskprocessing_tasks');
 
-			$table->addColumn('webhook_uri', Types::STRING, [
-				'notnull' => false,
-				'default' => null,
-				'length' => 4000,
-			]);
-			$table->addColumn('webhook_method', Types::STRING, [
-				'notnull' => false,
-				'default' => null,
-				'length' => 64,
-			]);
-
-			return $schema;
+			if (!$table->hasColumn('webhook_uri')) {
+				$table->addColumn('webhook_uri', Types::STRING, [
+					'notnull' => false,
+					'default' => null,
+					'length' => 4000,
+				]);
+				$schemaChanged = true;
+			}
+			if (!$table->hasColumn('webhook_method')) {
+				$table->addColumn('webhook_method', Types::STRING, [
+					'notnull' => false,
+					'default' => null,
+					'length' => 64,
+				]);
+				$schemaChanged = true;
+			}
 		}
 
-		return null;
+		return $schemaChanged ? $schema : null;
 	}
 }