Fix the migration class for Laravel 5.7

Author: MikeAlhayek

src/Commands/Framework/CreateModelCommand.php

@@ -500,7 +500,8 @@ protected function getAccessors(array $fields = null)
                 $accessors[] = $this->getAccessor($field, $content);
             }
 
-            if ($field->isDateOrTime()) {
+            if ($field->isDateOrTime() && !$field->isDate) {
+                // We should not create accessor for a datetime field that is casted to Carbon object
                 $content = $this->getStubContent('model-accessor-datetime');
                 $this->replaceDateFormat($content, $field->dateFormat);
 

src/Commands/Migrations/MigrateAllCommand.php

@@ -3,6 +3,7 @@
 namespace CrestApps\CodeGenerator\Commands\Migrations;
 
 use CrestApps\CodeGenerator\Commands\Bases\MigrationCommandBase;
+use CrestApps\CodeGenerator\Support\Helpers;
 use Illuminate\Console\ConfirmableTrait;
 use Illuminate\Database\Migrations\Migrator;
 
@@ -42,19 +43,31 @@ public function handle()
 
         $this->prepareDatabase();
 
-        // Next, we will check to see if a path option has been defined. If it has
-        // we will use the path relative to the root of this installation folder
-        // so that migrations may be run for any path within the applications.
-        $this->migrator->run($this->getMigrationPaths(), [
-            'pretend' => $this->option('pretend'),
-            'step' => $this->option('step'),
-        ]);
+        if (Helpers::isNewerThanOrEqualTo('5.7')) {
+            // Next, we will check to see if a path option has been defined. If it has
+            // we will use the path relative to the root of this installation folder
+            // so that migrations may be run for any path within the applications.
 
-        // Once the migrator has run we will grab the note output and send it out to
-        // the console screen, since the migrator itself functions without having
-        // any instances of the OutputInterface contract passed into the class.
-        foreach ($this->migrator->getNotes() as $note) {
-            $this->output->writeln($note);
+            $this->migrator->setOutput($this->output)
+                ->run($this->getMigrationPaths(), [
+                    'pretend' => $this->option('pretend'),
+                    'step' => $this->option('step'),
+                ]);
+        } else {
+            // Next, we will check to see if a path option has been defined. If it has
+            // we will use the path relative to the root of this installation folder
+            // so that migrations may be run for any path within the applications.
+            $this->migrator->run($this->getMigrationPaths(), [
+                'pretend' => $this->option('pretend'),
+                'step' => $this->option('step'),
+            ]);
+
+            // Once the migrator has run we will grab the note output and send it out to
+            // the console screen, since the migrator itself functions without having
+            // any instances of the OutputInterface contract passed into the class.
+            foreach ($this->migrator->getNotes() as $note) {
+                $this->output->writeln($note);
+            }
         }
 
         // Finally, if the "seed" option has been given, we will re-run the database

src/Traits/Migration.php

@@ -101,7 +101,7 @@ protected function getRan()
      */
     protected function getAlterMigrationName($name, $count)
     {
-        $filename = sprintf('%s_alter_%s_table_%s.php', date('Y_m_d_His'), strtolower($name), $count);
+        $filename = sprintf('%s_alter_%s_%s_table.php', date('Y_m_d_His'), strtolower($name), $count);
 
         return Str::postfix($filename, '.php');
     }
@@ -142,6 +142,6 @@ protected function makeCreateTableClassName($tableName)
      */
     protected function makeAlterTableClassName($tableName, $id)
     {
-        return sprintf('Alter%sTable%s', studly_case($tableName), $id);
+        return sprintf('Alter%s%sTable', studly_case($tableName), $id);
     }
 }

templates/default/model-accessor-datetime.stub

@@ -1 +1 @@
-return \DateTime::createFromFormat('[% date_format %]', $value);
+return \DateTime::createFromFormat($this->getDateFormat(), $value)->format('[% date_format %]');

templates/default/model-mutator-datetime.stub

@@ -1 +1 @@
-$this->attributes['[% field_name %]'] = !empty($value) ? \DateTime::createFromFormat($this->getDateFormat(), $value) : null;
+$this->attributes['[% field_name %]'] = !empty($value) ? \DateTime::createFromFormat('[% date_format %]', $value) : null;