nWidart / laravel-modules

src/Repository.php

Doc Comments +7 added lines, -7 removed lines

@@ -92,7 +92,7 @@ 
     /**
      * Get scanned modules paths.
      *
-     * @return array
+     * @return string[]
      */
     public function getScanPaths()
     {
@@ -402,7 +402,7 @@ 
     /**
      * Get module path for a specific module.
      *
-     * @param $module
+     * @param string $module
      *
      * @return string
      */
@@ -430,7 +430,7 @@ 
     /**
      * Get a specific config data from a configuration file.
      *
-     * @param $key
+     * @param string $key
      *
      * @param null $default
      * @return mixed
@@ -471,7 +471,7 @@ 
     /**
      * Get module used for cli session.
      *
-     * @return string
+     * @return Module
      */
     public function getUsedNow()
     {
@@ -481,7 +481,7 @@ 
     /**
      * Get used now.
      *
-     * @return string
+     * @return Module
      */
     public function getUsed()
     {
@@ -555,7 +555,7 @@ 
      *
      * @param string $name
      *
-     * @return bool
+     * @return boolean|null
      */
     public function enable($name)
     {
@@ -567,7 +567,7 @@ 
      *
      * @param string $name
      *
-     * @return bool
+     * @return boolean|null
      */
     public function disable($name)
     {

Spacing +9 added lines, -9 removed lines

@@ -99,7 +99,7 @@ 
     {
         $paths = $this->paths;
 
-        $paths[] = $this->getPath() . '/*';
+        $paths[] = $this->getPath().'/*';
 
         if ($this->config('scan.enabled')) {
             $paths = array_merge($paths, $this->config('scan.paths'));
@@ -145,7 +145,7 @@ 
      */
     public function getCached()
     {
-        return $this->app['cache']->remember($this->config('cache.key'), $this->config('cache.lifetime'), function () {
+        return $this->app['cache']->remember($this->config('cache.key'), $this->config('cache.lifetime'), function() {
             return $this->toCollection()->toArray();
         });
     }
@@ -233,7 +233,7 @@ 
     {
         $modules = $this->enabled();
 
-        uasort($modules, function (Module $a, Module $b) use ($direction) {
+        uasort($modules, function(Module $a, Module $b) use ($direction) {
             if ($a->order == $b->order) {
                 return 0;
             }
@@ -382,9 +382,9 @@ 
     public function getModulePath($module)
     {
         try {
-            return $this->findOrFail($module)->getPath() . '/';
+            return $this->findOrFail($module)->getPath().'/';
         } catch (ModuleNotFoundException $e) {
-            return $this->getPath() . '/' . Str::studly($module) . '/';
+            return $this->getPath().'/'.Str::studly($module).'/';
         }
     }
 
@@ -397,7 +397,7 @@ 
      */
     public function assetPath($module)
     {
-        return $this->config('paths.assets') . '/' . $module;
+        return $this->config('paths.assets').'/'.$module;
     }
 
     /**
@@ -410,7 +410,7 @@ 
      */
     public function config($key, $default = null)
     {
-        return $this->app['config']->get('modules.' . $key, $default);
+        return $this->app['config']->get('modules.'.$key, $default);
     }
 
     /**
@@ -501,9 +501,9 @@ 
         }
         list($name, $url) = explode(':', $asset);
 
-        $baseUrl = str_replace(public_path() . DIRECTORY_SEPARATOR, '', $this->getAssetsPath());
+        $baseUrl = str_replace(public_path().DIRECTORY_SEPARATOR, '', $this->getAssetsPath());
 
-        $url = $this->app['url']->asset($baseUrl . "/{$name}/" . $url);
+        $url = $this->app['url']->asset($baseUrl."/{$name}/".$url);
 
         return str_replace(['http://', 'https://'], '//', $url);
     }

src/Support/Migrations/NameParser.php

Doc Comments +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
     /**
      * Get matches data from regex.
      *
-     * @return array
+     * @return string[]
      */
     public function getMatches()
     {

src/Migrations/Migrator.php

Spacing +4 added lines, -4 removed lines

@@ -83,7 +83,7 @@ 
      */
     public function getMigrations($reverse = false)
     {
-        $files = $this->laravel['files']->glob($this->getPath() . '/*_*.php');
+        $files = $this->laravel['files']->glob($this->getPath().'/*_*.php');
 
         // Once we have the array of files in the directory we will just remove the
         // extension and take the basename of the file which is all we need when
@@ -92,7 +92,7 @@ 
             return array();
         }
 
-        $files = array_map(function ($file) {
+        $files = array_map(function($file) {
             return str_replace('.php', '', basename($file));
 
         }, $files);
@@ -210,7 +210,7 @@ 
     {
         $path = $this->getPath();
         foreach ($files as $file) {
-            $this->laravel['files']->requireOnce($path . '/' . $file . '.php');
+            $this->laravel['files']->requireOnce($path.'/'.$file.'.php');
         }
     }
 
@@ -289,7 +289,7 @@ 
 
         $result = $query->orderBy('migration', 'desc')->get();
 
-        return collect($result)->map(function ($item) {
+        return collect($result)->map(function($item) {
             return (array) $item;
         })->pluck('migration');
     }

Doc Comments +1 added lines, -1 removed lines

@@ -242,7 +242,7 @@
      *
      * @param string $migration
      *
-     * @return mixed
+     * @return boolean
      */
     public function log($migration)
     {

src/Commands/MigrateRollbackCommand.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
         }
 
         foreach ($this->module->getOrdered($this->option('direction')) as $module) {
-            $this->line('Running for module: <info>' . $module->getName() . '</info>');
+            $this->line('Running for module: <info>'.$module->getName().'</info>');
 
             $this->rollback($module);
         }

src/Commands/MigrateResetCommand.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
         }
 
         foreach ($this->module->getOrdered($this->option('direction')) as $module) {
-            $this->line('Running for module: <info>' . $module->getName() . '</info>');
+            $this->line('Running for module: <info>'.$module->getName().'</info>');
 
             $this->reset($module);
         }

src/Commands/GeneratorCommand.php

Spacing +3 added lines, -3 removed lines

@@ -86,11 +86,11 @@
 
         $namespace = $this->laravel['modules']->config('namespace');
 
-        $namespace .= '\\' . $module->getStudlyName();
+        $namespace .= '\\'.$module->getStudlyName();
 
-        $namespace .= '\\' . $this->getDefaultNamespace();
+        $namespace .= '\\'.$this->getDefaultNamespace();
 
-        $namespace .= '\\' . $extra;
+        $namespace .= '\\'.$extra;
 
         return trim($namespace, '\\');
     }

src/Json.php

Spacing +2 added lines, -2 removed lines

@@ -123,14 +123,14 @@
 
         // any JSON parsing errors should throw an exception
         if (json_last_error() > 0) {
-            throw new InvalidJsonException('Error processing file: ' . $this->getPath() . '. Error: ' . json_last_error_msg());
+            throw new InvalidJsonException('Error processing file: '.$this->getPath().'. Error: '.json_last_error_msg());
         }
 
         if (config('modules.cache.enabled') === false) {
             return $attributes;
         }
 
-        return app('cache')->remember($this->getPath(), config('modules.cache.lifetime'), function () use ($attributes) {
+        return app('cache')->remember($this->getPath(), config('modules.cache.lifetime'), function() use ($attributes) {
             return $attributes;
         });
     }

src/Support/Migrations/SchemaParser.php

Spacing +10 added lines, -10 removed lines

@@ -144,7 +144,7 @@ 
      */
     public function createField($column, $attributes, $type = 'add')
     {
-        $results = "\t\t\t" . '$table';
+        $results = "\t\t\t".'$table';
 
         foreach ($attributes as $key => $field) {
             if (in_array($column, $this->relationshipKeys)) {
@@ -154,7 +154,7 @@ 
             }
         }
 
-        return $results . ';' . PHP_EOL;
+        return $results.';'.PHP_EOL;
     }
 
     /**
@@ -168,7 +168,7 @@ 
      */
     protected function addRelationColumn($key, $field, $column)
     {
-        $relatedColumn = snake_case(class_basename($field)) . '_id';
+        $relatedColumn = snake_case(class_basename($field)).'_id';
 
         $method = 'integer';
 
@@ -187,18 +187,18 @@ 
     protected function addColumn($key, $field, $column)
     {
         if ($this->hasCustomAttribute($column)) {
-            return '->' . $field;
+            return '->'.$field;
         }
 
         if ($key == 0) {
-            return '->' . $field . "('" . $column . "')";
+            return '->'.$field."('".$column."')";
         }
 
         if (str_contains($field, '(')) {
-            return '->' . $field;
+            return '->'.$field;
         }
 
-        return '->' . $field . '()';
+        return '->'.$field.'()';
     }
 
     /**
@@ -213,10 +213,10 @@ 
     protected function removeColumn($key, $field, $column)
     {
         if ($this->hasCustomAttribute($column)) {
-            return '->' . $field;
+            return '->'.$field;
         }
 
-        return '->dropColumn(' . "'" . $column . "')";
+        return '->dropColumn('."'".$column."')";
     }
 
     /**
@@ -241,7 +241,7 @@ 
      */
     public function getAttributes($column, $schema)
     {
-        $fields = str_replace($column . ':', '', $schema);
+        $fields = str_replace($column.':', '', $schema);
 
         return $this->hasCustomAttribute($column) ? $this->getCustomAttribute($column) : explode(':', $fields);
     }

src/Commands/MigrateCommand.php

Spacing +2 added lines, -2 removed lines

@@ -47,7 +47,7 @@ 
         }
 
         foreach ($this->module->getOrdered($this->option('direction')) as $module) {
-            $this->line('Running for module: <info>' . $module->getName() . '</info>');
+            $this->line('Running for module: <info>'.$module->getName().'</info>');
 
             $this->migrate($module);
         }
@@ -63,7 +63,7 @@ 
         $path = str_replace(base_path(), '', (new Migrator($module))->getPath());
 
         if ($this->option('subpath')) {
-            $path = $path . "/" . $this->option("subpath");
+            $path = $path."/".$this->option("subpath");
         }
 
         $this->call('migrate', [