FIX added backtick in sql query to get the table columns. Added the c…

…reation of directories recursively if it does not find the livewire dir required. Added a fix for this to work on any laravel 9 version.
flightsadmin · Feb 4, 2023 · c714c36 · c714c36
1 parent f1e45ea
commit c714c36
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 20 deletions.
diff --git a/.gitignore b/.gitignore
@@ -3,4 +3,5 @@ composer.lock
 docs
 vendor
 coverage
-node_modules
+node_modules
+.idea
diff --git a/composer.json b/composer.json
@@ -17,7 +17,7 @@
     ],
     "require": {
         "php": "^7|^8",
-        "laravel/framework": "^6|^7|^8|<9.19",
+        "laravel/framework": "^6|^7|^8|^9",
         "laravel/ui": "^2|^3|^4",
         "livewire/livewire": "^2"
     },

diff --git a/src/Commands/LivewireGeneratorCommand.php b/src/Commands/LivewireGeneratorCommand.php
@@ -64,7 +64,7 @@ abstract class LivewireGeneratorCommand extends Command
      * Controller Namespace.
      * @var string
      */
-    protected $controllerNamespace = 'App\Http\Controllers';  
+    protected $controllerNamespace = 'App\Http\Controllers';
 	/**
      * Controller Namespace.
      * @var string
@@ -133,6 +133,7 @@ protected function makeDirectory($path)
      */
     protected function write($path, $content)
     {
+        $this->files->makeDirectory(dirname($path), 0755, true, true);
         $this->files->put($path, $content);
     }
 
@@ -181,11 +182,11 @@ private function _getSpace($no = 1)
     protected function _getMigrationPath($name)
     {
         return base_path("database/migrations/". date('Y-m-d_His') ."_create_". Str::lower(Str::plural($name)) ."_table.php");
-    } 
+    }
     protected function _getFactoryPath($name)
     {
         return base_path("database/factories/{$name}Factory.php");
-    } 
+    }
 
 	/**
      * @param $name
@@ -337,7 +338,7 @@ protected function buildLayout(): void
     protected function getColumns()
     {
         if (empty($this->tableColumns)) {
-            $this->tableColumns = DB::select('SHOW COLUMNS FROM ' . $this->table);
+            $this->tableColumns = DB::select('SHOW COLUMNS FROM `' . $this->table.'`');
         }
 
         return $this->tableColumns;
@@ -358,7 +359,7 @@ protected function getFilteredColumns()
         return array_filter($columns, function ($value) use ($unwanted) {
             return !in_array($value, $unwanted);
         });
-    }   
+    }
 
     /**
      * Make model attributes/replacements.
@@ -421,7 +422,7 @@ protected function modelReplacements()
 
             // CSV format
             return implode(', ', $filterColumns);
-        };      
+        };
 
 		$resetfields = function () {
 
@@ -436,8 +437,8 @@ protected function modelReplacements()
 
             // CSV format
             return implode('', $filterColumns);
-        };		
-		
+        };
+
 		$addfields = function () {
 
             /** @var array $filterColumns Exclude the unwanted columns */
@@ -450,8 +451,8 @@ protected function modelReplacements()
 
             // CSV format
             return implode(',', $filterColumns);
-        };		
-		
+        };
+
 		$keyWord = function () {
 
             /** @var array $filterColumns Exclude the unwanted columns */
@@ -464,7 +465,7 @@ protected function modelReplacements()
 
             // CSV format
             return implode('', $filterColumns);
-        };	
+        };
 
 		$factoryfields = function () {
 
@@ -479,7 +480,7 @@ protected function modelReplacements()
             // CSV format
             return implode('', $filterColumns);
         };
-		
+
 		$editfields = function () {
 
             /** @var array $filterColumns Exclude the unwanted columns */

diff --git a/src/ModelGenerator.php b/src/ModelGenerator.php
@@ -116,13 +116,13 @@ private function _getTableRelations()
     {
         $db = DB::getDatabaseName();
         $sql = <<<SQL
-SELECT TABLE_NAME ref_table, COLUMN_NAME foreign_key, REFERENCED_COLUMN_NAME local_key, '1' ref 
-  FROM INFORMATION_SCHEMA.KEY_COLUMN_USAGE 
+SELECT TABLE_NAME ref_table, COLUMN_NAME foreign_key, REFERENCED_COLUMN_NAME local_key, '1' ref
+  FROM INFORMATION_SCHEMA.KEY_COLUMN_USAGE
   WHERE REFERENCED_TABLE_NAME = '$this->table' AND TABLE_SCHEMA = '$db'
 UNION
-SELECT REFERENCED_TABLE_NAME ref_table, REFERENCED_COLUMN_NAME foreign_key, COLUMN_NAME local_key, '0' ref 
-  FROM INFORMATION_SCHEMA.KEY_COLUMN_USAGE 
-  WHERE TABLE_NAME = '$this->table' AND TABLE_SCHEMA = '$db' AND REFERENCED_TABLE_NAME IS NOT NULL 
+SELECT REFERENCED_TABLE_NAME ref_table, REFERENCED_COLUMN_NAME foreign_key, COLUMN_NAME local_key, '0' ref
+  FROM INFORMATION_SCHEMA.KEY_COLUMN_USAGE
+  WHERE TABLE_NAME = '$this->table' AND TABLE_SCHEMA = '$db' AND REFERENCED_TABLE_NAME IS NOT NULL
 
 ORDER BY ref_table ASC
 SQL;
@@ -135,6 +135,6 @@ private function _getTableRelations()
      */
     private function _getTableKeys($table)
     {
-        return DB::select("SHOW KEYS FROM {$table}");
+        return DB::select("SHOW KEYS FROM `{$table}`");
     }
 }