nWidart / laravel-modules

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, $this->getLaravel()))->getPath());
 
         if ($this->option('subpath')) {
-            $path = $path . "/" . $this->option("subpath");
+            $path = $path."/".$this->option("subpath");
         }
 
         $this->call('migrate', [

src/FileRepository.php

Doc Comments +2 added lines, -4 removed lines

@@ -126,9 +126,7 @@ 
     /**
      * Creates a new Module instance
      *
-     * @param Container $app
-     * @param string $args
-     * @param string $path
+     * @param string[] $args
      * @return \Nwidart\Modules\Module
      */
     abstract protected function createModule(...$args);
@@ -632,7 +630,7 @@ 
     /**
      * Get stub path.
      *
-     * @return string|null
+     * @return string|boolean
      */
     public function getStubPath()
     {

Spacing +9 added lines, -9 removed lines

@@ -116,7 +116,7 @@ 
             $paths = array_merge($paths, $this->config('scan.paths'));
         }
 
-        $paths = array_map(function ($path) {
+        $paths = array_map(function($path) {
             return Str::endsWith($path, '/*') ? $path : Str::finish($path, '/*');
         }, $paths);
 
@@ -200,7 +200,7 @@ 
      */
     public function getCached()
     {
-        return $this->cache->remember($this->config('cache.key'), $this->config('cache.lifetime'), function () {
+        return $this->cache->remember($this->config('cache.key'), $this->config('cache.lifetime'), function() {
             return $this->toCollection()->toArray();
         });
     }
@@ -289,7 +289,7 @@ 
     {
         $modules = $this->allEnabled();
 
-        uasort($modules, function (Module $a, Module $b) use ($direction) {
+        uasort($modules, function(Module $a, Module $b) use ($direction) {
             if ($a->get('priority') === $b->get('priority')) {
                 return 0;
             }
@@ -418,9 +418,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).'/';
         }
     }
 
@@ -429,7 +429,7 @@ 
      */
     public function assetPath(string $module) : string
     {
-        return $this->config('paths.assets') . '/' . $module;
+        return $this->config('paths.assets').'/'.$module;
     }
 
     /**
@@ -437,7 +437,7 @@ 
      */
     public function config(string $key, $default = null)
     {
-        return $this->config->get('modules.' . $key, $default);
+        return $this->config->get('modules.'.$key, $default);
     }
 
     /**
@@ -527,9 +527,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->url->asset($baseUrl . "/{$name}/" . $url);
+        $url = $this->url->asset($baseUrl."/{$name}/".$url);
 
         return str_replace(['http://', 'https://'], '//', $url);
     }

src/Module.php

Doc Comments +1 added lines, -1 removed lines

@@ -239,7 +239,7 @@
     /**
      * Get a specific data from composer.json file by given the key.
      *
-     * @param $key
+     * @param string $key
      * @param null $default
      *
      * @return mixed

Spacing +6 added lines, -6 removed lines

@@ -203,7 +203,7 @@ 
     {
         $lowerName = $this->getLowerName();
 
-        $langPath = $this->getPath() . '/Resources/lang';
+        $langPath = $this->getPath().'/Resources/lang';
 
         if (is_dir($langPath)) {
             $this->loadTranslationsFrom($langPath, $lowerName);
@@ -223,8 +223,8 @@ 
             $file = 'module.json';
         }
 
-        return Arr::get($this->moduleJson, $file, function () use ($file) {
-            return $this->moduleJson[$file] = new Json($this->getPath() . '/' . $file, $this->files);
+        return Arr::get($this->moduleJson, $file, function() use ($file) {
+            return $this->moduleJson[$file] = new Json($this->getPath().'/'.$file, $this->files);
         });
     }
 
@@ -277,7 +277,7 @@ 
      */
     protected function fireEvent($event): void
     {
-        $this->app['events']->dispatch(sprintf('modules.%s.' . $event, $this->getLowerName()), [$this]);
+        $this->app['events']->dispatch(sprintf('modules.%s.'.$event, $this->getLowerName()), [$this]);
     }
     /**
      * Register the aliases from this module.
@@ -302,7 +302,7 @@ 
     protected function registerFiles(): void
     {
         foreach ($this->get('files', []) as $file) {
-            include $this->path . '/' . $file;
+            include $this->path.'/'.$file;
         }
     }
 
@@ -407,7 +407,7 @@ 
      */
     public function getExtraPath(string $path) : string
     {
-        return $this->getPath() . '/' . $path;
+        return $this->getPath().'/'.$path;
     }
 
     /**

src/LumenModulesServiceProvider.php

Spacing +4 added lines, -4 removed lines

@@ -31,7 +31,7 @@ 
      */
     public function setupStubPath()
     {
-        Stub::setBasePath(__DIR__ . '/Commands/stubs');
+        Stub::setBasePath(__DIR__.'/Commands/stubs');
 
         if (app('modules')->config('stubs.enabled') === true) {
             Stub::setBasePath(app('modules')->config('stubs.path'));
@@ -43,14 +43,14 @@ 
      */
     protected function registerServices()
     {
-        $this->app->singleton(Contracts\RepositoryInterface::class, function ($app) {
+        $this->app->singleton(Contracts\RepositoryInterface::class, function($app) {
             $path = $app['config']->get('modules.paths.modules');
 
             return new Lumen\LumenFileRepository($app, $path);
         });
-        $this->app->singleton(Contracts\ActivatorInterface::class, function ($app) {
+        $this->app->singleton(Contracts\ActivatorInterface::class, function($app) {
             $activator = $app['config']->get('modules.activator');
-            $class = $app['config']->get('modules.activators.' . $activator)['class'];
+            $class = $app['config']->get('modules.activators.'.$activator)['class'];
 
             return new $class($app);
         });

src/Activators/FileActivator.php

Spacing +2 added lines, -2 removed lines

@@ -79,7 +79,7 @@ 
             return $this->readJson();
         }
 
-        return $this->cache->remember($this->cacheKey, $this->cacheLifetime, function () {
+        return $this->cache->remember($this->cacheKey, $this->cacheLifetime, function() {
             return $this->readJson();
         });
     }
@@ -101,7 +101,7 @@ 
      */
     private function config(string $key, $default = null)
     {
-        return $this->config->get('modules.activators.file.' . $key, $default);
+        return $this->config->get('modules.activators.file.'.$key, $default);
     }
 
     /**

src/Lumen/Module.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
      */
     public function getCachedServicesPath(): string
     {
-        return Str::replaceLast('services.php', $this->getSnakeName() . '_module.php', $this->app->basePath('storage/app/') . 'services.php');
+        return Str::replaceLast('services.php', $this->getSnakeName().'_module.php', $this->app->basePath('storage/app/').'services.php');
     }
 
     /**

src/Support/Migrations/SchemaParser.php

Spacing +13 added lines, -13 removed lines

@@ -146,7 +146,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)) {
@@ -156,7 +156,7 @@ 
             }
         }
 
-        return $results . ';' . PHP_EOL;
+        return $results.';'.PHP_EOL;
     }
 
     /**
@@ -171,9 +171,9 @@ 
     protected function addRelationColumn($key, $field, $column)
     {
         if ($key === 0) {
-            $relatedColumn = Str::snake(class_basename($field)) . '_id';
+            $relatedColumn = Str::snake(class_basename($field)).'_id';
 
-            return "->integer('{$relatedColumn}')->unsigned();" . PHP_EOL . "\t\t\t" . "\$table->foreign('{$relatedColumn}')";
+            return "->integer('{$relatedColumn}')->unsigned();".PHP_EOL."\t\t\t"."\$table->foreign('{$relatedColumn}')";
         }
         if ($key === 1) {
             return "->references('{$field}')";
@@ -182,9 +182,9 @@ 
             return "->on('{$field}')";
         }
         if (Str::contains($field, '(')) {
-            return '->' . $field;
+            return '->'.$field;
         }
-        return '->' . $field . '()';
+        return '->'.$field.'()';
     }
 
     /**
@@ -199,18 +199,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.'()';
     }
 
     /**
@@ -225,10 +225,10 @@ 
     protected function removeColumn($key, $field, $column)
     {
         if ($this->hasCustomAttribute($column)) {
-            return '->' . $field;
+            return '->'.$field;
         }
 
-        return '->dropColumn(' . "'" . $column . "')";
+        return '->dropColumn('."'".$column."')";
     }
 
     /**
@@ -253,7 +253,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/helpers.php

Spacing +6 added lines, -6 removed lines

@@ -1,15 +1,15 @@ 
 <?php
 
-if (! function_exists('module_path')) {
+if (!function_exists('module_path')) {
     function module_path($name, $path = '')
     {
         $module = app('modules')->find($name);
 
-        return $module->getPath() . ($path ? DIRECTORY_SEPARATOR . $path : $path);
+        return $module->getPath().($path ? DIRECTORY_SEPARATOR . $path : $path);
     }
 }
 
-if (! function_exists('config_path')) {
+if (!function_exists('config_path')) {
     /**
      * Get the configuration path.
      *
@@ -18,11 +18,11 @@ 
      */
     function config_path($path = '')
     {
-        return app()->basePath() . '/config' . ($path ? DIRECTORY_SEPARATOR . $path : $path);
+        return app()->basePath().'/config'.($path ? DIRECTORY_SEPARATOR . $path : $path);
     }
 }
 
-if (! function_exists('public_path')) {
+if (!function_exists('public_path')) {
     /**
      * Get the path to the public folder.
      *
@@ -31,6 +31,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/Generators/ModuleGenerator.php

Spacing +10 added lines, -10 removed lines

@@ -329,7 +329,7 @@ 
                 continue;
             }
 
-            $path = $this->module->getModulePath($this->getName()) . '/' . $folder->getPath();
+            $path = $this->module->getModulePath($this->getName()).'/'.$folder->getPath();
 
             $this->filesystem->makeDirectory($path, 0755, true);
             if (config('modules.stubs.gitkeep')) {
@@ -345,7 +345,7 @@ 
      */
     public function generateGitKeep($path)
     {
-        $this->filesystem->put($path . '/.gitkeep', '');
+        $this->filesystem->put($path.'/.gitkeep', '');
     }
 
     /**
@@ -354,7 +354,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);
@@ -381,7 +381,7 @@ 
 
         if (GenerateConfigReader::read('provider')->generate() === true) {
             $this->console->call('module:make-provider', [
-                'name' => $this->getName() . 'ServiceProvider',
+                'name' => $this->getName().'ServiceProvider',
                 'module' => $this->getName(),
                 '--master' => true,
             ]);
@@ -392,7 +392,7 @@ 
 
         if (GenerateConfigReader::read('controller')->generate() === true) {
             $this->console->call('module:make-controller', [
-                'controller' => $this->getName() . 'Controller',
+                'controller' => $this->getName().'Controller',
                 'module' => $this->getName(),
             ]);
         }
@@ -408,7 +408,7 @@ 
     protected function getStubContents($stub)
     {
         return (new Stub(
-            '/' . $stub . '.stub',
+            '/'.$stub.'.stub',
             $this->getReplacement($stub)
         )
         )->render();
@@ -447,7 +447,7 @@ 
             }
         }
         foreach ($keys as $key) {
-            if (method_exists($this, $method = 'get' . ucfirst(Str::studly(strtolower($key))) . 'Replacement')) {
+            if (method_exists($this, $method = 'get'.ucfirst(Str::studly(strtolower($key))).'Replacement')) {
                 $replaces[$key] = $this->$method();
             } else {
                 $replaces[$key] = null;
@@ -462,7 +462,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);
@@ -479,13 +479,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);