xpressengine / xpressengine

app/Providers/RegisterServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     {
         $this->app->singleton(
             'xe.uiobject',
-            function ($app) {
+            function($app) {
 
                 $aliases = $app['config']->get('xe.uiobject.aliases');
 

app/Providers/SiteServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     {
         $this->app->singleton(
             'xe.uiobject',
-            function ($app) {
+            function($app) {
 
                 $aliases = $app['config']->get('xe.uiobject.aliases');
 

app/Providers/SkinServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     {
         $this->app->singleton(
             'xe.uiobject',
-            function ($app) {
+            function($app) {
 
                 $aliases = $app['config']->get('xe.uiobject.aliases');
 

app/Providers/TemporaryServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     {
         $this->app->singleton(
             'xe.uiobject',
-            function ($app) {
+            function($app) {
 
                 $aliases = $app['config']->get('xe.uiobject.aliases');
 

app/Console/Commands/PluginMakeCommand.php

Doc Comments +20 added lines, -6 removed lines

@@ -55,7 +55,7 @@ 
     /**
      * Execute the console command.
      *
-     * @return bool|null
+     * @return false|null
      */
     public function fire()
     {
@@ -100,6 +100,10 @@ 
         $this->info("Input and modify your plugin information in ./plugins/$name/composer.json file.");
     }
 
+    /**
+     * @param string $path
+     * @param string $namespace
+     */
     protected function checkEnv($path, $name, $namespace, $title)
     {
         // check directory exists
@@ -119,9 +123,9 @@ 
     /**
      * makePluginClass
      *
-     * @param $path
+     * @param string $path
      * @param $name
-     * @param $namespace
+     * @param string $namespace
      *
      * @return void
      */
@@ -188,9 +192,9 @@ 
     /**
      * makeComposerJson
      *
-     * @param $path
+     * @param string $path
      * @param $pluginName
-     * @param $namespace
+     * @param string $namespace
      *
      * @return void
      */
@@ -319,7 +323,6 @@ 
     /**
      * Get the full namespace name for a given class.
      *
-     * @param  string $name
      *
      * @return string
      */
@@ -342,6 +345,7 @@ 
     /**
      * Get the stub file for the generator.
      *
+     * @param string $filename
      * @return string
      */
     protected function getStub($filename)
@@ -389,6 +393,9 @@ 
         return $this->argument('title');
     }
 
+    /**
+     * @param string $path
+     */
     protected function makeDirectoryStructure($path)
     {
         $defaultDirectories = [
@@ -409,6 +416,9 @@ 
         $this->files->put($path.'/assets/style.css', $stub);
     }
 
+    /**
+     * @param string $path
+     */
     protected function runComposerUpdate($path)
     {
         $composer = $this->findComposer();
@@ -421,6 +431,10 @@ 
         $output = $this->output;
 
         $process->run(
+
+            /**
+             * @param string $type
+             */
             function ($type, $line) use ($output) {
                 $output->write($line);
             }

Spacing +3 added lines, -3 removed lines

@@ -58,7 +58,7 @@ 
         $title = $this->getTitleInput();
         $path = app('xe.plugin')->getPluginsDir().'/'.$name;
 
-        if($this->checkEnv($path, $name, $namespace, $title) === false) {
+        if ($this->checkEnv($path, $name, $namespace, $title) === false) {
             return false;
         }
 
@@ -102,7 +102,7 @@ 
         }
 
         // check namespace
-        if(!str_contains($namespace, '\\')) {
+        if (!str_contains($namespace, '\\')) {
             $this->error('The namespace must have at least 1 delimiter(\\), use double backslash(\\\\) as delimiter');
             return false;
         }
@@ -421,7 +421,7 @@ 
         $output = $this->output;
 
         $process->run(
-            function ($type, $line) use ($output) {
+            function($type, $line) use ($output) {
                 $output->write($line);
             }
         );

core/tests/Frontend/PackageTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
     /**
      * getHandler
      *
-     * @return FrontendHandler
+     * @return Mockery\MockInterface
      */
     protected function getHandler()
     {

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
     public function testRegister()
     {
         $p = new Package('mypackage');
-        $p->register(function($handler){
+        $p->register(function($handler) {
         });
 
         $packages = Package::packages();

core/tests/Storage/UrlMakerTest.php

Doc Comments +3 added lines

@@ -315,6 +315,9 @@
         ];
     }
 
+    /**
+     * @param string $methodName
+     */
     private function invokeMethod(&$object, $methodName, array $parameters = array())
     {
         $reflection = new \ReflectionClass(get_class($object));

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
         $instance->shouldReceive('getUrl')->once()->with($mockFile)->andReturnNull();
         $instance->shouldReceive('route')->once()->with($mockFile)->andReturn('/url/path');
 
-        $closure = function ($file, &$url) {
+        $closure = function($file, &$url) {
             $url .= '/1';
         };
 
@@ -41,7 +41,7 @@ 
         $mockFile->id = 1;
 
         $urlGenerator->shouldReceive('route')->once()
-            ->with(m::on(function () { return true; }), ['id' => 1])
+            ->with(m::on(function() { return true; }), ['id' => 1])
             ->andReturn();
 
         $instance->route($mockFile);

app/Providers/HttpServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
         // set config to request when rebind request
         $this->app->rebinding(
             'request',
-            function ($app, $request) {
+            function($app, $request) {
                 return $request->setConfig($app['config']);
             }
         );

app/Providers/MobileServiceProvider.php

Spacing +6 added lines, -6 removed lines

@@ -39,7 +39,7 @@ 
     {
         $this->app->rebinding(
             'request',
-            function ($app, $request) {
+            function($app, $request) {
                 $request->setMobileResolver($this->getMobileResolver());
             }
         );
@@ -54,25 +54,25 @@ 
     {
         Blade::directive(
             'mobileonly',
-            function ($expression) {
+            function($expression) {
                 return "<?php if(app('request')->isMobile()) { ?>";
             }
         );
         Blade::directive(
             'endmobileonly',
-            function ($expression) {
+            function($expression) {
                 return "<?php } ?>";
             }
         );
         Blade::directive(
             'desktoponly',
-            function ($expression) {
+            function($expression) {
                 return "<?php if(!app('request')->isMobile()) { ?>";
             }
         );
         Blade::directive(
             'enddesktoponly',
-            function ($expression) {
+            function($expression) {
                 return "<?php } ?>";
             }
         );
@@ -80,7 +80,7 @@ 
 
     protected function getMobileResolver()
     {
-        return function (Request $request, $byUserAgent = false) {
+        return function(Request $request, $byUserAgent = false) {
             $app = $this->app;
             if ($byUserAgent === false) {
                 $isMobile = $request->get('_m', null);