Elimuswift / db-exporter

src/DbExporter/Commands/CopyToRemoteCommand.php

Doc Comments +3 added lines

@@ -94,6 +94,9 @@
         }
     }
 
+    /**
+     * @param string $what
+     */
     private function upload($what)
     {
         $localPath = "{$what}Path";

Spacing +5 added lines, -5 removed lines

@@ -13,7 +13,7 @@ 
     protected $name = 'db-exporter:remote';
     protected $description = 'Command to copy the migrations and/or the seeds to a remote host.';
 
-    protected $ignoredFiles =  array('..', '.', '.gitkeep');
+    protected $ignoredFiles = array('..', '.', '.gitkeep');
 
     protected $migrationsPath;
     protected $seedsPath;
@@ -41,7 +41,7 @@ 
             foreach ($this->uploadedFiles as $type => $files) {
                 $this->info(ucfirst($type));
                 foreach ($files as $file) {
-                    $this->sectionMessage($type, $file .' uploaded.');
+                    $this->sectionMessage($type, $file . ' uploaded.');
                 }
             }
 
@@ -99,14 +99,14 @@ 
         $localPath = "{$what}Path";
 
         $dir = scandir($this->$localPath);
-        $remotePath = Config::get('db-exporter.remote.'.$what);
+        $remotePath = Config::get('db-exporter.remote.' . $what);
 
         // Prepare the progress bar
         $progress = $this->getHelperSet()->get('progress');
         $filesCount = count($dir) - count($this->ignoredFiles);
         $progress->start($this->output, $filesCount);
         $this->info(ucfirst($what));
-        foreach($dir as $file) {
+        foreach ($dir as $file) {
             if (in_array($file, $this->ignoredFiles)) {
                 continue;
             }
@@ -116,7 +116,7 @@ 
 
             // Copy the files
             SSH::into($this->getRemoteName())->put(
-                $this->$localPath .'/' . $file,
+                $this->$localPath . '/' . $file,
                 $remotePath . $file
             );
             $progress->advance();

Braces +3 added lines, -1 removed lines

@@ -79,7 +79,9 @@
         $options = $this->option();
         switch ($options) {
             case (($options['seeds'] === true) && ($options['migrations'] === true)):
-                if (!$this->upload('migrations')) return false;
+                if (!$this->upload('migrations')) {
+                    return false;
+                }
                 return $this->upload('seeds');
                 break;
             case $options['migrations'] === true:

src/DbExporter/DbExportHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getMigrationsFilePath()
     {

src/DbExporter/DbSeeding.php

Unused Use Statements +4 added lines, -2 removed lines

@@ -1,8 +1,10 @@
 <?php 
 namespace Elimuswift\DbExporter;
 
-use Config, DB;
-use Illuminate\Support\Str, File;
+use Config;
+use DB;
+use Illuminate\Support\Str;
+use File;
 
 class DbSeeding extends DbExporter
 {

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
 
         $filename = Str::camel($this->database) . "TableSeeder";
 
-        file_put_contents(Config::get('db-exporter.export_path.seeds')."/{$filename}.php", $seed);
+        file_put_contents(Config::get('db-exporter.export_path.seeds') . "/{$filename}.php", $seed);
     }
 
     /**
@@ -123,7 +123,7 @@ 
         $value = addslashes($value);
         if (is_numeric($value)) {
             return "                '{$prop}' => {$value},\n";
-        } elseif($value == '') {
+        } elseif ($value == '') {
             return "                '{$prop}' => NULL,\n";
         } else {
             return "                '{$prop}' => '{$value}',\n";

src/DbExporter/Commands/GeneratorCommand.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     protected function getDatabaseName()
     {
         $connType = Config::get('database.default');
-        $database = Config::get('database.connections.' .$connType );
+        $database = Config::get('database.connections.' . $connType);
 
         return $database['database'];
     }

src/DbExporter/Commands/SeedGeneratorCommand.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
     private function getFilename()
     {
         $filename = Str::camel($this->getDatabaseName()) . "TableSeeder";
-        return Config::get('db-exporter.export_path.seeds')."/{$filename}.php";
+        return Config::get('db-exporter.export_path.seeds') . "/{$filename}.php";
     }
 
     

src/DbExporter/DbMigrations.php

Indentation +12 added lines, -12 removed lines

@@ -177,11 +177,11 @@ 
 
             $tableIndexes = $this->getTableIndexes($value['table_name']);
             if (!is_null($tableIndexes) && count($tableIndexes)){
-            	foreach ($tableIndexes as $index) {
+                foreach ($tableIndexes as $index) {
                     if(Str::endsWith($index['Key_name'], '_index'))
-                	   $up .= '                $' . "table->index('" . $index['Key_name'] . "');\n";
+                        $up .= '                $' . "table->index('" . $index['Key_name'] . "');\n";
                     }
-            	}
+                }
 
             $up .= "            });\n\n";
             $Constraint = $ConstraintDown = "";
@@ -225,24 +225,24 @@ 
 
         // prevent of failure when no table
         if (!is_null($this->schema) && count($this->schema)) {
-	        foreach ($this->schema as $name => $values) {
-	            // check again for ignored tables
-	            if (in_array($name, self::$ignore)) {
-	                continue;
-	            }
-	            $upSchema .= "
+            foreach ($this->schema as $name => $values) {
+                // check again for ignored tables
+                if (in_array($name, self::$ignore)) {
+                    continue;
+                }
+                $upSchema .= "
 	    /**
 	     * Table: {$name}
 	     */
 	    {$values['up']}";
                 $upConstraint.="
                 {$values['constraint']}";
-                 $downConstraint.="
+                    $downConstraint.="
                 {$values['constraint_down']}";
 
-	            $downSchema .= "
+                $downSchema .= "
 	            {$values['down']}";
-	        }
+            }
         }
 
         // Grab the template

Spacing +9 added lines, -9 removed lines

@@ -62,7 +62,7 @@ 
 
         $schema = $this->compile();
         $filename = date('Y_m_d_His') . "_create_" . $this->database . "_database.php";
-        static::$filePath = Config::get('db-exporter.export_path.migrations')."{$filename}";
+        static::$filePath = Config::get('db-exporter.export_path.migrations') . "{$filename}";
 
         file_put_contents(static::$filePath, $schema);
 
@@ -176,9 +176,9 @@ 
             }
 
             $tableIndexes = $this->getTableIndexes($value['table_name']);
-            if (!is_null($tableIndexes) && count($tableIndexes)){
+            if (!is_null($tableIndexes) && count($tableIndexes)) {
             	foreach ($tableIndexes as $index) {
-                    if(Str::endsWith($index['Key_name'], '_index'))
+                    if (Str::endsWith($index['Key_name'], '_index'))
                 	   $up .= '                $' . "table->index('" . $index['Key_name'] . "');\n";
                     }
             	}
@@ -186,14 +186,14 @@ 
             $up .= "            });\n\n";
             $Constraint = $ConstraintDown = "";
             $tableConstraints = $this->getTableConstraints($value['table_name']);
-            if (!is_null($tableConstraints) && count($tableConstraints)){
+            if (!is_null($tableConstraints) && count($tableConstraints)) {
             $Constraint = $ConstraintDown = "
             Schema::table('{$value['table_name']}', function(Blueprint $" . "table) {\n";
             $tables = [];
                 foreach ($tableConstraints as $foreign) {
-                    if(!in_array($foreign->Field, $tables)){
-                        $ConstraintDown .= '                $' . "table->dropForeign('" . $foreign->Field. "');\n";
-                        $Constraint .= '                $' . "table->foreign('" . $foreign->Field. "')->references('" . $foreign->References . "')->on('" . $foreign->ON. "')->onDelete('" . $foreign->onDelete. "');\n";
+                    if (!in_array($foreign->Field, $tables)) {
+                        $ConstraintDown .= '                $' . "table->dropForeign('" . $foreign->Field . "');\n";
+                        $Constraint .= '                $' . "table->foreign('" . $foreign->Field . "')->references('" . $foreign->References . "')->on('" . $foreign->ON . "')->onDelete('" . $foreign->onDelete . "');\n";
                         $tables[$foreign->Field] = $foreign->Field;
                     }
                 }
@@ -235,9 +235,9 @@ 
 	     * Table: {$name}
 	     */
 	    {$values['up']}";
-                $upConstraint.="
+                $upConstraint .= "
                 {$values['constraint']}";
-                 $downConstraint.="
+                 $downConstraint .= "
                 {$values['constraint_down']}";
 
 	            $downSchema .= "

Braces +3 added lines, -2 removed lines

@@ -178,8 +178,9 @@
             $tableIndexes = $this->getTableIndexes($value['table_name']);
             if (!is_null($tableIndexes) && count($tableIndexes)){
             	foreach ($tableIndexes as $index) {
-                    if(Str::endsWith($index['Key_name'], '_index'))
-                	   $up .= '                $' . "table->index('" . $index['Key_name'] . "');\n";
+                    if(Str::endsWith($index['Key_name'], '_index')) {
+                                    	   $up .= '                $' . "table->index('" . $index['Key_name'] . "');\n";
+                    }
                     }
             	}
 

src/DbExporter/DbExportHandlerServiceProvider.php

Indentation +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
 
     public function register()
     {
-         $this->publishes([
+            $this->publishes([
             realpath(__DIR__ .'/../').'/config/db-exporter.php' => config_path('db-exporter.php'),
         ]);
 
@@ -37,7 +37,7 @@ 
             realpath(__DIR__ .'/../').'/config/db-exporter.php', 'db-exporter'
         );
 
-       $this->app->register(DbMigrationsServiceProvider::class);
+        $this->app->register(DbMigrationsServiceProvider::class);
 
         // Load the classes
         $this->loadClasses();

Spacing +3 added lines, -3 removed lines

@@ -30,11 +30,11 @@ 
     public function register()
     {
          $this->publishes([
-            realpath(__DIR__ .'/../').'/config/db-exporter.php' => config_path('db-exporter.php'),
+            realpath(__DIR__ . '/../') . '/config/db-exporter.php' => config_path('db-exporter.php'),
         ]);
 
         $this->mergeConfigFrom(
-            realpath(__DIR__ .'/../').'/config/db-exporter.php', 'db-exporter'
+            realpath(__DIR__ . '/../') . '/config/db-exporter.php', 'db-exporter'
         );
 
        $this->app->register(DbMigrationsServiceProvider::class);
@@ -74,7 +74,7 @@ 
     private function getDatabaseName()
     {
         $connType = Config::get('database.default');
-        $database = Config::get('database.connections.' .$connType );
+        $database = Config::get('database.connections.' . $connType);
 
         return $database['database'];
     }

src/DbExporter/Server.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
         $dir = scandir($localPath);
         $remotePath = Config::get('db-exporter.remote.' . $what);
 
-        foreach($dir as $file) {
+        foreach ($dir as $file) {
             if (in_array($file, $this->ignoredFiles)) {
                 continue;
             }
@@ -32,7 +32,7 @@ 
 
             // Copy the files
             SSH::into($this->getRemoteName())->put(
-                $localPath .'/' . $file,
+                $localPath . '/' . $file,
                 $remotePath . $file
             );
         }

src/DbExporter/DbMigrationsServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
         $this->app->singleton('DbMigrations', function()
         {
             $connType = Config::get('database.default');
-            $database = Config::get('database.connections.' .$connType );
+            $database = Config::get('database.connections.' . $connType);
             return new DbMigrations($database);
         });