mnabialek / laravel-modular

src/Console/Traits/ModuleCreator.php

Doc Comments +3 added lines

@@ -85,6 +85,9 @@
         return $this->option('group') ?: $this->laravel['modular.config']->stubsDefaultGroup();
     }
 
+    /**
+     * @return string
+     */
     protected function getFilesStubGroup()
     {
         return $this->option('group') ?: $this->laravel['modular.config']->filesStubsDefaultGroup();

Spacing +2 added lines, -2 removed lines

@@ -104,7 +104,7 @@ 
         if ($directories->isEmpty()) {
             $this->warn("[Module {$module->name()}] No explicit directories created");
         } else {
-            $directories->each(function ($directory) use ($module) {
+            $directories->each(function($directory) use ($module) {
                 $this->createDirectory($module, $directory);
             });
         }
@@ -162,7 +162,7 @@ 
 
         $replacements = $subModule ? ['class' => $subModule] : [];
 
-        $files->each(function ($stubFile, $moduleFile) use ($module, $stubGroup, $replacements) {
+        $files->each(function($stubFile, $moduleFile) use ($module, $stubGroup, $replacements) {
             $this->copyStubFileIntoModule($module, $stubFile, $stubGroup,
                 $moduleFile, $replacements);
         });

src/Services/Modular.php

Doc Comments +4 added lines, -4 removed lines

@@ -101,7 +101,7 @@ 
      *
      * @param string $type
      *
-     * @return array
+     * @return Collection
      */
     public function withRoutes($type)
     {
@@ -111,7 +111,7 @@ 
     /**
      * Get all routable modules (active and having routes file)
      *
-     * @return array
+     * @return Collection
      */
     public function withFactories()
     {
@@ -122,7 +122,7 @@ 
      * Get all modules that have service providers (active and having service
      * provider file)
      *
-     * @return array
+     * @return Collection
      */
     public function withServiceProviders()
     {
@@ -132,7 +132,7 @@ 
     /**
      * Get all modules that have seeders (active and having seeder file)
      *
-     * @return array
+     * @return Collection
      */
     public function withSeeders()
     {

Spacing +9 added lines, -9 removed lines

@@ -49,7 +49,7 @@ 
      */
     public function seed(Seeder $seeder)
     {
-        $this->withSeeders()->each(function ($module) use ($seeder) {
+        $this->withSeeders()->each(function($module) use ($seeder) {
             /* @var Module $module */
             $seeder->call($module->seederClass());
         });
@@ -63,10 +63,10 @@ 
      */
     public function loadRoutes(Registrar $router, $type = null)
     {
-        $this->withRoutes($type)->each(function ($module) use ($router, $type) {
+        $this->withRoutes($type)->each(function($module) use ($router, $type) {
             /* @var Module $module */
             $router->group(['namespace' => $module->routingControllerNamespace()],
-                function ($router) use ($module, $type) {
+                function($router) use ($module, $type) {
                     $this->app['files']->requireOnce($this->app->basePath() .
                         DIRECTORY_SEPARATOR .
                         $module->routesFilePath(compact('type')));
@@ -79,7 +79,7 @@ 
      */
     public function loadFactories()
     {
-        $this->withFactories()->each(function ($module) {
+        $this->withFactories()->each(function($module) {
             /* @var Module $module */
             $this->app['files']->requireOnce($module->factoryFilePath());
         });
@@ -90,7 +90,7 @@ 
      */
     public function loadServiceProviders()
     {
-        $this->withServiceProviders()->each(function ($module) {
+        $this->withServiceProviders()->each(function($module) {
             /* @var Module $module */
             $this->app->register($module->serviceProviderClass());
         });
@@ -151,7 +151,7 @@ 
     protected function filterActiveByMethod($requirement, array $data = [])
     {
         return $this->modules()
-            ->filter(function ($module) use ($requirement, $data) {
+            ->filter(function($module) use ($requirement, $data) {
                 return $module->active() && $module->$requirement($data);
             })->values();
     }
@@ -173,7 +173,7 @@ 
      */
     public function active()
     {
-        return $this->modules()->filter(function ($module) {
+        return $this->modules()->filter(function($module) {
             return $module->active();
         })->values();
     }
@@ -199,7 +199,7 @@ 
     {
         $this->modules = collect();
 
-        collect($this->config->modules())->each(function ($options, $name) {
+        collect($this->config->modules())->each(function($options, $name) {
             $this->modules->push(new Module($name, $this->app, $options));
         });
     }
@@ -213,7 +213,7 @@ 
      */
     public function find($name)
     {
-        return $this->modules()->first(function ($module) use ($name) {
+        return $this->modules()->first(function($module) use ($name) {
             /* @var Module $module */
             return $module->name() == $name;
         });

stubs/app/Core/AbstractRepository.php

Doc Comments +1 added lines, -1 removed lines

@@ -113,7 +113,7 @@
      *
      * @param array|int $ids
      *
-     * @return bool
+     * @return integer
      */
     public function destroy($ids)
     {

src/Providers/ModularServiceProvider.php

Spacing +5 added lines, -5 removed lines

@@ -27,11 +27,11 @@ 
     public function register()
     {
         // register module binding
-        $this->app->singleton('modular.config', function ($app) {
+        $this->app->singleton('modular.config', function($app) {
             return new Config($app);
         });
 
-        $this->app->singleton('modular', function ($app) {
+        $this->app->singleton('modular', function($app) {
             return new Modular($app, $app['modular.config']);
         });
 
@@ -112,7 +112,7 @@ 
         $publishedStubsPath = $this->app['modular.config']->stubsPath();
 
         collect(glob($templatesPath . '/*/{,.}*.stub', GLOB_BRACE))
-            ->each(function ($file) use ($publishedStubsPath, $pathLength) {
+            ->each(function($file) use ($publishedStubsPath, $pathLength) {
                 $this->filesToPublish->put($file,
                     $publishedStubsPath . DIRECTORY_SEPARATOR .
                     mb_substr($file, $pathLength + 1));
@@ -129,7 +129,7 @@ 
     protected function addAppFilesToPublished()
     {
         $appPath = $this->getAppSamplePath();
-        collect(glob($appPath . '/*/*'))->each(function ($file) use ($appPath) {
+        collect(glob($appPath . '/*/*'))->each(function($file) use ($appPath) {
             $this->filesToPublish->put($file,
                 $this->app['path'] . DIRECTORY_SEPARATOR .
                 mb_substr($file, mb_strlen($appPath) + 1));
@@ -177,7 +177,7 @@ 
 
         // add to paths all migration directories from modules
         collect($this->app['modular']->active())
-            ->each(function ($module) use ($paths) {
+            ->each(function($module) use ($paths) {
                 /* @var Module $module */
                 $paths->push($module->migrationsPath());
             });

src/Console/Commands/ModuleFiles.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
             return;
         }
 
-        $subModules->each(function ($subModule) use ($module, $stubGroup) {
+        $subModules->each(function($subModule) use ($module, $stubGroup) {
             $this->createSubModule($module, $subModule, $stubGroup);
         });
     }

src/Console/Commands/ModuleMake.php

Spacing +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
         // verify whether stub directory exists
         $this->verifyStubGroup($stubGroup);
 
-        $moduleNames->each(function ($moduleName) use ($stubGroup) {
+        $moduleNames->each(function($moduleName) use ($stubGroup) {
             $module = $this->createModuleObject($moduleName);
 
             // module added to configuration or module directory exists
@@ -103,7 +103,7 @@ 
         // getting modified content of config file
         $result =
             preg_replace_callback($this->laravel['modular.config']->autoAddPattern(),
-                function ($matches) use ($module, $configFile) {
+                function($matches) use ($module, $configFile) {
                     return $matches[1] . $matches[2] .
                     $this->replace($this->laravel['modular.config']->autoAddTemplate(),
                         $module) .

src/Console/Commands/ModuleSeed.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
 
         $options = $this->getOptions();
 
-        $modules->each(function ($module) use ($options) {
+        $modules->each(function($module) use ($options) {
             /* @var Module $module */
             $class = $module->seederClass($this->option('class'));
 

src/Console/Traits/ModuleVerification.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
     {
         $modules = collect();
 
-        $moduleNames->each(function ($name) use ($verifyActive, $modules) {
+        $moduleNames->each(function($name) use ($verifyActive, $modules) {
             /** @var Module $module */
             $module = $this->laravel['modular']->find($name);
 

src/Models/Module.php

Spacing +1 added lines, -2 removed lines

@@ -180,8 +180,7 @@
     protected function hasFile($option, $pathFunction, $prefix = '')
     {
         return (bool)($this->options->has($prefix . $option) ?
-            $this->options->get($prefix . $option) :
-            $this->laravel['files']->exists($this->$pathFunction($prefix)));
+            $this->options->get($prefix . $option) : $this->laravel['files']->exists($this->$pathFunction($prefix)));
     }
 
     /**