injitools / cms-Inji

system/modules/Ui/objects/DataManager.php

Doc Comments +3 added lines

@@ -329,6 +329,9 @@
         return $rows;
     }
 
+    /**
+     * @param DataManager $dataManager
+     */
     public static function drawCol($item, $colName, $params = [], $dataManager = null, $originalCol = '', $originalItem = null)
     {
         $modelName = get_class($item);

Indentation +6 added lines, -6 removed lines

@@ -164,12 +164,12 @@
             <li><a href ='' onclick='inji.Ui.dataManagers.get(this).rowSelection("inverse");return false;'>Инвертировать</a></li>
             <li role="separator" class="divider"></li>
               <?php
-              foreach ($actions as $action => $actionParams) {
-                  if (class_exists($actionParams['className']) && $actionParams['className']::$groupAction) {
-                      echo "<li><a href ='' onclick='inji.Ui.dataManagers.get(this).groupAction(\"" . str_replace('\\', '\\\\', $action) . "\");return false;'>{$actionParams['className']::$name}</a></li>";
-                  }
-              }
-              ?>
+                foreach ($actions as $action => $actionParams) {
+                    if (class_exists($actionParams['className']) && $actionParams['className']::$groupAction) {
+                        echo "<li><a href ='' onclick='inji.Ui.dataManagers.get(this).groupAction(\"" . str_replace('\\', '\\\\', $action) . "\");return false;'>{$actionParams['className']::$name}</a></li>";
+                    }
+                }
+                ?>
           </ul>
         </div>
         <?php

Spacing +30 added lines, -30 removed lines

@@ -58,7 +58,7 @@ 
             $this->name = $this->managerOptions['name'];
         }
 
-        $this->managerId = str_replace('\\', '_', 'dataManager_' . $this->modelName . '_' . $this->managerName . '_' . \Tools::randomString());
+        $this->managerId = str_replace('\\', '_', 'dataManager_'.$this->modelName.'_'.$this->managerName.'_'.\Tools::randomString());
     }
 
     /**
@@ -102,13 +102,13 @@ 
         if (!empty($this->managerOptions['filters'])) {
             $buttons[] = [
                 'text' => 'Фильтры',
-                'onclick' => '  var modal = $("#' . $this->managerId . '_filters");
+                'onclick' => '  var modal = $("#'.$this->managerId.'_filters");
                 modal.modal("show");',
             ];
         }
         $buttons[] = [
             'text' => 'Добавить элемент',
-            'onclick' => 'inji.Ui.forms.popUp("' . str_replace('\\', '\\\\', $modelName) . '",' . json_encode($formParams) . ')',
+            'onclick' => 'inji.Ui.forms.popUp("'.str_replace('\\', '\\\\', $modelName).'",'.json_encode($formParams).')',
         ];
 
         return $buttons;
@@ -138,7 +138,7 @@ 
                     'className' => $action
                 ];
             }
-            $return[$key]['className'] = strpos($return[$key]['className'], '\\') === false && class_exists('Ui\DataManager\Action\\' . $return[$key]['className']) ? 'Ui\DataManager\Action\\' . $return[$key]['className'] : $return[$key]['className'];
+            $return[$key]['className'] = strpos($return[$key]['className'], '\\') === false && class_exists('Ui\DataManager\Action\\'.$return[$key]['className']) ? 'Ui\DataManager\Action\\'.$return[$key]['className'] : $return[$key]['className'];
             if (!class_exists($return[$key]['className']) || ($onlyGroupActions && !$return[$key]['className']::$groupAction)) {
                 unset($return[$key]);
             }
@@ -170,7 +170,7 @@ 
               <?php
               foreach ($actions as $action => $actionParams) {
                   if (class_exists($actionParams['className']) && $actionParams['className']::$groupAction) {
-                      echo "<li><a href ='' onclick='inji.Ui.dataManagers.get(this).groupAction(\"" . str_replace('\\', '\\\\', $action) . "\");return false;'>{$actionParams['className']::$name}</a></li>";
+                      echo "<li><a href ='' onclick='inji.Ui.dataManagers.get(this).groupAction(\"".str_replace('\\', '\\\\', $action)."\");return false;'>{$actionParams['className']::$name}</a></li>";
                   }
               }
               ?>
@@ -221,7 +221,7 @@ 
             return [];
         }
         if (!$this->checkAccess()) {
-            $this->drawError('you not have access to "' . $this->modelName . '" manager with name: "' . $this->managerName . '"');
+            $this->drawError('you not have access to "'.$this->modelName.'" manager with name: "'.$this->managerName.'"');
             return [];
         }
         $modelName = $this->modelName;
@@ -237,7 +237,7 @@ 
             $queryParams['start'] = $this->page * $this->limit - $this->limit;
         }
         if (!empty($params['categoryPath']) && $modelName::$categoryModel) {
-            $queryParams['where'][] = ['tree_path', $params['categoryPath'] . '%', 'LIKE'];
+            $queryParams['where'][] = ['tree_path', $params['categoryPath'].'%', 'LIKE'];
         }
         if (!empty($params['appType'])) {
             $queryParams['appType'] = $params['appType'];
@@ -296,7 +296,7 @@ 
                         if (!empty($params['filters'][$col]['max'])) {
                             if ($colInfo['colParams']['type'] == 'dateTime' && !strpos($params['filters'][$col]['max'], ' ')) {
 
-                                $date = $params['filters'][$col]['max'] . ' 23:59:59';
+                                $date = $params['filters'][$col]['max'].' 23:59:59';
                             } else {
                                 $date = $params['filters'][$col]['max'];
                             }
@@ -323,16 +323,16 @@ 
                         }
                         switch ($params['filters'][$col]['compareType']) {
                             case 'contains':
-                                $queryParams['where'][] = [$col, '%' . $params['filters'][$col]['value'] . '%', 'LIKE'];
+                                $queryParams['where'][] = [$col, '%'.$params['filters'][$col]['value'].'%', 'LIKE'];
                                 break;
                             case 'equals':
                                 $queryParams['where'][] = [$col, $params['filters'][$col]['value']];
                                 break;
                             case 'starts_with':
-                                $queryParams['where'][] = [$col, $params['filters'][$col]['value'] . '%', 'LIKE'];
+                                $queryParams['where'][] = [$col, $params['filters'][$col]['value'].'%', 'LIKE'];
                                 break;
                             case 'ends_with':
-                                $queryParams['where'][] = [$col, '%' . $params['filters'][$col]['value'], 'LIKE'];
+                                $queryParams['where'][] = [$col, '%'.$params['filters'][$col]['value'], 'LIKE'];
                                 break;
                         }
                         break;
@@ -371,7 +371,7 @@ 
                 $item = $relation['relModel']::get([[$item->index(), $item->id], [$model->index(), $model->id]]);
             }
             $row = [];
-            $row[] = '<input type ="checkbox" name = "pk[]" value =' . $item->pk() . '>';
+            $row[] = '<input type ="checkbox" name = "pk[]" value ='.$item->pk().'>';
             $row[] = $item->pk();
             foreach ($this->managerOptions['cols'] as $key => $colName) {
                 if (!empty($params['download'])) {
@@ -417,16 +417,16 @@ 
                     $managerParams = ['relation' => $modelName::$cols[$colName]['relation']];
                     $count = $item->{$modelName::$cols[$colName]['relation']}(array_merge($params, ['count' => 1]));
                     $count = $count ? $count : 'Нет';
-                    return "<a class = 'btn btn-xs btn-primary' onclick = 'inji.Ui.dataManagers.popUp(\"" . str_replace('\\', '\\\\', $modelName) . ":" . $item->pk() . "\"," . json_encode(array_merge($params, $managerParams)) . ")'>{$count}</a>";
+                    return "<a class = 'btn btn-xs btn-primary' onclick = 'inji.Ui.dataManagers.popUp(\"".str_replace('\\', '\\\\', $modelName).":".$item->pk()."\",".json_encode(array_merge($params, $managerParams)).")'>{$count}</a>";
                 case 'many':
                     $managerParams = ['relation' => $modelName::$cols[$colName]['relation']];
                     $count = $item->{$modelName::$cols[$colName]['relation']}(array_merge($params, ['count' => 1]));
                     $count = $count ? $count : 'Нет';
-                    return "<a class = 'btn btn-xs btn-primary' onclick = 'inji.Ui.dataManagers.popUp(\"" . str_replace('\\', '\\\\', $modelName) . ":" . $item->pk() . "\"," . json_encode(array_merge($params, $managerParams)) . ")'>{$count}</a>";
+                    return "<a class = 'btn btn-xs btn-primary' onclick = 'inji.Ui.dataManagers.popUp(\"".str_replace('\\', '\\\\', $modelName).":".$item->pk()."\",".json_encode(array_merge($params, $managerParams)).")'>{$count}</a>";
                 default :
                     if ($item->{$modelName::$cols[$colName]['relation']}) {
                         if (\App::$cur->name == 'admin') {
-                            $href = "<a href ='/admin/" . str_replace('\\', '/view/', $relations[$modelName::$cols[$colName]['relation']]['model']) . "/" . $item->{$modelName::$cols[$colName]['relation']}->pk() . "'>";
+                            $href = "<a href ='/admin/".str_replace('\\', '/view/', $relations[$modelName::$cols[$colName]['relation']]['model'])."/".$item->{$modelName::$cols[$colName]['relation']}->pk()."'>";
                             if (!empty($modelName::$cols[$colName]['showCol'])) {
                                 $href .= $item->{$modelName::$cols[$colName]['relation']}->{$modelName::$cols[$colName]['showCol']};
                             } else {
@@ -456,18 +456,18 @@ 
                     case 'many':
                         $managerParams = ['relation' => $modelName::$cols[$colName]['relation']];
                         $count = $item->{$modelName::$cols[$colName]['relation']}(array_merge($params, ['count' => 1]));
-                        return "<a class = 'btn btn-xs btn-primary' onclick = 'inji.Ui.dataManagers.popUp(\"" . str_replace('\\', '\\\\', $modelName) . ":" . $item->pk() . "\"," . json_encode(array_merge($params, $managerParams)) . ")'>{$count} " . \Tools::getNumEnding($count, ['Элемент', 'Элемента', 'Элементов']) . "</a>";
+                        return "<a class = 'btn btn-xs btn-primary' onclick = 'inji.Ui.dataManagers.popUp(\"".str_replace('\\', '\\\\', $modelName).":".$item->pk()."\",".json_encode(array_merge($params, $managerParams)).")'>{$count} ".\Tools::getNumEnding($count, ['Элемент', 'Элемента', 'Элементов'])."</a>";
                     default:
                         return $item->$colName;
                 }
             } elseif (!empty($modelName::$cols[$colName]['type'])) {
-                if (\App::$cur->name == 'admin' && $originalCol == 'name' || ( $dataManager && !empty($dataManager->managerOptions['colToView']) && $dataManager->managerOptions['colToView'] == $originalCol)) {
+                if (\App::$cur->name == 'admin' && $originalCol == 'name' || ($dataManager && !empty($dataManager->managerOptions['colToView']) && $dataManager->managerOptions['colToView'] == $originalCol)) {
                     $formName = $dataManager && !empty($dataManager->managerOptions['editForm']) ? $dataManager->managerOptions['editForm'] : 'manager';
-                    $redirectUrl = !empty($_SERVER['HTTP_REFERER']) ? $_SERVER['HTTP_REFERER'] : '/admin/' . str_replace('\\', '/', get_class($originalItem));
-                    return "<a href ='/admin/" . str_replace('\\', '/view/', get_class($originalItem)) . "/{$originalItem->id}?formName={$formName}&redirectUrl={$redirectUrl}'>{$item->$colName}</a>";
+                    $redirectUrl = !empty($_SERVER['HTTP_REFERER']) ? $_SERVER['HTTP_REFERER'] : '/admin/'.str_replace('\\', '/', get_class($originalItem));
+                    return "<a href ='/admin/".str_replace('\\', '/view/', get_class($originalItem))."/{$originalItem->id}?formName={$formName}&redirectUrl={$redirectUrl}'>{$item->$colName}</a>";
                 } elseif (\App::$cur->name == 'admin' && $colName == 'name') {
-                    $redirectUrl = !empty($_SERVER['HTTP_REFERER']) ? $_SERVER['HTTP_REFERER'] : '/admin/' . str_replace('\\', '/', get_class($originalItem));
-                    return "<a href ='/admin/" . str_replace('\\', '/view/', get_class($item)) . "/{$item->id}?redirectUrl={$redirectUrl}'>{$item->$colName}</a>";
+                    $redirectUrl = !empty($_SERVER['HTTP_REFERER']) ? $_SERVER['HTTP_REFERER'] : '/admin/'.str_replace('\\', '/', get_class($originalItem));
+                    return "<a href ='/admin/".str_replace('\\', '/view/', get_class($item))."/{$item->id}?redirectUrl={$redirectUrl}'>{$item->$colName}</a>";
                 } else {
                     return \Model::resloveTypeValue($item, $colName);
                 }
@@ -502,7 +502,7 @@ 
             return [];
         }
         if (!$this->checkAccess()) {
-            $this->drawError('you not have access to "' . $this->modelName . '" manager with name: "' . $this->managerName . '"');
+            $this->drawError('you not have access to "'.$this->modelName.'" manager with name: "'.$this->managerName.'"');
             return [];
         }
         if (!empty($params['limit'])) {
@@ -516,7 +516,7 @@ 
         ];
         $modelName = $this->modelName;
         if (!empty($params['categoryPath']) && $modelName::$categoryModel) {
-            $queryParams['where'][] = ['tree_path', $params['categoryPath'] . '%', 'LIKE'];
+            $queryParams['where'][] = ['tree_path', $params['categoryPath'].'%', 'LIKE'];
         }
         if (!empty($this->managerOptions['userGroupFilter'][\Users\User::$cur->group_id]['getRows'])) {
             foreach ($this->managerOptions['userGroupFilter'][\Users\User::$cur->group_id]['getRows'] as $colName => $colOptions) {
@@ -570,7 +570,7 @@ 
                         if (!empty($params['filters'][$col]['max'])) {
                             if ($colInfo['colParams']['type'] == 'dateTime' && !strpos($params['filters'][$col]['max'], ' ')) {
 
-                                $date = $params['filters'][$col]['max'] . ' 23:59:59';
+                                $date = $params['filters'][$col]['max'].' 23:59:59';
                             } else {
                                 $date = $params['filters'][$col]['max'];
                             }
@@ -597,16 +597,16 @@ 
                         }
                         switch ($params['filters'][$col]['compareType']) {
                             case 'contains':
-                                $queryParams['where'][] = [$col, '%' . $params['filters'][$col]['value'] . '%', 'LIKE'];
+                                $queryParams['where'][] = [$col, '%'.$params['filters'][$col]['value'].'%', 'LIKE'];
                                 break;
                             case 'equals':
                                 $queryParams['where'][] = [$col, $params['filters'][$col]['value']];
                                 break;
                             case 'starts_with':
-                                $queryParams['where'][] = [$col, $params['filters'][$col]['value'] . '%', 'LIKE'];
+                                $queryParams['where'][] = [$col, $params['filters'][$col]['value'].'%', 'LIKE'];
                                 break;
                             case 'ends_with':
-                                $queryParams['where'][] = [$col, '%' . $params['filters'][$col]['value'], 'LIKE'];
+                                $queryParams['where'][] = [$col, '%'.$params['filters'][$col]['value'], 'LIKE'];
                                 break;
                         }
                         break;
@@ -640,7 +640,7 @@ 
             $tableCols[] = !empty($colOptions['label']) ? $colOptions['label'] : $colName;
         }
         $tableCols[] = '';
-        $this->table->class .=' datamanagertable';
+        $this->table->class .= ' datamanagertable';
         $this->table->setCols($tableCols);
     }
 
@@ -663,12 +663,12 @@ 
             return false;
         }
         if (!$this->checkAccess()) {
-            $this->drawError('you not have access to "' . $this->modelName . '" manager with name: "' . $this->managerName . '"');
+            $this->drawError('you not have access to "'.$this->modelName.'" manager with name: "'.$this->managerName.'"');
             return [];
         }
         $tree = new Tree();
         $tree->ul($this->managerOptions['categorys']['model'], 0, function($category) {
-            $path = $category->tree_path . ($category->pk() ? $category->pk() . "/" : '');
+            $path = $category->tree_path.($category->pk() ? $category->pk()."/" : '');
             $cleanClassName = str_replace('\\', '\\\\', get_class($category));
             return "<a href='#' onclick='inji.Ui.dataManagers.get(this).switchCategory(this);return false;' data-index='{$category->index()}' data-path ='{$path}' data-id='{$category->pk()}'> {$category->name}</a> 
                 

system/modules/Ecommerce/appControllers/CartController.php

Braces +5 added lines, -4 removed lines

@@ -270,10 +270,11 @@
         $item->sales ++;
         $item->save();
 
-        if (empty($_GET['count']))
-            $count = 1;
-        else
-            $count = (float) $_GET['count'];
+        if (empty($_GET['count'])) {
+                    $count = 1;
+        } else {
+                    $count = (float) $_GET['count'];
+        }
 
         $cart = $this->ecommerce->getCurCart();
 

Spacing +13 added lines, -13 removed lines

@@ -62,7 +62,7 @@ 
                         $warecount = $cartitem->price->offer->warehouseCount($cart->id);
                         if ($cartitem->count > $warecount) {
                             $error = true;
-                            Msg::add('Вы заказали <b>' . $cartitem->item->name . '</b> больше чем есть на складе. на складе: <b>' . $warecount . '</b>', 'danger');
+                            Msg::add('Вы заказали <b>'.$cartitem->item->name.'</b> больше чем есть на складе. на складе: <b>'.$warecount.'</b>', 'danger');
                         }
                     }
                 }
@@ -74,7 +74,7 @@ 
                     foreach ($deliverys[$cart->delivery_id]->fields as $field) {
                         if (empty($_POST['deliveryFields'][$field->id]) && $field->required) {
                             $error = 1;
-                            Msg::add('Вы не указали: ' . $field->name);
+                            Msg::add('Вы не указали: '.$field->name);
                         }
                     }
                 }
@@ -90,7 +90,7 @@ 
                 foreach (\Ecommerce\UserAdds\Field::getList() as $field) {
                     if (empty($_POST['userAdds']['fields'][$field->id]) && $field->required) {
                         $error = 1;
-                        Msg::add('Вы не указали: ' . $field->name);
+                        Msg::add('Вы не указали: '.$field->name);
                     }
                 }
                 if (!empty($_POST['discounts']['card_item_id'])) {
@@ -127,13 +127,13 @@ 
                     }
                     $cart = \Ecommerce\Cart::get($cart->id);
                     if (!empty(\App::$cur->ecommerce->config['notify_mail'])) {
-                        $text = 'Перейдите в админ панель чтобы просмотреть новый заказ <a href = "http://' . idn_to_utf8(INJI_DOMAIN_NAME) . '/admin/ecommerce/Cart">Админ панель</a>';
-                        $title = 'Новый заказ в интернет магазине на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME);
-                        \Tools::sendMail('noreply@' . INJI_DOMAIN_NAME, \App::$cur->ecommerce->config['notify_mail'], $title, $text);
+                        $text = 'Перейдите в админ панель чтобы просмотреть новый заказ <a href = "http://'.idn_to_utf8(INJI_DOMAIN_NAME).'/admin/ecommerce/Cart">Админ панель</a>';
+                        $title = 'Новый заказ в интернет магазине на сайте '.idn_to_utf8(INJI_DOMAIN_NAME);
+                        \Tools::sendMail('noreply@'.INJI_DOMAIN_NAME, \App::$cur->ecommerce->config['notify_mail'], $title, $text);
                     }
                     if ($this->notifications) {
                         $notification = new Notifications\Notification();
-                        $notification->name = 'Новый заказ в интернет магазине на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME);
+                        $notification->name = 'Новый заказ в интернет магазине на сайте '.idn_to_utf8(INJI_DOMAIN_NAME);
                         $notification->text = 'Перейдите в админ панель чтобы просмотреть новый заказ';
                         $notification->chanel_id = $this->notifications->getChanel('Ecommerce-orders')->id;
                         $notification->save();
@@ -179,10 +179,10 @@ 
             'href' => '/ecommerce/cart'
         ];
         $bread[] = [
-            'text' => 'Заказ: №' . $cart->id,
-            'href' => '/ecommerce/cart/orderDetail/' . $cart->id
+            'text' => 'Заказ: №'.$cart->id,
+            'href' => '/ecommerce/cart/orderDetail/'.$cart->id
         ];
-        $this->view->setTitle('Заказ №' . $cart->id);
+        $this->view->setTitle('Заказ №'.$cart->id);
         $this->view->page(['data' => compact('cart', 'bread')]);
     }
 
@@ -275,7 +275,7 @@ 
             $result->send();
         }
 
-        $item->sales ++;
+        $item->sales++;
         $item->save();
 
         if (empty($_GET['count']))
@@ -288,7 +288,7 @@ 
 
         if (empty($this->module->config['sell_over_warehouse']) && $price->offer->warehouseCount() < $count) {
             $result->success = false;
-            $result->content = 'На складе недостаточно товара! Доступно: ' . $price->offer->warehouseCount();
+            $result->content = 'На складе недостаточно товара! Доступно: '.$price->offer->warehouseCount();
             $result->send();
         }
 
@@ -305,7 +305,7 @@ 
             $cart->addItem($price->id, $count);
         }
         $cart->calc();
-        $result->successMsg = '<a href="/ecommerce/view/' . $item->id . '">' . $item->name() . ($price->offer->name() ? ' (' . $price->offer->name() . ')' : '') . '</a> добавлен <a href="/ecommerce/cart">в корзину покупок</a>!';
+        $result->successMsg = '<a href="/ecommerce/view/'.$item->id.'">'.$item->name().($price->offer->name() ? ' ('.$price->offer->name().')' : '').'</a> добавлен <a href="/ecommerce/cart">в корзину покупок</a>!';
         $result->send();
     }
 

system/modules/Ui/appAdminControllers/content/dataManager/view.php

Indentation +12 added lines, -12 removed lines

@@ -58,11 +58,11 @@ 
     ]]);
     ?>)</h3>
   <?php
-  foreach (\Dashboard\Comment::getList([ 'where' => [
-          ['item_id', $item->id],
-          ['model', $modelName],
-      ], 'order' => ['date_create', 'desc']]) as $comment) {
-      ?>
+    foreach (\Dashboard\Comment::getList([ 'where' => [
+            ['item_id', $item->id],
+            ['model', $modelName],
+        ], 'order' => ['date_create', 'desc']]) as $comment) {
+        ?>
       <div class="row">
         <div class="col-sm-3" style="max-width: 300px;">
           <a href='/admin/Users/view/User/<?= $comment->user->pk(); ?>'><?= $comment->user->name(); ?></a><br />
@@ -73,14 +73,14 @@ 
         </div>
       </div>
       <?php
-  }
-  ?>
+    }
+    ?>
 </div>
 <div>
   <?php
-  $form = new \Ui\Form();
-  $form->begin();
-  $form->input('textarea', 'comment', 'Комментарий');
-  $form->end();
-  ?>
+    $form = new \Ui\Form();
+    $form->begin();
+    $form->input('textarea', 'comment', 'Комментарий');
+    $form->end();
+    ?>
 </div>

Spacing +3 added lines, -3 removed lines

@@ -24,14 +24,14 @@ 
                 case 'relation':
                     $relations = $colInfo['modelName']::relations();
                     $relValue = $relations[$colInfo['colParams']['relation']]['model']::get($item->$colName);
-                    $value = $relValue ? "<a href='/admin/" . str_replace('\\', '/view/', $relations[$colInfo['colParams']['relation']]['model']) . "/" . $relValue->pk() . "'>" . $relValue->name() . "</a>" : 'Не задано';
+                    $value = $relValue ? "<a href='/admin/".str_replace('\\', '/view/', $relations[$colInfo['colParams']['relation']]['model'])."/".$relValue->pk()."'>".$relValue->name()."</a>" : 'Не задано';
                     break;
             }
             break;
         case 'image':
             $file = Files\File::get($item->$colName);
             if ($file) {
-                $value = '<img src="' . $file->path . '?resize=60x120" />';
+                $value = '<img src="'.$file->path.'?resize=60x120" />';
             } else {
                 $value = '<img src="/static/system/images/no-image.png?resize=60x120" />';
             }
@@ -58,7 +58,7 @@ 
     ]]);
     ?>)</h3>
   <?php
-  foreach (\Dashboard\Comment::getList([ 'where' => [
+  foreach (\Dashboard\Comment::getList(['where' => [
           ['item_id', $item->id],
           ['model', $modelName],
       ], 'order' => ['date_create', 'desc']]) as $comment) {

system/modules/Ui/widgets/DataManager/rowButtons.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 $buttons = [];
 foreach ($actions as $action => $actionParams) {
     if (class_exists($actionParams['className']) && $actionParams['className']::$rowAction) {
-        $buttons[]= $actionParams['className']::rowButton($dataManager, $item, $params, $actionParams);
+        $buttons[] = $actionParams['className']::rowButton($dataManager, $item, $params, $actionParams);
     }
 }
 echo implode(' ', $buttons);

system/modules/Users/Users.php

Doc Comments +9 added lines

@@ -92,6 +92,9 @@ 
         }
     }
 
+    /**
+     * @param string $user_mail
+     */
     public function passre($user_mail)
     {
         $user = $this->get($user_mail, 'mail');
@@ -322,6 +325,9 @@ 
         return $user->id;
     }
 
+    /**
+     * @param string $pass
+     */
     public function hashpass($pass)
     {
         return password_hash($pass, PASSWORD_DEFAULT);
@@ -359,6 +365,9 @@ 
         return $return;
     }
 
+    /**
+     * @param integer $cat_id
+     */
     public function addUserActivity($user_id, $cat_id , $text = '')
     {
         $ua = new Users\Activity([

Braces +13 added lines, -10 removed lines

@@ -196,17 +196,20 @@
      */

     public function get($idn, $ltype = 'id')

     {

-        if (!$idn)

-            return false;

+        if (!$idn) {
+                    return false;
+        }
 

-        if (is_numeric($idn) && $ltype != 'login')

-            $user = Users\User::get($idn, 'id');

-        elseif ($ltype == 'login')

-            $user = Users\User::get($idn, 'login');

-        else

-            $user = Users\User::get($idn, 'mail');

-        if (!$user)

-            return [];

+        if (is_numeric($idn) && $ltype != 'login') {
+                    $user = Users\User::get($idn, 'id');
+        } elseif ($ltype == 'login') {
+                    $user = Users\User::get($idn, 'login');
+        } else {
+                    $user = Users\User::get($idn, 'mail');
+        }
+        if (!$user) {
+                    return [];
+        }
 

         return $user;

     }

Spacing +34 added lines, -34 removed lines

@@ -24,8 +24,8 @@ 
         if (isset($_GET['logout'])) {

             return $this->logOut();

         }

-        if (filter_input(INPUT_COOKIE, $this->cookiePrefix . '_user_session_hash') && filter_input(INPUT_COOKIE, $this->cookiePrefix . '_user_id')) {

-            return $this->cuntinueSession(filter_input(INPUT_COOKIE, $this->cookiePrefix . '_user_session_hash'), filter_input(INPUT_COOKIE, $this->cookiePrefix . '_user_id'));

+        if (filter_input(INPUT_COOKIE, $this->cookiePrefix.'_user_session_hash') && filter_input(INPUT_COOKIE, $this->cookiePrefix.'_user_id')) {

+            return $this->cuntinueSession(filter_input(INPUT_COOKIE, $this->cookiePrefix.'_user_session_hash'), filter_input(INPUT_COOKIE, $this->cookiePrefix.'_user_id'));

         }

         if (isset($_POST['autorization']) && trim(filter_input(INPUT_POST, 'user_login')) && trim(filter_input(INPUT_POST, 'user_pass'))) {

             unset($_POST['autorization']);

@@ -41,18 +41,18 @@ 
 

     public function logOut($redirect = true)

     {

-        if (!empty($_COOKIE[$this->cookiePrefix . "_user_session_hash"]) && !empty($_COOKIE[$this->cookiePrefix . "_user_id"])) {

+        if (!empty($_COOKIE[$this->cookiePrefix."_user_session_hash"]) && !empty($_COOKIE[$this->cookiePrefix."_user_id"])) {

             $session = Users\Session::get([

-                        ['user_id', $_COOKIE[$this->cookiePrefix . "_user_id"]],

-                        ['hash', $_COOKIE[$this->cookiePrefix . "_user_session_hash"]]

+                        ['user_id', $_COOKIE[$this->cookiePrefix."_user_id"]],

+                        ['hash', $_COOKIE[$this->cookiePrefix."_user_session_hash"]]

             ]);

             if ($session) {

                 $session->delete();

             }

         }

         if (!headers_sent()) {

-            setcookie($this->cookiePrefix . "_user_session_hash", '', 0, "/");

-            setcookie($this->cookiePrefix . "_user_id", '', 0, "/");

+            setcookie($this->cookiePrefix."_user_session_hash", '', 0, "/");

+            setcookie($this->cookiePrefix."_user_id", '', 0, "/");

         }

         if ($redirect) {

             if (!empty($this->config['logoutUrl'][$this->app->type])) {

@@ -70,25 +70,25 @@ 
         ]);

         if ($session && $session->user && $session->user->blocked) {

             if (!headers_sent()) {

-                setcookie($this->cookiePrefix . "_user_session_hash", '', 0, "/");

-                setcookie($this->cookiePrefix . "_user_id", '', 0, "/");

+                setcookie($this->cookiePrefix."_user_session_hash", '', 0, "/");

+                setcookie($this->cookiePrefix."_user_id", '', 0, "/");

             }

             Msg::add('Ваш аккаунт заблокирован', 'info');

             return;

         }

         if ($session && $session->user && !$session->user->blocked) {

             if (!headers_sent()) {

-                setcookie($this->cookiePrefix . "_user_session_hash", $session->hash, time() + 360000, "/");

-                setcookie($this->cookiePrefix . "_user_id", $session->user_id, time() + 360000, "/");

+                setcookie($this->cookiePrefix."_user_session_hash", $session->hash, time() + 360000, "/");

+                setcookie($this->cookiePrefix."_user_id", $session->user_id, time() + 360000, "/");

             }

             if (!empty($this->config['needActivation']) && $session->user->activation) {

                 if (!headers_sent()) {

-                    setcookie($this->cookiePrefix . "_user_session_hash", '', 0, "/");

-                    setcookie($this->cookiePrefix . "_user_id", '', 0, "/");

+                    setcookie($this->cookiePrefix."_user_session_hash", '', 0, "/");

+                    setcookie($this->cookiePrefix."_user_id", '', 0, "/");

                 }

-                Tools::redirect('/', 'Этот аккаунт ещё не активирован. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/' . $session->user->id . '"><b>повторно выслать ссылку активации</b></a>');

+                Tools::redirect('/', 'Этот аккаунт ещё не активирован. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/'.$session->user->id.'"><b>повторно выслать ссылку активации</b></a>');

             } elseif ($session->user->activation) {

-                Msg::add('Этот аккаунт ещё не активирован, не все функции могут быть доступны. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/' . $session->user->id . '"><b>повторно выслать ссылку активации</b></a>');

+                Msg::add('Этот аккаунт ещё не активирован, не все функции могут быть доступны. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/'.$session->user->id.'"><b>повторно выслать ссылку активации</b></a>');

             }

             if (!$session->user->mail && !empty($this->config['noMailNotify'])) {

                 Msg::add($this->config['noMailNotify']);

@@ -98,8 +98,8 @@ 
             Users\User::$cur->save();

         } else {

             if (!headers_sent()) {

-                setcookie($this->cookiePrefix . "_user_session_hash", '', 0, "/");

-                setcookie($this->cookiePrefix . "_user_id", '', 0, "/");

+                setcookie($this->cookiePrefix."_user_session_hash", '', 0, "/");

+                setcookie($this->cookiePrefix."_user_id", '', 0, "/");

             }

             Msg::add('Ваша сессия устарела или более недействительна, вам необходимо пройти <a href = "/users/login">авторизацию</a> заново', 'info');

         }

@@ -109,7 +109,7 @@ 
     {

         $user = $this->get($user_mail, 'mail');

         if (!$user) {

-            Msg::add('Пользователь ' . $user_mail . ' не найден, проверьте првильность ввода e-mail или зарегистрируйтесь', 'danger');

+            Msg::add('Пользователь '.$user_mail.' не найден, проверьте првильность ввода e-mail или зарегистрируйтесь', 'danger');

             return false;

         }

         $passre = Users\Passre::get([['user_id', $user->id], ['status', 1]]);

@@ -117,10 +117,10 @@ 
             $passre->status = 2;

             $passre->save();

         }

-        $hash = $user->id . '_' . Tools::randomString(50);

+        $hash = $user->id.'_'.Tools::randomString(50);

         $passre = new Users\Passre(['user_id' => $user->id, 'status' => 1, 'hash' => $hash]);

         $passre->save();

-        Tools::sendMail('noreply@' . INJI_DOMAIN_NAME, $user_mail, 'Восстановление пароля на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME), 'Было запрошено восстановление пароля на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME) . '<br />для продолжения восстановления пароля перейдите по ссылке: <a href = "http://' . idn_to_utf8(INJI_DOMAIN_NAME) . '/?passrecont=1&hash=' . $hash . '">' . idn_to_utf8(INJI_DOMAIN_NAME) . '/?passrecont=1&hash=' . $hash . '</a>');

+        Tools::sendMail('noreply@'.INJI_DOMAIN_NAME, $user_mail, 'Восстановление пароля на сайте '.idn_to_utf8(INJI_DOMAIN_NAME), 'Было запрошено восстановление пароля на сайте '.idn_to_utf8(INJI_DOMAIN_NAME).'<br />для продолжения восстановления пароля перейдите по ссылке: <a href = "http://'.idn_to_utf8(INJI_DOMAIN_NAME).'/?passrecont=1&hash='.$hash.'">'.idn_to_utf8(INJI_DOMAIN_NAME).'/?passrecont=1&hash='.$hash.'</a>');

         Tools::redirect('/', 'На указанный почтовый ящик была выслана инструкция по восстановлению пароля', 'success');

     }

 

@@ -138,7 +138,7 @@ 
             $user->pass = $this->hashpass($pass);

             $user->save();

             $this->autorization($user->mail, $user->pass, 'mail');

-            Tools::sendMail('noreply@' . INJI_DOMAIN_NAME, $user->mail, 'Новый пароль на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME), 'Было запрошено восстановление пароля на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME) . '<br />Ваш новый пароль: ' . $pass);

+            Tools::sendMail('noreply@'.INJI_DOMAIN_NAME, $user->mail, 'Новый пароль на сайте '.idn_to_utf8(INJI_DOMAIN_NAME), 'Было запрошено восстановление пароля на сайте '.idn_to_utf8(INJI_DOMAIN_NAME).'<br />Ваш новый пароль: '.$pass);

             Tools::redirect('/', 'На указанный почтовый ящик был выслан новый пароль', 'success');

         }

     }

@@ -151,9 +151,9 @@ 
         $user = $this->get($login, $ltype);

         if ($user && $this->verifypass($pass, $user->pass) && !$user->blocked) {

             if (!empty($this->config['needActivation']) && $user->activation) {

-                Tools::redirect('/', 'Этот аккаунт ещё не активирован. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/' . $user->id . '"><b>повторно выслать ссылку активации</b></a>');

+                Tools::redirect('/', 'Этот аккаунт ещё не активирован. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/'.$user->id.'"><b>повторно выслать ссылку активации</b></a>');

             } elseif ($user->activation) {

-                Msg::add('Этот аккаунт ещё не активирован, не все функции могут быть доступны. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/' . $user->id . '"><b>повторно выслать ссылку активации</b></a>');

+                Msg::add('Этот аккаунт ещё не активирован, не все функции могут быть доступны. <br />Если вы не получали письмо с ссылкой для активации, нажмите на - <a href = "/users/resendActivation/'.$user->id.'"><b>повторно выслать ссылку активации</b></a>');

             }

             if (!$user->mail && !empty($this->config['noMailNotify'])) {

                 Msg::add($this->config['noMailNotify']);

@@ -172,7 +172,7 @@ 
             if ($user && $user->blocked) {

                 Msg::add('Вы заблокированы', 'danger');

             } elseif ($user) {

-                Msg::add('Вы ошиблись при наборе пароля или логина, попробуйте ещё раз или воспользуйтесь <a href = "?passre=1&user_mail=' . $user->mail . '">Восстановлением пароля</a>', 'danger');

+                Msg::add('Вы ошиблись при наборе пароля или логина, попробуйте ещё раз или воспользуйтесь <a href = "?passre=1&user_mail='.$user->mail.'">Восстановлением пароля</a>', 'danger');

             } else {

                 Msg::add('Данный почтовый ящик не зарегистрирован в системе', 'danger');

             }

@@ -195,8 +195,8 @@ 
         $session->save();

 

         if (!headers_sent()) {

-            setcookie($this->cookiePrefix . "_user_session_hash", $session->hash, time() + 360000, "/");

-            setcookie($this->cookiePrefix . "_user_id", $session->user_id, time() + 360000, "/");

+            setcookie($this->cookiePrefix."_user_session_hash", $session->hash, time() + 360000, "/");

+            setcookie($this->cookiePrefix."_user_id", $session->user_id, time() + 360000, "/");

         } else {

             Msg::add('Не удалось провести авторизацию. Попробуйте позже', 'info');

         }

@@ -318,20 +318,20 @@ 
             $this->autorization($user_mail, $pass, 'mail');

         }

         if (!empty($this->config['needActivation'])) {

-            $from = 'noreply@' . INJI_DOMAIN_NAME;

+            $from = 'noreply@'.INJI_DOMAIN_NAME;

             $to = $user_mail;

-            $subject = 'Регистрация на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME);

-            $text = 'Вы были зарегистрированы на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME) . '<br />для входа используйте ваш почтовый ящик в качестве логина и пароль: ' . $pass;

-            $text .='<br />';

+            $subject = 'Регистрация на сайте '.idn_to_utf8(INJI_DOMAIN_NAME);

+            $text = 'Вы были зарегистрированы на сайте '.idn_to_utf8(INJI_DOMAIN_NAME).'<br />для входа используйте ваш почтовый ящик в качестве логина и пароль: '.$pass;

             $text .= '<br />';

-            $text .= 'Для активации вашего аккаунта перейдите по ссылке <a href = "http://' . INJI_DOMAIN_NAME . '/users/activation/' . $user->id . '/' . $user->activation . '">http://' . idn_to_utf8(INJI_DOMAIN_NAME) . '/users/activation/' . $user->id . '/' . $user->activation . '</a>';

+            $text .= '<br />';

+            $text .= 'Для активации вашего аккаунта перейдите по ссылке <a href = "http://'.INJI_DOMAIN_NAME.'/users/activation/'.$user->id.'/'.$user->activation.'">http://'.idn_to_utf8(INJI_DOMAIN_NAME).'/users/activation/'.$user->id.'/'.$user->activation.'</a>';

             Tools::sendMail($from, $to, $subject, $text);

             Msg::add('Вы были зарегистрированы. На указанный почтовый ящик был выслан ваш пароль и ссылка для активации', 'success');

         } else {

-            $from = 'noreply@' . INJI_DOMAIN_NAME;

+            $from = 'noreply@'.INJI_DOMAIN_NAME;

             $to = $user_mail;

-            $subject = 'Регистрация на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME);

-            $text = 'Вы были зарегистрированы на сайте ' . idn_to_utf8(INJI_DOMAIN_NAME) . '<br />для входа используйте ваш почтовый ящик в качестве логина и пароль: ' . $pass;

+            $subject = 'Регистрация на сайте '.idn_to_utf8(INJI_DOMAIN_NAME);

+            $text = 'Вы были зарегистрированы на сайте '.idn_to_utf8(INJI_DOMAIN_NAME).'<br />для входа используйте ваш почтовый ящик в качестве логина и пароль: '.$pass;

             Tools::sendMail($from, $to, $subject, $text);

             Msg::add('Вы были зарегистрированы. На указанный почтовый ящик был выслан ваш пароль', 'success');

         }

system/modules/Ecommerce/models/UserAdds/Field.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
             'map' => [
                 ['name', 'type'],
                 ['required', 'save'],
-                [ 'userfield']
+                ['userfield']
             ]
         ]
     ];

system/modules/Ecommerce/models/Delivery/Field.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
             'map' => [
                 ['name', 'type'],
                 ['required', 'save'],
-                [ 'userfield']
+                ['userfield']
             ]
         ]
     ];

system/modules/Ecommerce/widgets/cart/fields.php

Indentation +16 added lines, -16 removed lines

@@ -1,16 +1,16 @@ 
 <fieldset>
   <?php
-  if (\Users\User::$cur->id) {
-      $userAdds = Ecommerce\UserAdds::getList(['where' => ['user_id', \Users\User::$cur->id]]);
-      $values = [];
-      foreach ($userAdds as $userAdd) {
-          $values[$userAdd->id] = $userAdd->values(['array' => true]);
-      }
-      if ($userAdds) {
-          $form->input('select', 'userAddsId', 'Ваши контакты', ['values' => ['' => 'Выберите'] + Ecommerce\UserAdds::getList(['where' => ['user_id', \Users\User::$cur->id], 'forSelect' => true])]);
-          echo '<hr />';
-      }
-      ?>
+    if (\Users\User::$cur->id) {
+        $userAdds = Ecommerce\UserAdds::getList(['where' => ['user_id', \Users\User::$cur->id]]);
+        $values = [];
+        foreach ($userAdds as $userAdd) {
+            $values[$userAdd->id] = $userAdd->values(['array' => true]);
+        }
+        if ($userAdds) {
+            $form->input('select', 'userAddsId', 'Ваши контакты', ['values' => ['' => 'Выберите'] + Ecommerce\UserAdds::getList(['where' => ['user_id', \Users\User::$cur->id], 'forSelect' => true])]);
+            echo '<hr />';
+        }
+        ?>
       <script>
           var userAddsValues = <?= json_encode($values); ?>;
           inji.onLoad(function () {
@@ -24,9 +24,9 @@ 
           })
       </script>
       <?php
-  }
-  foreach (Ecommerce\UserAdds\Field::getList(['order' => ['weight', 'asc']]) as $field) {
-      $form->input($field->type, "userAdds[fields][{$field->id}]", $field->name, ['required' => $field->required]);
-  }
-  ?>
+    }
+    foreach (Ecommerce\UserAdds\Field::getList(['order' => ['weight', 'asc']]) as $field) {
+        $form->input($field->type, "userAdds[fields][{$field->id}]", $field->name, ['required' => $field->required]);
+    }
+    ?>
 </fieldset>
\ No newline at end of file

system/modules/YandexExport/appControllers/YandexExportController.php

Doc Comments +5 added lines

@@ -11,6 +11,11 @@
 {
     function indexAction()
     {
+
+        /**
+         * @param DOMDocument $xml
+         * @param string $nodeName
+         */
         function addToXml($xml, $parent, $nodeName, $text)
         {
             $node = $parent->appendChild($xml->createElement($nodeName));

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
 
         addToXml($xml, $shop, 'name', \App::$cur->config['site']['name']);
         addToXml($xml, $shop, 'company', \App::$cur->config['site']['company_name']);
-        addToXml($xml, $shop, 'url', 'http://' . INJI_DOMAIN_NAME);
+        addToXml($xml, $shop, 'url', 'http://'.INJI_DOMAIN_NAME);
 
         $currencies = $xml->createElement('currencies');
         $currency = $currencies->appendChild($xml->createElement('currency'));
@@ -60,7 +60,7 @@ 
         $offers = $xml->createElement('offers');
         foreach (App::$cur->ecommerce->getItems() as $item) {
             $offer = $offers->appendChild($xml->createElement('offer'));
-            addToXml($xml, $offer, 'url', 'http://' . INJI_DOMAIN_NAME . '/ecommerce/view/' . $item->id);
+            addToXml($xml, $offer, 'url', 'http://'.INJI_DOMAIN_NAME.'/ecommerce/view/'.$item->id);
             addToXml($xml, $offer, 'price', $item->getPrice()->price);
             addToXml($xml, $offer, 'currencyId', 'RUR');
             addToXml($xml, $offer, 'categoryId', $item->category_id);
@@ -86,7 +86,7 @@ 
 
         header("Content-Type: text/xml");
         header("Expires: Thu, 19 Feb 1998 13:24:18 GMT");
-        header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");
+        header("Last-Modified: ".gmdate("D, d M Y H:i:s")." GMT");
         header("Cache-Control: no-cache, must-revalidate");
         header("Cache-Control: post-check=0,pre-check=0");
         header("Cache-Control: max-age=0");