You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Database\Console\Migrations\MigrateMakeCommand::writeMigration($name, $table, $create) is sometimes called with $create equals null, whereas it should be false. This might lead to future bugs and ideally should be fixed.
This isn't caught by the tests, also because of the loose typing. If you look for example at DatabaseMigrationMakeCommandTest::testBasicCreateDumpsAutoload, line 23:
Database\Console\Migrations\MigrateMakeCommand::writeMigration($name, $table, $create)
is sometimes called with$create
equalsnull
, whereas it should befalse
. This might lead to future bugs and ideally should be fixed.This isn't caught by the tests, also because of the loose typing. If you look for example at
DatabaseMigrationMakeCommandTest::testBasicCreateDumpsAutoload
, line 23:but in fact it receives
(<path>, null, NULL)
.The bug is because in
Database\Console\Migrations\MigrateMakeCommand::fire
, line 70 gives NULL:The text was updated successfully, but these errors were encountered: