nWidart / laravel-modules

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/GenerateNotificationCommand.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
 
         $mailPath = $this->laravel['modules']->config('paths.generator.notifications', 'Notifications');
 
-        return $path . $mailPath . '/' . $this->getFileName() . '.php';
+        return $path.$mailPath.'/'.$this->getFileName().'.php';
     }
 
     /**

Indentation +5 added lines, -5 removed lines

@@ -19,11 +19,11 @@
 
     protected $argumentName = 'name';
     
-     /**
-     * The console command description.
-     *
-     * @var string
-     */
+        /**
+         * The console command description.
+         *
+         * @var string
+         */
     protected $description = 'Generate new notification class for the specified 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/Commands/GenerateMailCommand.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
 
         $mailPath = $this->laravel['modules']->config('paths.generator.emails', 'Emails');
 
-        return $path . $mailPath . '/' . $this->getFileName() . '.php';
+        return $path.$mailPath.'/'.$this->getFileName().'.php';
     }
 
     /**

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/Module.php

Doc Comments +6 added lines, -6 removed lines

@@ -38,7 +38,7 @@ 
      *
      * @param Application $app
      * @param $name
-     * @param $path
+     * @param string $path
      */
     public function __construct(Application $app, $name, $path)
     {
@@ -182,7 +182,7 @@ 
     /**
      * Get json contents.
      *
-     * @param $file
+     * @param string $file
      *
      * @return Json
      */
@@ -211,7 +211,7 @@ 
     /**
      * Get a specific data from composer.json file by given the key.
      *
-     * @param $key
+     * @param string $key
      * @param null $default
      *
      * @return mixed
@@ -289,7 +289,7 @@ 
     /**
      * Determine whether the given status same with the current module status.
      *
-     * @param $status
+     * @param integer $status
      *
      * @return bool
      */
@@ -341,9 +341,9 @@ 
     /**
      * Set active state for current module.
      *
-     * @param $active
+     * @param integer $active
      *
-     * @return bool
+     * @return integer
      */
     public function setActive($active)
     {

Spacing +6 added lines, -6 removed lines

@@ -49,7 +49,7 @@ 
         parent::__construct($app);
         $this->name = $name;
         $this->path = realpath($path);
-        $this->moduleJson = new Json($this->getPath() . '/module.json', $this->app['files']);
+        $this->moduleJson = new Json($this->getPath().'/module.json', $this->app['files']);
     }
 
     /**
@@ -177,7 +177,7 @@ 
     {
         $lowerName = $this->getLowerName();
 
-        $langPath = $this->getPath() . "/Resources/lang";
+        $langPath = $this->getPath()."/Resources/lang";
 
         if (is_dir($langPath)) {
             $this->loadTranslationsFrom($langPath, $lowerName);
@@ -197,7 +197,7 @@ 
             return $this->moduleJson;
         }
 
-        return new Json($this->getPath() . '/' . $file, $this->app['files']);
+        return new Json($this->getPath().'/'.$file, $this->app['files']);
     }
 
     /**
@@ -247,7 +247,7 @@ 
      */
     protected function fireEvent($event)
     {
-        $this->app['events']->fire(sprintf('modules.%s.' . $event, $this->getLowerName()), [$this]);
+        $this->app['events']->fire(sprintf('modules.%s.'.$event, $this->getLowerName()), [$this]);
     }
 
     /**
@@ -277,7 +277,7 @@ 
     protected function registerFiles()
     {
         foreach ($this->get('files', []) as $file) {
-            include $this->path . '/' . $file;
+            include $this->path.'/'.$file;
         }
     }
 
@@ -398,7 +398,7 @@ 
      */
     public function getExtraPath($path)
     {
-        return $this->getPath() . '/' . $path;
+        return $this->getPath().'/'.$path;
     }
 
     /**