Adds update_date_on_publish to configuration file (#49330)

Author: nunomaduro

config/database.php

@@ -121,7 +121,10 @@
     |
     */
 
-    'migrations' => 'migrations',
+    'migrations' => [
+        'table' => 'migrations',
+        'update_date_on_publish' => true,
+    ],
 
     /*
     |--------------------------------------------------------------------------

src/Illuminate/Database/Console/DumpCommand.php

@@ -69,8 +69,12 @@ public function handle(ConnectionResolverInterface $connections, Dispatcher $dis
      */
     protected function schemaState(Connection $connection)
     {
+        $migrations = Config::get('database.migrations', 'migrations');
+
+        $migrationTable = is_array($migrations) ? ($migrations['table'] ?? 'migrations') : $migrations;
+
         return $connection->getSchemaState()
-                ->withMigrationTable($connection->getTablePrefix().Config::get('database.migrations', 'migrations'))
+                ->withMigrationTable($connection->getTablePrefix().$migrationTable)
                 ->handleOutputUsing(function ($type, $buffer) {
                     $this->output->write($buffer);
                 });

src/Illuminate/Database/MigrationServiceProvider.php

@@ -59,7 +59,9 @@ public function register()
     protected function registerRepository()
     {
         $this->app->singleton('migration.repository', function ($app) {
-            $table = $app['config']['database.migrations'];
+            $migrations = $app['config']['database.migrations'];
+
+            $table = is_array($migrations) ? ($migrations['table'] ?? null) : $migrations;
 
             return new DatabaseMigrationRepository($app['db'], $table);
         });

src/Illuminate/Foundation/Testing/DatabaseTruncation.php

@@ -130,9 +130,11 @@ protected function connectionsToTruncate(): array
      */
     protected function exceptTables(?string $connectionName): array
     {
-        if (property_exists($this, 'exceptTables')) {
-            $migrationsTable = $this->app['config']->get('database.migrations');
+        $migrations = $this->app['config']->get('database.migrations');
+
+        $migrationsTable = is_array($migrations) ? ($migrations['table'] ?? null) : $migrations;
 
+        if (property_exists($this, 'exceptTables')) {
             if (array_is_list($this->exceptTables ?? [])) {
                 return array_merge(
                     $this->exceptTables ?? [],
@@ -146,7 +148,7 @@ protected function exceptTables(?string $connectionName): array
             );
         }
 
-        return [$this->app['config']->get('database.migrations')];
+        return [$migrationsTable];
     }
 
     /**

src/Illuminate/Support/ServiceProvider.php

@@ -285,7 +285,9 @@ protected function publishesMigrations(array $paths, $groups = null)
     {
         $this->publishes($paths, $groups);
 
-        static::$publishableMigrationPaths = array_unique(array_merge(static::$publishableMigrationPaths, array_keys($paths)));
+        if ($this->app->config->get('database.migrations.update_date_on_publish', false)) {
+            static::$publishableMigrationPaths = array_unique(array_merge(static::$publishableMigrationPaths, array_keys($paths)));
+        }
     }
 
     /**

tests/Support/SupportServiceProviderTest.php

@@ -2,19 +2,29 @@
 
 namespace Illuminate\Tests\Support;
 
+use Illuminate\Config\Repository as Config;
 use Illuminate\Foundation\Application;
 use Illuminate\Support\ServiceProvider;
 use Mockery as m;
 use PHPUnit\Framework\TestCase;
 
 class SupportServiceProviderTest extends TestCase
 {
+    protected $app;
+
     protected function setUp(): void
     {
         ServiceProvider::$publishes = [];
         ServiceProvider::$publishGroups = [];
 
-        $app = m::mock(Application::class)->makePartial();
+        $this->app = $app = m::mock(Application::class)->makePartial();
+        $config = m::mock(Config::class)->makePartial();
+
+        $config = new Config();
+
+        $app->instance('config', $config);
+        $config->set('database.migrations.update_date_on_publish', true);
+
         $one = new ServiceProviderForTestingOne($app);
         $one->boot();
         $two = new ServiceProviderForTestingTwo($app);
@@ -119,6 +129,37 @@ public function testMultipleTaggedAssetsAreMergedCorrectly()
         ];
         $this->assertEquals($expected, $toPublish, 'Service provider does not return expected set of published tagged paths.');
     }
+
+    public function testPublishesMigrations()
+    {
+        $serviceProvider = new ServiceProviderForTestingOne($this->app);
+
+        (fn () => $this->publishesMigrations(['source/tagged/four' => 'destination/tagged/four'], 'tag_four'))
+            ->call($serviceProvider);
+
+        $this->assertContains('source/tagged/four', ServiceProvider::publishableMigrationPaths());
+
+        $this->app->config->set('database.migrations.update_date_on_publish', false);
+
+        (fn () => $this->publishesMigrations(['source/tagged/five' => 'destination/tagged/five'], 'tag_four'))
+            ->call($serviceProvider);
+
+        $this->assertNotContains('source/tagged/five', ServiceProvider::publishableMigrationPaths());
+
+        $this->app->config->set('database.migrations', 'migrations');
+
+        (fn () => $this->publishesMigrations(['source/tagged/five' => 'destination/tagged/five'], 'tag_four'))
+            ->call($serviceProvider);
+
+        $this->assertNotContains('source/tagged/five', ServiceProvider::publishableMigrationPaths());
+
+        $this->app->config->set('database.migrations', null);
+
+        (fn () => $this->publishesMigrations(['source/tagged/five' => 'destination/tagged/five'], 'tag_four'))
+            ->call($serviceProvider);
+
+        $this->assertNotContains('source/tagged/five', ServiceProvider::publishableMigrationPaths());
+    }
 }
 
 class ServiceProviderForTestingOne extends ServiceProvider