luyadev / luya

modules/cmsadmin/src/models/Block.php

Doc Comments +3 added lines

@@ -71,6 +71,9 @@
         parent::afterDelete();
     }
 
+    /**
+     * @param string $context
+     */
     public static function objectId($blockId, $id, $context, $pageObject = null)
     {
         $block = self::find()->where(['id' => $blockId])->asArray()->one();

modules/cmsadmin/src/models/Nav.php

Doc Comments +3 added lines

@@ -39,6 +39,9 @@
         ];
     }
 
+    /**
+     * @param integer $navId
+     */
     public static function findContent($navId)
     {
         $lang = Yii::$app->composition->getKey('langShortCode');

Spacing +5 added lines, -5 removed lines

@@ -120,13 +120,13 @@ 
 
         switch ($e->name) {
             case 'afterInsert':
-                Log::add(1, "nav.insert, cms_nav.id '" . $this->id . "'", $this->toArray());
+                Log::add(1, "nav.insert, cms_nav.id '".$this->id."'", $this->toArray());
                 break;
             case 'afterUpdate':
-                Log::add(2, "nav.update, cms_nav.id '" . $this->id . "'", $this->toArray());
+                Log::add(2, "nav.update, cms_nav.id '".$this->id."'", $this->toArray());
                 break;
             case 'afterDelete':
-                Log::add(3, "nav.delete, cms_nav.id '" . $this->id . "'", $this->toArray());
+                Log::add(3, "nav.delete, cms_nav.id '".$this->id."'", $this->toArray());
                 break;
         }
     }
@@ -227,7 +227,7 @@ 
 
     public function moveUpstairs()
     {
-        $startIndex = (int)$this->sort_index;
+        $startIndex = (int) $this->sort_index;
         foreach (self::find()->where('sort_index >= :index', ['index' => $startIndex])->andWhere(['nav_container_id' => $this->nav_container_id, 'parent_nav_id' => $this->parent_nav_id])->orderBy('sort_index ASC')->asArray()->all() as $item) {
             ++$startIndex;
             Yii::$app->db->createCommand()->update(self::tableName(), ['sort_index' => $startIndex], 'id=:id', ['id' => $item['id']])->execute();
@@ -236,7 +236,7 @@ 
 
     public function moveDownstairs()
     {
-        $startIndex = (int)$this->sort_index;
+        $startIndex = (int) $this->sort_index;
         foreach (self::find()->where('sort_index >= :index', ['index' => $startIndex])->andWhere(['nav_container_id' => $this->nav_container_id, 'parent_nav_id' => $this->parent_nav_id])->orderBy('sort_index ASC')->asArray()->all() as $item) {
             --$startIndex;
             Yii::$app->db->createCommand()->update(self::tableName(), ['sort_index' => $startIndex], 'id=:id', ['id' => $item['id']])->execute();

modules/cmsadmin/src/models/NavItem.php

Doc Comments +5 added lines, -5 removed lines

@@ -183,7 +183,7 @@ 
      * 
      * @todo fix me above!
      *
-     * @param unknown $moduleName
+     * @param string $moduleName
      */
     public static function fromModule($moduleName)
     {
@@ -235,7 +235,7 @@ 
     /**
      * @todo use AR or queryCommand? NavItem::find()->leftJoin('cms_nav_item_module', 'nav_item_type_id=cms_nav_item_module.id')->where(['nav_item_type' => 2, 'cms_nav_item_module.module_name' => $moduleName])->asArray()->one()
      *
-     * @param unknown $moduleName
+     * @param string $moduleName
      *
      * @return unknown
      */
@@ -266,7 +266,7 @@ 
      *
      * Copy content of type cms_nav_item_page to a target nav item. This will create a new entry in cms_nav_item_page and for every used block a new entry in cms_nav_item_page_block_item
      *
-     * @param $targetNavItem nav item object
+     * @param \yii\db\ActiveRecord $targetNavItem nav item object
      * @return bool
      */
     public function copyPageItem($targetNavItem)
@@ -320,7 +320,7 @@ 
      *
      * Copy content of type cms_nav_item_module to a target nav item. This will create a new entry in cms_nav_item_module.
      *
-     * @param $targetNavItem
+     * @param \yii\db\ActiveRecord $targetNavItem
      * @return bool
      */
     public function copyModuleItem($targetNavItem)
@@ -348,7 +348,7 @@ 
      *
      * Copy content of type cms_nav_item_redirect to a target nav item. This will create a new entry in cms_nav_item_redirect.
      *
-     * @param $targetNavItem
+     * @param \yii\db\ActiveRecord $targetNavItem
      * @return bool
      */
     public function copyRedirectItem($targetNavItem)

modules/cmsadmin/src/models/NavItemModule.php

Doc Comments +3 added lines

@@ -20,6 +20,9 @@
 
     private $_module = null;
 
+    /**
+     * @return \luya\base\Module
+     */
     private function getModule()
     {
         if ($this->_module !== null) {

modules/cmsadmin/src/models/NavItemPage.php

Doc Comments +9 added lines

@@ -79,6 +79,9 @@ 
         throw new Exception("Could not find the requested cms layout id '".$this->layout_id."' for nav item page id '". $this->id . "'. Make sure your page does not have an old inactive/deleted cms layout selected.");
     }
     
+    /**
+     * @param string $key
+     */
     private function setHasCache($key, $value, $expirationTime)
     {
         if (Yii::$app->has('cache')) {
@@ -86,6 +89,9 @@ 
         }
     }
     
+    /**
+     * @param string $key
+     */
     private function getHasCache($key)
     {
         if (Yii::$app->has('cache')) {
@@ -98,6 +104,9 @@ 
         return false;
     }
 
+    /**
+     * @param integer $prevId
+     */
     public function renderPlaceholder($navItemPageId, $placeholderVar, $prevId)
     {
         $string = '';

Spacing +2 added lines, -2 removed lines

@@ -76,7 +76,7 @@ 
             return $data;
         }
 
-        throw new Exception("Could not find the requested cms layout id '".$this->layout_id."' for nav item page id '". $this->id . "'. Make sure your page does not have an old inactive/deleted cms layout selected.");
+        throw new Exception("Could not find the requested cms layout id '".$this->layout_id."' for nav item page id '".$this->id."'. Make sure your page does not have an old inactive/deleted cms layout selected.");
     }
     
     private function setHasCache($key, $value, $expirationTime)
@@ -144,7 +144,7 @@ 
                 }
             }
             
-            $string.= $blockResponse;
+            $string .= $blockResponse;
         }
 
         return $string;

modules/cmsadmin/src/Module.php

Doc Comments +3 added lines

@@ -46,6 +46,9 @@
         return $model->getErrors();
     }
 
+    /**
+     * @param string $message
+     */
     public function slack($message, $room)
     {
         $ch = curl_init('https://slack.com/api/chat.postMessage');

modules/errorapi/src/controllers/DefaultController.php

Doc Comments +3 added lines

@@ -46,6 +46,9 @@
         return $model->getErrors();
     }
 
+    /**
+     * @param string $message
+     */
     public function slack($message, $room)
     {
         $ch = curl_init('https://slack.com/api/chat.postMessage');

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
             $mailHtml = $this->renderPartial('_mail', ['model' => $model]);
             
             if (!empty($this->module->recipient)) {
-                $mailer = Yii::$app->mail->compose('Error Api: ' . $model->serverName, $mailHtml);
+                $mailer = Yii::$app->mail->compose('Error Api: '.$model->serverName, $mailHtml);
                 foreach ($this->module->recipient as $recipient) {
                     $mailer->address($recipient);
                 }

modules/errorapi/src/views/default/_mail.php

Spacing +3 added lines, -3 removed lines

@@ -5,7 +5,7 @@ 
 ?>
 <h1 style="color: #f00;"><?php echo $model->message; ?></h1>
 <p style="color: #800000;">from <strong><?php echo $model->serverName; ?></strong></p>
-<a href="https://github.com/luyadev/luya/issues/new?title=<?php echo urlencode('#'. $model->identifier . ' ' . $model->message);?>"><?php echo errorapi\Module::t('mail_create_issue') ?></a>
+<a href="https://github.com/luyadev/luya/issues/new?title=<?php echo urlencode('#'.$model->identifier.' '.$model->message); ?>"><?php echo errorapi\Module::t('mail_create_issue') ?></a>
 <table cellspacing="2" cellpadding="6" border="0" width="1200">
 <?php foreach ($model->errorArray as $key => $value): ?>
 <tr>
@@ -34,13 +34,13 @@ 
             <table cellspacing="0" cellpadding="4" border="0">
                 <?php foreach ($value as $k => $v): ?>
                 <tr>
-                    <td><?php echo $k; ?>:</td><td><?php echo (is_array($v)) ?  print_r($v, true) : $v; ?></td>
+                    <td><?php echo $k; ?>:</td><td><?php echo (is_array($v)) ? print_r($v, true) : $v; ?></td>
                 </tr>
                 <?php endforeach; ?>
             </table>
         <?php else: ?>
             <?php echo $value; ?>
-        <?php endif;?>
+        <?php endif; ?>
 </tr>
 <?php endforeach; ?>
 </table>
\ No newline at end of file

Braces +5 added lines, -2 removed lines

@@ -38,8 +38,11 @@
                 </tr>
                 <?php endforeach; ?>
             </table>
-        <?php else: ?>
-            <?php echo $value; ?>
+        <?php else {
+    : ?>
+            <?php echo $value;
+}
+?>
         <?php endif;?>
 </tr>
 <?php endforeach; ?>

Indentation +2 added lines, -2 removed lines

@@ -1,7 +1,7 @@
 <?php
 /**
- * @var $model
- */
+     * @var $model
+     */
 ?>
 <h1 style="color: #f00;"><?php echo $model->message; ?></h1>
 <p style="color: #800000;">from <strong><?php echo $model->serverName; ?></strong></p>

modules/cms/src/components/RouteBehaviorUrlRule.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
     public function parseRequest($manager, $request)
     {
         // add trace info
-        Yii::info('LUYA-CMS RouteBehaviorUrlRule is parsing the Request for path info \'' . $request->pathInfo .'\'', __METHOD__);
+        Yii::info('LUYA-CMS RouteBehaviorUrlRule is parsing the Request for path info \''.$request->pathInfo.'\'', __METHOD__);
         // return the custom route
         
         $parts = explode("/", $request->pathInfo);