nWidart / laravel-modules

src/helpers.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if (! function_exists('module_path')) {
+if (!function_exists('module_path')) {
     function module_path($name)
     {
         $module = app('modules')->find($name);
@@ -9,7 +9,7 @@ 
     }
 }
 
-if (! function_exists('public_path')) {
+if (!function_exists('public_path')) {
     /**
      * Get the path to the public folder.
      *
@@ -18,6 +18,6 @@ 
      */
     function public_path($path = '')
     {
-        return app()->make('path.public') . ($path ? DIRECTORY_SEPARATOR . ltrim($path, DIRECTORY_SEPARATOR) : $path);
+        return app()->make('path.public').($path ? DIRECTORY_SEPARATOR . ltrim($path, DIRECTORY_SEPARATOR) : $path);
     }
 }

src/Commands/MigrationMakeCommand.php

Doc Comments +2 added lines, -2 removed lines

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

Spacing +2 added lines, -2 removed lines

@@ -116,7 +116,7 @@ 
 
         $generatorPath = GenerateConfigReader::read('migration');
 
-        return $path . $generatorPath->getPath() . '/' . $this->getFileName() . '.php';
+        return $path.$generatorPath->getPath().'/'.$this->getFileName().'.php';
     }
 
     /**
@@ -124,7 +124,7 @@ 
      */
     private function getFileName()
     {
-        return date('Y_m_d_His_') . $this->getSchemaName();
+        return date('Y_m_d_His_').$this->getSchemaName();
     }
 
     /**

src/Commands/ModelMakeCommand.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 +4 added lines, -4 removed lines

@@ -53,8 +53,8 @@ 
 
         $string = '';
         foreach ($pieces as $i => $piece) {
-            if ($i+1 < count($pieces)) {
-                $string .= strtolower($piece) . '_';
+            if ($i + 1 < count($pieces)) {
+                $string .= strtolower($piece).'_';
             } else {
                 $string .= Str::plural(strtolower($piece));
             }
@@ -95,7 +95,7 @@ 
     private function handleOptionalMigrationOption()
     {
         if ($this->option('migration') === true) {
-            $migrationName = 'create_' . $this->createMigrationName() . '_table';
+            $migrationName = 'create_'.$this->createMigrationName().'_table';
             $this->call('module:make-migration', ['name' => $migrationName, 'module' => $this->argument('module')]);
         }
     }
@@ -128,7 +128,7 @@ 
 
         $modelPath = GenerateConfigReader::read('model');
 
-        return $path . $modelPath->getPath() . '/' . $this->getModelName() . '.php';
+        return $path.$modelPath->getPath().'/'.$this->getModelName().'.php';
     }
 
     /**

src/Module.php

Doc Comments +5 added lines, -5 removed lines

@@ -42,7 +42,7 @@ 
      *
      * @param Container $app
      * @param $name
-     * @param $path
+     * @param string $path
      */
     public function __construct(Container $app, $name, $path)
     {
@@ -227,7 +227,7 @@ 
     /**
      * Get a specific data from composer.json file by given the key.
      *
-     * @param $key
+     * @param string $key
      * @param null $default
      *
      * @return mixed
@@ -300,7 +300,7 @@ 
     /**
      * Determine whether the given status same with the current module status.
      *
-     * @param $status
+     * @param integer $status
      *
      * @return bool
      */
@@ -354,9 +354,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

@@ -186,7 +186,7 @@ 
     {
         $lowerName = $this->getLowerName();
 
-        $langPath = $this->getPath() . '/Resources/lang';
+        $langPath = $this->getPath().'/Resources/lang';
 
         if (is_dir($langPath)) {
             $this->loadTranslationsFrom($langPath, $lowerName);
@@ -206,8 +206,8 @@ 
             $file = 'module.json';
         }
 
-        return array_get($this->moduleJson, $file, function () use ($file) {
-            return $this->moduleJson[$file] = new Json($this->getPath() . '/' . $file, $this->app['files']);
+        return array_get($this->moduleJson, $file, function() use ($file) {
+            return $this->moduleJson[$file] = new Json($this->getPath().'/'.$file, $this->app['files']);
         });
     }
 
@@ -258,7 +258,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]);
     }
     /**
      * Register the aliases from this module.
@@ -283,7 +283,7 @@ 
     protected function registerFiles()
     {
         foreach ($this->get('files', []) as $file) {
-            include $this->path . '/' . $file;
+            include $this->path.'/'.$file;
         }
     }
 
@@ -406,7 +406,7 @@ 
      */
     public function getExtraPath(string $path) : string
     {
-        return $this->getPath() . '/' . $path;
+        return $this->getPath().'/'.$path;
     }
 
     /**

src/Repository.php

Spacing +9 added lines, -9 removed lines

@@ -100,7 +100,7 @@ 
     {
         $paths = $this->paths;
 
-        $paths[] = $this->getPath() . '/*';
+        $paths[] = $this->getPath().'/*';
 
         if ($this->config('scan.enabled')) {
             $paths = array_merge($paths, $this->config('scan.paths'));
@@ -146,7 +146,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();
         });
     }
@@ -234,7 +234,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;
             }
@@ -384,9 +384,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).'/';
         }
     }
 
@@ -399,7 +399,7 @@ 
      */
     public function assetPath($module) : string
     {
-        return $this->config('paths.assets') . '/' . $module;
+        return $this->config('paths.assets').'/'.$module;
     }
 
     /**
@@ -412,7 +412,7 @@ 
      */
     public function config($key, $default = null)
     {
-        return $this->app['config']->get('modules.' . $key, $default);
+        return $this->app['config']->get('modules.'.$key, $default);
     }
 
     /**
@@ -513,9 +513,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);
     }

Doc Comments +4 added lines, -4 removed lines

@@ -377,7 +377,7 @@ 
     /**
      * Get module path for a specific module.
      *
-     * @param $module
+     * @param string $module
      *
      * @return string
      */
@@ -405,9 +405,9 @@ 
     /**
      * Get a specific config data from a configuration file.
      *
-     * @param $key
+     * @param string $key
      *
-     * @param null $default
+     * @param string $default
      * @return mixed
      */
     public function config($key, $default = null)
@@ -607,7 +607,7 @@ 
     /**
      * Get stub path.
      *
-     * @return string|null
+     * @return string|boolean
      */
     public function getStubPath()
     {

src/Commands/ResourceMakeCommand.php

Doc Comments +2 added lines, -2 removed lines

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

Spacing +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
 
         $resourcePath = GenerateConfigReader::read('resource');
 
-        return $path . $resourcePath->getPath() . '/' . $this->getFileName() . '.php';
+        return $path.$resourcePath->getPath().'/'.$this->getFileName().'.php';
     }
 
     /**

src/Commands/TestMakeCommand.php

Doc Comments +2 added lines, -2 removed lines

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

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
 
         $testPath = GenerateConfigReader::read('test');
 
-        return $path . $testPath->getPath() . '/' . $this->getFileName() . '.php';
+        return $path.$testPath->getPath().'/'.$this->getFileName().'.php';
     }
 
     /**

src/Generators/ModuleGenerator.php

Spacing +11 added lines, -11 removed lines

@@ -283,7 +283,7 @@ 
                 continue;
             }
 
-            $path = $this->module->getModulePath($this->getName()) . '/' . $folder->getPath();
+            $path = $this->module->getModulePath($this->getName()).'/'.$folder->getPath();
 
             $this->filesystem->makeDirectory($path, 0755, true);
 
@@ -298,7 +298,7 @@ 
      */
     public function generateGitKeep($path)
     {
-        $this->filesystem->put($path . '/.gitkeep', '');
+        $this->filesystem->put($path.'/.gitkeep', '');
     }
 
     /**
@@ -307,7 +307,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);
@@ -331,13 +331,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(),
         ]);
     }
@@ -352,7 +352,7 @@ 
     protected function getStubContents($stub)
     {
         return (new Stub(
-            '/' . $stub . '.stub',
+            '/'.$stub.'.stub',
             $this->getReplacement($stub)
         )
         )->render();
@@ -386,7 +386,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] = $this->$method();
             } else {
                 $replaces[$key] = null;
@@ -401,7 +401,7 @@ 
      */
     private function generateModuleJsonFile()
     {
-        $path = $this->module->getModulePath($this->getName()) . 'module.json';
+        $path = $this->module->getModulePath($this->getName()).'module.json';
 
         if (!$this->filesystem->isDirectory($dir = dirname($path))) {
             $this->filesystem->makeDirectory($dir, 0775, true);
@@ -418,13 +418,13 @@ 
      */
     private function cleanModuleJsonFile()
     {
-        $path = $this->module->getModulePath($this->getName()) . 'module.json';
+        $path = $this->module->getModulePath($this->getName()).'module.json';
 
         $content = $this->filesystem->get($path);
         $namespace = $this->getModuleNamespaceReplacement();
         $studlyName = $this->getStudlyNameReplacement();
 
-        $provider = '"' . $namespace . '\\\\' . $studlyName . '\\\\Providers\\\\' . $studlyName . 'ServiceProvider"';
+        $provider = '"'.$namespace.'\\\\'.$studlyName.'\\\\Providers\\\\'.$studlyName.'ServiceProvider"';
 
         $content = str_replace($provider, '', $content);
 
@@ -493,6 +493,6 @@ 
 
     protected function getRoutesLocationReplacement()
     {
-        return '/' . $this->module->config('stubs.files.routes');
+        return '/'.$this->module->config('stubs.files.routes');
     }
 }

src/Commands/ControllerMakeCommand.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
 
         $controllerPath = GenerateConfigReader::read('controller');
 
-        return $path . $controllerPath->getPath() . '/' . $this->getControllerName() . '.php';
+        return $path.$controllerPath->getPath().'/'.$this->getControllerName().'.php';
     }
 
     /**