luyadev / luya

core/web/EmailLink.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
      */
     public function getHref()
     {
-        return empty($this->getEmail()) ?: 'mailto:' . $this->getEmail();
+        return empty($this->getEmail()) ?: 'mailto:'.$this->getEmail();
     }
 
     /**

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

Doc Comments +2 added lines, -2 removed lines

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

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;

dev/RepoController.php

Doc Comments +7 added lines

@@ -192,6 +192,10 @@ 
         return $this->_gitWrapper;
     }
     
+    /**
+     * @param boolean $isFork
+     * @param boolean $exists
+     */
     private function summaryItem($repo, $isFork, $exists)
     {
         return [$repo, $exists, $isFork];
@@ -202,6 +206,9 @@ 
         return 'repos' . DIRECTORY_SEPARATOR . $repo;
     }
     
+    /**
+     * @param string|boolean $username
+     */
     private function forkExists($username, $repo)
     {
         return (new Curl())->get('https://api.github.com/repos/'.$username.'/'.$repo)->isSuccess();

Spacing +5 added lines, -5 removed lines

@@ -88,7 +88,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);
@@ -147,13 +147,13 @@ 
         $wrapper = new GitWrapper();
         
         foreach ($this->repos as $repo) {
-            $wrapper->git('checkout master', 'repos' . DIRECTORY_SEPARATOR . $repo);
+            $wrapper->git('checkout master', 'repos'.DIRECTORY_SEPARATOR.$repo);
             $this->outputInfo("{$repo}: checkout master ✔");
             
-            $wrapper->git('fetch upstream', 'repos' . DIRECTORY_SEPARATOR . $repo);
+            $wrapper->git('fetch upstream', 'repos'.DIRECTORY_SEPARATOR.$repo);
             $this->outputInfo("{$repo}: fetch upstream ✔");
             
-            $wrapper->git('rebase upstream/master master', 'repos' . DIRECTORY_SEPARATOR . $repo);
+            $wrapper->git('rebase upstream/master master', 'repos'.DIRECTORY_SEPARATOR.$repo);
             $this->outputInfo("{$repo}: rebase master ✔");
         }
     }
@@ -199,7 +199,7 @@ 
     
     private function getFilesystemRepoPath($repo)
     {
-        return 'repos' . DIRECTORY_SEPARATOR . $repo;
+        return 'repos'.DIRECTORY_SEPARATOR.$repo;
     }
     
     private function forkExists($username, $repo)

Indentation +5 added lines, -5 removed lines

@@ -166,17 +166,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)");
         }
         
         return $this->cloneRepo($repo, $this->getCloneUrlBasedOnType($repo, $vendor), $this->getFilesystemRepoPath($repo), $vendor);