fixes #4138

Closes #4138

Merge request studip/studip!2980

app/views/web_migrate/index.php

@@ -51,7 +51,7 @@
                     <?= htmlReady($number) ?>
                 </td>
                 <td>
-                    <?= htmlReady(get_class($migration)) ?>
+                    <?= htmlReady($migration->getName()) ?>
                 </td>
                 <td>
                 <? if ($migration->description()): ?>

lib/migrations/Migration.php

@@ -48,6 +48,24 @@ abstract class Migration
         return '';
     }
 
+    /**
+     * Returns the name of the migration. If the migration is an anonymous
+     * class, the the name is created from the filename. Otherwise, it's the
+     * class name of the migration.
+     *
+     * @return string
+     */
+    public function getName(): string
+    {
+        $reflection = new ReflectionClass($this);
+        if ($reflection->isAnonymous()) {
+            $filename = basename($reflection->getFileName(), '.php');
+            $name = implode(' ', array_slice(explode('_', $filename), 1));
+            return ucfirst($name);
+        }
+        return static::class;
+    }
+
     /**
      * Abstract method performing this migration step.
      * This method should be implemented in a migration subclass.
@@ -67,12 +85,12 @@ abstract class Migration
     /**
      * Perform or revert this migration, depending on the indicated direction.
      *
-     * @param string $direction migration direction (either 'up' or 'down')
+     * @param ?string $direction migration direction (either 'up' or 'down')
      */
     public function migrate($direction)
     {
         if (!in_array($direction, ['up', 'down'])) {
-            return;
+            return null;
         }
 
         $result = $this->$direction();
@@ -107,7 +125,7 @@ abstract class Migration
         $args = func_get_args();
         $message = vsprintf(array_shift($args), $args);
 
-        return $this->write($this->mark($message));
+        $this->write($this->mark($message));
     }
 
     /**