Remove unneeded preSchemaChange and return null if no changes

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

diff --git a/apps/files_external/lib/Migration/Version1016Date20220324154536.php b/apps/files_external/lib/Migration/Version1016Date20220324154536.php
index 0f3b55220940bb285dba39ff29e6ff28108a0785..a2a7ecaff1750a127e22cf9ca0fbff2c39afaeb2 100644 (file)
--- a/apps/files_external/lib/Migration/Version1016Date20220324154536.php
+++ b/apps/files_external/lib/Migration/Version1016Date20220324154536.php
@@ -31,15 +31,6 @@ use OCP\Migration\SimpleMigrationStep;
 
 class Version1016Date20220324154536 extends SimpleMigrationStep {
 
-       /**
-        * @param IOutput $output
-        * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
-        * @param array $options
-        */
-       public function preSchemaChange(IOutput $output, Closure $schemaClosure, array $options): void {
-               // FIXME do we need to check for 4000+ values?
-       }
-
        /**
         * @param IOutput $output
         * @param Closure $schemaClosure The `\Closure` returns a `ISchemaWrapper`
@@ -55,8 +46,9 @@ class Version1016Date20220324154536 extends SimpleMigrationStep {
 
                if ($column->getLength() > 4000) {
                        $column->setLength(4000);
+                       return $schema;
                }
 
-               return $schema;
+               return null;
        }
 }