nWidart / laravel-modules

src/Commands/MigrationCommand.php

Doc Comments +2 added lines, -2 removed lines

@@ -67,7 +67,7 @@ 
     /**
      * @throws InvalidMigrationNameException
      *
-     * @return mixed
+     * @return Stub
      */
     protected function getTemplateContents()
     {
@@ -105,7 +105,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     protected function getDestinationFilePath()
     {

Spacing +2 added lines, -2 removed lines

@@ -113,7 +113,7 @@ 
 
         $generatorPath = $this->laravel['modules']->config('paths.generator.migration');
 
-        return $path . $generatorPath . '/' . $this->getFileName() . '.php';
+        return $path.$generatorPath.'/'.$this->getFileName().'.php';
     }
 
     /**
@@ -121,7 +121,7 @@ 
      */
     private function getFileName()
     {
-        return date('Y_m_d_His_') . $this->getSchemaName();
+        return date('Y_m_d_His_').$this->getSchemaName();
     }
 
     /**

src/Commands/ModelCommand.php

Doc Comments +3 added lines, -3 removed lines

@@ -59,7 +59,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     protected function getTemplateContents()
     {
@@ -78,7 +78,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     protected function getDestinationFilePath()
     {
@@ -90,7 +90,7 @@ 
     }
 
     /**
-     * @return mixed|string
+     * @return string
      */
     private function getModelName()
     {

Spacing +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
 
         $seederPath = $this->laravel['modules']->config('paths.generator.model');
 
-        return $path . $seederPath . '/' . $this->getModelName() . '.php';
+        return $path.$seederPath.'/'.$this->getModelName().'.php';
     }
 
     /**

src/Commands/SeedMakeCommand.php

Doc Comments +2 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     protected function getTemplateContents()
     {
@@ -70,7 +70,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     protected function getDestinationFilePath()
     {

Spacing +2 added lines, -2 removed lines

@@ -81,7 +81,7 @@ 
 
         $seederPath = $this->laravel['modules']->config('paths.generator.seeder');
 
-        return $path . $seederPath . '/' . $this->getSeederName() . '.php';
+        return $path.$seederPath.'/'.$this->getSeederName().'.php';
     }
 
     /**
@@ -93,6 +93,6 @@ 
     {
         $end = $this->option('master') ? 'DatabaseSeeder' : 'TableSeeder';
 
-        return Str::studly($this->argument('name')) . $end;
+        return Str::studly($this->argument('name')).$end;
     }
 }

src/Generators/ModuleGenerator.php

Doc Comments +1 added lines, -1 removed lines

@@ -335,7 +335,7 @@
      *
      * @param $stub
      *
-     * @return Stub
+     * @return string
      */
     protected function getStubContents($stub)
     {

Spacing +7 added lines, -7 removed lines

@@ -271,7 +271,7 @@ 
     public function generateFolders()
     {
         foreach ($this->getFolders() as $folder) {
-            $path = $this->module->getModulePath($this->getName()) . '/' . $folder;
+            $path = $this->module->getModulePath($this->getName()).'/'.$folder;
 
             $this->filesystem->makeDirectory($path, 0755, true);
 
@@ -286,7 +286,7 @@ 
      */
     public function generateGitKeep($path)
     {
-        $this->filesystem->put($path . '/.gitkeep', '');
+        $this->filesystem->put($path.'/.gitkeep', '');
     }
 
     /**
@@ -295,7 +295,7 @@ 
     public function generateFiles()
     {
         foreach ($this->getFiles() as $stub => $file) {
-            $path = $this->module->getModulePath($this->getName()) . $file;
+            $path = $this->module->getModulePath($this->getName()).$file;
 
             if (!$this->filesystem->isDirectory($dir = dirname($path))) {
                 $this->filesystem->makeDirectory($dir, 0775, true);
@@ -319,13 +319,13 @@ 
         ]);
 
         $this->console->call('module:make-provider', [
-            'name' => $this->getName() . 'ServiceProvider',
+            'name' => $this->getName().'ServiceProvider',
             'module' => $this->getName(),
             '--master' => true,
         ]);
 
         $this->console->call('module:make-controller', [
-            'controller' => $this->getName() . 'Controller',
+            'controller' => $this->getName().'Controller',
             'module' => $this->getName(),
         ]);
     }
@@ -340,7 +340,7 @@ 
     protected function getStubContents($stub)
     {
         return (new Stub(
-            '/' . $stub . '.stub',
+            '/'.$stub.'.stub',
             $this->getReplacement($stub))
         )->render();
     }
@@ -373,7 +373,7 @@ 
         $replaces = [];
 
         foreach ($keys as $key) {
-            if (method_exists($this, $method = 'get' . ucfirst(studly_case(strtolower($key))) . 'Replacement')) {
+            if (method_exists($this, $method = 'get'.ucfirst(studly_case(strtolower($key))).'Replacement')) {
                 $replaces[$key] = call_user_func([$this, $method]);
             } else {
                 $replaces[$key] = null;

src/LaravelModulesServiceProvider.php

Doc Comments +1 added lines, -1 removed lines

@@ -107,7 +107,7 @@
     /**
      * Get the services provided by the provider.
      *
-     * @return array
+     * @return string[]
      */
     public function provides()
     {

Spacing +4 added lines, -4 removed lines

@@ -55,8 +55,8 @@ 
      */
     public function setupStubPath()
     {
-        $this->app->booted(function ($app) {
-            Stub::setBasePath(__DIR__ . '/Commands/stubs');
+        $this->app->booted(function($app) {
+            Stub::setBasePath(__DIR__.'/Commands/stubs');
 
             if ($app['modules']->config('stubs.enabled') === true) {
                 Stub::setBasePath($app['modules']->config('stubs.path'));
@@ -69,7 +69,7 @@ 
      */
     protected function registerNamespaces()
     {
-        $configPath = __DIR__ . '/../config/config.php';
+        $configPath = __DIR__.'/../config/config.php';
         $this->mergeConfigFrom($configPath, 'modules');
         $this->publishes([
             $configPath => config_path('modules.php'),
@@ -97,7 +97,7 @@ 
      */
     protected function registerServices()
     {
-        $this->app->singleton('modules', function ($app) {
+        $this->app->singleton('modules', function($app) {
             $path = $app['config']->get('modules.paths.modules');
 
             return new Repository($app, $path);

Unused Use Statements -5 removed lines

@@ -2,12 +2,7 @@
 
 namespace Nwidart\Modules;
 
-use Collective\Html\FormFacade;
-use Collective\Html\HtmlFacade;
-use Collective\Html\HtmlServiceProvider;
-use Illuminate\Foundation\AliasLoader;
 use Illuminate\Support\ServiceProvider;
-use Nwidart\Modules\Facades\Module;
 use Nwidart\Modules\Providers\BootstrapServiceProvider;
 use Nwidart\Modules\Providers\ConsoleServiceProvider;
 use Nwidart\Modules\Providers\ContractsServiceProvider;

src/Migrations/Migrator.php

Doc Comments +1 added lines, -1 removed lines

@@ -248,7 +248,7 @@
      *
      * @param array $migrations
      *
-     * @return array
+     * @return \Illuminate\Support\Collection
      */
     public function getLast($migrations)
     {

Spacing +4 added lines, -4 removed lines

@@ -51,7 +51,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
@@ -60,7 +60,7 @@ 
             return array();
         }
 
-        $files = array_map(function ($file) {
+        $files = array_map(function($file) {
             return str_replace('.php', '', basename($file));
 
         }, $files);
@@ -179,7 +179,7 @@ 
         $path = $this->getPath();
 
         foreach ($files as $file) {
-            $this->laravel['files']->requireOnce($path . '/' . $file . '.php');
+            $this->laravel['files']->requireOnce($path.'/'.$file.'.php');
         }
     }
 
@@ -259,7 +259,7 @@ 
 
         $result = $query->orderBy('migration', 'desc')->get();
 
-        return collect($result)->map(function ($item) {
+        return collect($result)->map(function($item) {
             return (array) $item;
         })->lists('migration');
     }

src/Module.php

Doc Comments +8 added lines, -7 removed lines

@@ -34,8 +34,8 @@ 
      * The constructor.
      *
      * @param Application $app
-     * @param $name
-     * @param $path
+     * @param string $name
+     * @param string $path
      */
     public function __construct(Application $app, $name, $path)
     {
@@ -167,6 +167,7 @@ 
     /**
      * Get json contents.
      *
+     * @param string $file
      * @return Json
      */
     public function json($file = null)
@@ -194,7 +195,7 @@ 
     /**
      * Get a specific data from composer.json file by given the key.
      *
-     * @param $key
+     * @param string $key
      * @param null $default
      *
      * @return mixed
@@ -272,7 +273,7 @@ 
     /**
      * Determine whether the given status same with the current module status.
      *
-     * @param $status
+     * @param integer $status
      *
      * @return bool
      */
@@ -324,9 +325,9 @@ 
     /**
      * Set active state for current module.
      *
-     * @param $active
+     * @param integer $active
      *
-     * @return bool
+     * @return integer
      */
     public function setActive($active)
     {
@@ -336,7 +337,7 @@ 
     /**
      * Disable the current module.
      *
-     * @return bool
+     * @return boolean|null
      */
     public function disable()
     {

Spacing +5 added lines, -5 removed lines

@@ -159,7 +159,7 @@ 
     {
         $lowerName = $this->getLowerName();
 
-        $langPath = $this->getPath() . "/Resources/lang";
+        $langPath = $this->getPath()."/Resources/lang";
 
         if (is_dir($langPath)) {
             $this->loadTranslationsFrom($langPath, $lowerName);
@@ -177,7 +177,7 @@ 
             $file = 'module.json';
         }
 
-        return new Json($this->getPath() . '/' . $file, $this->app['files']);
+        return new Json($this->getPath().'/'.$file, $this->app['files']);
     }
 
     /**
@@ -227,7 +227,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]);
     }
 
     /**
@@ -257,7 +257,7 @@ 
     protected function registerFiles()
     {
         foreach ($this->get('files', []) as $file) {
-            include $this->path . '/' . $file;
+            include $this->path.'/'.$file;
         }
     }
 
@@ -380,7 +380,7 @@ 
      */
     public function getExtraPath($path)
     {
-        return $this->getPath() . '/' . $path;
+        return $this->getPath().'/'.$path;
     }
 
     /**

src/Publishing/Publisher.php

Doc Comments +1 added lines, -1 removed lines

@@ -88,7 +88,7 @@
     /**
      * Get module instance.
      *
-     * @return \Nwidart\Modules\Module
+     * @return string
      */
     public function getModule()
     {

src/Collection.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
      */
     public function toArray()
     {
-        return array_map(function ($value) {
+        return array_map(function($value) {
             if ($value instanceof Module) {
                 return $value->json()->getAttributes();
             }