luyadev / luya

core/web/JsonLd.php

Spacing +2 added lines, -2 removed lines

@@ -138,8 +138,8 @@
     protected static function registerView()
     {
         if (self::$_view === null) {
-            Yii::$app->view->on(View::EVENT_BEGIN_BODY, function ($event) {
-                echo '<script type="application/ld+json">' . Json::encode($event->sender->params) . '</script>';
+            Yii::$app->view->on(View::EVENT_BEGIN_BODY, function($event) {
+                echo '<script type="application/ld+json">'.Json::encode($event->sender->params).'</script>';
             });
                     
             self::$_view = true;

core/web/jsonld/EventInterface.php

Doc Comments +2 added lines

@@ -193,6 +193,7 @@ 
 
     /**
      * @param int $maximumAttendeeCapacity
+     * @return void
      */
     public function setMaximumAttendeeCapacity($maximumAttendeeCapacity);
 
@@ -302,6 +303,7 @@ 
 
     /**
      * @param Event $subEvent
+     * @return void
      */
     public function setSubEvent($subEvent);
 

core/web/Svg.php

Spacing +5 added lines, -5 removed lines

@@ -73,27 +73,27 @@
     public function run()
     {
         // Cache generated code
-        return $this->getOrSetHasCache(['svg', $this->folder, $this->file, $this->cssClass], function () {
+        return $this->getOrSetHasCache(['svg', $this->folder, $this->file, $this->cssClass], function() {
 
             // Check if file ends with .svg, if not add the extension
-            $svgFile = StringHelper::endsWith($this->file, '.svg') ? $this->file : $this->file . '.svg';
+            $svgFile = StringHelper::endsWith($this->file, '.svg') ? $this->file : $this->file.'.svg';
 
             // Build the full svg file path
-            $svgPath = $this->folder . DIRECTORY_SEPARATOR . $svgFile;
+            $svgPath = $this->folder.DIRECTORY_SEPARATOR.$svgFile;
 
             // Get the svg contents
             $content = FileHelper::getFileContent($svgPath);
 
             // If a cssClass string is given, add it to the <svg> tag
             if ($this->cssClass && is_string($this->cssClass)) {
-                $content = preg_replace('/<svg/', '<svg class="' . $this->cssClass . '"', $content);
+                $content = preg_replace('/<svg/', '<svg class="'.$this->cssClass.'"', $content);
             }
 
             if ($content) {
                 return $content;
             }
 
-            throw new Exception('Unable to access SVG File: ' . $svgPath);
+            throw new Exception('Unable to access SVG File: '.$svgPath);
         });
     }
 }

core/helpers/ExportHelper.php

Spacing +4 added lines, -4 removed lines

@@ -85,10 +85,10 @@ 
 
         $output = null;
         if ($generateHeader) {
-            $output.= self::generateRow($header, $delimiter, '"');
+            $output .= self::generateRow($header, $delimiter, '"');
         }
         foreach ($rows as $row) {
-            $output.= self::generateRow($row, $delimiter, '"');
+            $output .= self::generateRow($row, $delimiter, '"');
         }
 
         return $output;
@@ -104,13 +104,13 @@ 
      */
     protected static function generateRow(array $row, $delimiter, $enclose)
     {
-        array_walk($row, function (&$item) use ($enclose) {
+        array_walk($row, function(&$item) use ($enclose) {
             if (!is_scalar($item)) {
                 $item = "array";
             }
             $item = $enclose.Html::encode($item).$enclose;
         });
         
-        return implode($delimiter, $row) . PHP_EOL;
+        return implode($delimiter, $row).PHP_EOL;
     }
 }

core/traits/CacheableTrait.php

Indentation +27 added lines, -27 removed lines

@@ -80,33 +80,33 @@
     }
     
     /**
-    * Method combines both [[setHasCache()]] and [[getHasCache()]] methods to retrieve value identified by a $key,
-    * or to store the result of $closure execution if there is no cache available for the $key.
-    *
-    * Usage example:
-    *
-    * ```php
-    * use CacheableTrait;
-    *
-    * public function getTopProducts($count = 10)
-    * {
-    *     return $this->getOrSetHasCache(['top-n-products', 'n' => $count], function ($cache) use ($count) {
-    *         return Products::find()->mostPopular()->limit(10)->all();
-    *     }, 1000);
-    * }
-    * ```
-    *
-    * @param mixed $key a key identifying the value to be cached. This can be a simple string or
-    * a complex data structure consisting of factors representing the key.
-    * @param \Closure $closure the closure that will be used to generate a value to be cached.
-    * In case $closure returns `false`, the value will not be cached.
-    * @param int $duration default duration in seconds before the cache will expire. If not set,
-    * [[defaultDuration]] value will be used. 0 means never expire.
-    * @param Dependency $dependency dependency of the cached item. If the dependency changes,
-    * the corresponding value in the cache will be invalidated when it is fetched via [[get()]].
-    * This parameter is ignored if [[serializer]] is `false`.
-    * @return mixed result of $closure execution
-    */
+     * Method combines both [[setHasCache()]] and [[getHasCache()]] methods to retrieve value identified by a $key,
+     * or to store the result of $closure execution if there is no cache available for the $key.
+     *
+     * Usage example:
+     *
+     * ```php
+     * use CacheableTrait;
+     *
+     * public function getTopProducts($count = 10)
+     * {
+     *     return $this->getOrSetHasCache(['top-n-products', 'n' => $count], function ($cache) use ($count) {
+     *         return Products::find()->mostPopular()->limit(10)->all();
+     *     }, 1000);
+     * }
+     * ```
+     *
+     * @param mixed $key a key identifying the value to be cached. This can be a simple string or
+     * a complex data structure consisting of factors representing the key.
+     * @param \Closure $closure the closure that will be used to generate a value to be cached.
+     * In case $closure returns `false`, the value will not be cached.
+     * @param int $duration default duration in seconds before the cache will expire. If not set,
+     * [[defaultDuration]] value will be used. 0 means never expire.
+     * @param Dependency $dependency dependency of the cached item. If the dependency changes,
+     * the corresponding value in the cache will be invalidated when it is fetched via [[get()]].
+     * This parameter is ignored if [[serializer]] is `false`.
+     * @return mixed result of $closure execution
+     */
     public function getOrSetHasCache($key, \Closure $closure, $duration = null, $dependency = null)
     {
         if (($value = $this->getHasCache($key)) !== false) {

dev/BaseDevCommand.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
      */
     public function actionConfigInfo()
     {
-        $this->outputInfo("dev config file: " . Yii::getAlias($this->configFile));
+        $this->outputInfo("dev config file: ".Yii::getAlias($this->configFile));
         
         $config = $this->readConfig();
         
@@ -91,7 +91,7 @@ 
         $save = FileHelper::writeFile($this->configFile, Json::encode($content));
         
         if (!$save) {
-            return $this->outputError("Unable to find config file " . $this->configFile. ". Please create and provide Permissions.");
+            return $this->outputError("Unable to find config file ".$this->configFile.". Please create and provide Permissions.");
         }
         
         return $value;

Doc Comments +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
     /**
      * Display config data and location.
      *
-     * @return boolean|void
+     * @return integer|null
      */
     public function actionConfigInfo()
     {
@@ -77,7 +77,7 @@ 
      * Save a value in the config for a given key.
      *
      * @param string $key
-     * @param mixed $value
+     * @param string|boolean $value
      * @return mixed
      */
     protected function saveConfig($key, $value)

dev/RepoController.php

Doc Comments +1 added lines, -1 removed lines

@@ -203,7 +203,7 @@
      * @param string $repo
      * @param string $isFork
      * @param string $exists
-     * @return array
+     * @return string[]
      */
     private function summaryItem($repo, $isFork, $exists)
     {

Indentation +21 added lines, -21 removed lines

@@ -154,11 +154,11 @@ 
     public function actionUpdate()
     {
         foreach ($this->repos as $repo) {
-        	$this->rebaseRepo($repo, $this->getFilesystemRepoPath($repo));
+            $this->rebaseRepo($repo, $this->getFilesystemRepoPath($repo));
         }
         
         foreach ($this->getConfig(self::CONFIG_VAR_CUSTOMCLONES, []) as $repo => $path) {
-        	$this->rebaseRepo($repo, $path);
+            $this->rebaseRepo($repo, $path);
         }
     }
     
@@ -170,17 +170,17 @@ 
      */
     public function actionClone($vendor = null, $repo = null)
     {
-    	// if `vendor/repo` notation is provided
-    	if ($vendor !== null && strpos($vendor, '/')) {
-    		list($vendor, $repo) = explode("/", $vendor);	
-    	}
+        // if `vendor/repo` notation is provided
+        if ($vendor !== null && strpos($vendor, '/')) {
+            list($vendor, $repo) = explode("/", $vendor);	
+        }
     	
         if (empty($vendor)) {
             $vendor = $this->prompt("Enter the username/vendor for this repo (e.g. luyadev)");
         }
         
         if (empty($repo)) {
-        	$repo = $this->prompt("Enter the name of the repo you like to clone (e.g. luya-module-news)");
+            $repo = $this->prompt("Enter the name of the repo you like to clone (e.g. luya-module-news)");
         }
         
         $clones = $this->getConfig(self::CONFIG_VAR_CUSTOMCLONES, []);
@@ -201,14 +201,14 @@ 
      */
     public function actionRemove($repo)
     {
-    	FileHelper::removeDirectory($this->getFilesystemRepoPath($repo));
-    	$clones = $this->getConfig(self::CONFIG_VAR_CUSTOMCLONES, []);
-    	if (isset($clones[$repo])) {
-	    	unset($clones[$repo]);
-	    	$this->saveConfig(self::CONFIG_VAR_CUSTOMCLONES, $clones);
-    	}
+        FileHelper::removeDirectory($this->getFilesystemRepoPath($repo));
+        $clones = $this->getConfig(self::CONFIG_VAR_CUSTOMCLONES, []);
+        if (isset($clones[$repo])) {
+            unset($clones[$repo]);
+            $this->saveConfig(self::CONFIG_VAR_CUSTOMCLONES, $clones);
+        }
     	
-    	return $this->outputSuccess("Removed repo {$repo}.");
+        return $this->outputSuccess("Removed repo {$repo}.");
     }
     
     private $_gitWrapper;
@@ -296,16 +296,16 @@ 
      */
     private function rebaseRepo($repo, $repoFileSystemPath)
     {
-    	$wrapper = new GitWrapper();
+        $wrapper = new GitWrapper();
     	 
-    	$wrapper->git('checkout master', $repoFileSystemPath);
-    	$this->outputInfo("{$repo}: checkout master ✔");
+        $wrapper->git('checkout master', $repoFileSystemPath);
+        $this->outputInfo("{$repo}: checkout master ✔");
     	 
-    	$wrapper->git('fetch upstream', $repoFileSystemPath);
-    	$this->outputInfo("{$repo}: fetch upstream ✔");
+        $wrapper->git('fetch upstream', $repoFileSystemPath);
+        $this->outputInfo("{$repo}: fetch upstream ✔");
     	 
-    	$wrapper->git('rebase upstream/master master', $repoFileSystemPath);
-    	$this->outputInfo("{$repo}: rebase master ✔");
+        $wrapper->git('rebase upstream/master master', $repoFileSystemPath);
+        $this->outputInfo("{$repo}: rebase master ✔");
     }
     
     /**

Spacing +2 added lines, -2 removed lines

@@ -97,7 +97,7 @@ 
         // generate summary overview
         foreach ($this->repos as $repo) {
             $newRepoHome = $this->getFilesystemRepoPath($repo);
-            if (file_exists($newRepoHome . DIRECTORY_SEPARATOR . '.git')) {
+            if (file_exists($newRepoHome.DIRECTORY_SEPARATOR.'.git')) {
                 $summary[] = $this->summaryItem($repo, false, true);
             } elseif ($this->forkExists($username, $repo)) {
                 $summary[] = $this->summaryItem($repo, true, false);
@@ -245,7 +245,7 @@ 
      */
     private function getFilesystemRepoPath($repo)
     {
-        return 'repos' . DIRECTORY_SEPARATOR . $repo;
+        return 'repos'.DIRECTORY_SEPARATOR.$repo;
     }
     
     /**

core/base/Module.php

Doc Comments +3 added lines

@@ -241,6 +241,9 @@
         return $files;
     }
     
+    /**
+     * @param string $prefix
+     */
     private function fileToName($prefix, $file)
     {
         $value = ltrim(str_replace([$prefix, 'Controller.php'], '', $file), DIRECTORY_SEPARATOR);

Indentation +1 added lines, -1 removed lines

@@ -234,7 +234,7 @@
                     $files[$this->fileToName($staticPath, $file)] = $file;
                 }
             } catch (InvalidParamException $e) {
-                 // catch if folder not found.
+                    // catch if folder not found.
             }
         };
         

Spacing +2 added lines, -2 removed lines

@@ -229,7 +229,7 @@ 
             }
         } catch (InvalidParamException $e) {
             try {
-                $staticPath = static::staticBasePath() . DIRECTORY_SEPARATOR . 'controllers';
+                $staticPath = static::staticBasePath().DIRECTORY_SEPARATOR.'controllers';
                 foreach (FileHelper::findFiles($staticPath) as $file) {
                     $files[$this->fileToName($staticPath, $file)] = $file;
                 }
@@ -257,7 +257,7 @@ 
      */
     public function getControllerPath()
     {
-        return Yii::getAlias('@' . str_replace('\\', '/', $this->controllerNamespace), false);
+        return Yii::getAlias('@'.str_replace('\\', '/', $this->controllerNamespace), false);
     }
 
     // STATIC METHODS