Test Failed
Push — master ( 723691...51d55d )
by Alexey
05:35
created
system/modules/Modules/appAdminControllers/content/install.php 3 patches
Indentation   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -1,21 +1,21 @@
 block discarded – undo
1 1
 <h1>Установка модулей</h1>
2 2
 <form>
3 3
   <?php
4
-  $config = Config::app(App::$primary ? App::$primary : App::$cur);
5
-  $modules = array_flip(Module::getInstalled(App::$cur));
6
-  if (file_exists(App::$primary->path . '/modules')) {
7
-      $appModules = array_slice(scandir(App::$primary->path . '/modules'), 2);
8
-  }
9
-  else {
10
-      $appModules=[];
11
-  }
12
-  $systemModules = array_merge(array_slice(scandir(INJI_SYSTEM_DIR . '/modules'), 2), $appModules);
13
-  foreach ($systemModules as $module) {
14
-      $info = Module::getInfo($module);
15
-      if (!$info || isset($modules[$module])) {
16
-          continue;
17
-      }
18
-      ?>
4
+    $config = Config::app(App::$primary ? App::$primary : App::$cur);
5
+    $modules = array_flip(Module::getInstalled(App::$cur));
6
+    if (file_exists(App::$primary->path . '/modules')) {
7
+        $appModules = array_slice(scandir(App::$primary->path . '/modules'), 2);
8
+    }
9
+    else {
10
+        $appModules=[];
11
+    }
12
+    $systemModules = array_merge(array_slice(scandir(INJI_SYSTEM_DIR . '/modules'), 2), $appModules);
13
+    foreach ($systemModules as $module) {
14
+        $info = Module::getInfo($module);
15
+        if (!$info || isset($modules[$module])) {
16
+            continue;
17
+        }
18
+        ?>
19 19
         <div class ="form-group">
20 20
             <div class="checkbox">
21 21
                 <label>
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -7,7 +7,7 @@
 block discarded – undo
7 7
       $appModules = array_slice(scandir(App::$primary->path . '/modules'), 2);
8 8
   }
9 9
   else {
10
-      $appModules=[];
10
+      $appModules = [];
11 11
   }
12 12
   $systemModules = array_merge(array_slice(scandir(INJI_SYSTEM_DIR . '/modules'), 2), $appModules);
13 13
   foreach ($systemModules as $module) {
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -5,8 +5,7 @@
 block discarded – undo
5 5
   $modules = array_flip(Module::getInstalled(App::$cur));
6 6
   if (file_exists(App::$primary->path . '/modules')) {
7 7
       $appModules = array_slice(scandir(App::$primary->path . '/modules'), 2);
8
-  }
9
-  else {
8
+  } else {
10 9
       $appModules=[];
11 10
   }
12 11
   $systemModules = array_merge(array_slice(scandir(INJI_SYSTEM_DIR . '/modules'), 2), $appModules);
Please login to merge, or discard this patch.
system/modules/UserForms/widgets/userForm.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@  discard block
 block discarded – undo
4 4
     echo 'form not found';
5 5
     return;
6 6
 }
7
-$userForm = \UserForms\Form::get((int)$formId);
7
+$userForm = \UserForms\Form::get((int) $formId);
8 8
 if (!$userForm) {
9 9
     echo 'form not found';
10 10
     return;
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
18 18
     echo "<p class = 'text-center'>{$userForm->description}</p>";
19 19
 }
20 20
 foreach ($userForm->inputs(['order' => ['weight']]) as $input) {
21
-    $form->input($input->type, 'UserForms[' . (int)$formId . '][input' . $input->id . ']', $input->label, ['required' => $input->required]);
21
+    $form->input($input->type, 'UserForms[' . (int) $formId . '][input' . $input->id . ']', $input->label, ['required' => $input->required]);
22 22
 }
23 23
 ?>
24 24
 <button class='btn btn-success btn-block'><?= urldecode($btnText); ?></button>
Please login to merge, or discard this patch.
system/modules/Exchange1c/objects/Parser/Item/Offer/Warehouse.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@
 block discarded – undo
25 25
 
26 26
     function parseWarehouse($warehouseCount) {
27 27
         $count = $warehouseCount['@attributes']['КоличествоНаСкладе'];
28
-        $objectId = \App::$cur->migrations->findObject((string)$warehouseCount['@attributes']['ИдСклада'], 'Ecommerce\Warehouse');
28
+        $objectId = \App::$cur->migrations->findObject((string) $warehouseCount['@attributes']['ИдСклада'], 'Ecommerce\Warehouse');
29 29
         if ($objectId) {
30 30
             $modelName = get_class($this->model);
31 31
             $warehouse = \Ecommerce\Item\Offer\Warehouse::get([[$modelName::index(), $this->model->pk()], [\Ecommerce\Warehouse::index(), $objectId->object_id]]);
Please login to merge, or discard this patch.
system/modules/Migrations/objects/Walker.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@
 block discarded – undo
27 27
         //walk know pathes
28 28
         foreach ($this->map->paths(['where' => ['path', $this->curPath]]) as $path) {
29 29
             if (defined('mdebug')) {
30
-                echo $path->item ;
30
+                echo $path->item;
31 31
             }
32 32
             if (isset($this->data[$path->item])) {
33 33
                 if ($path->type == 'container') {
Please login to merge, or discard this patch.
system/modules/Migrations/objects/Parser/Object.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 
48 48
             $walked = [];
49 49
             foreach ($this->object->params as $param) {
50
-                if(defined('mdebug')){
50
+                if (defined('mdebug')) {
51 51
                     echo " -> param ($param->id,$param->type,$param->value) ";
52 52
                 }
53 53
                 if ($model && $param->type && $param->type != 'item_key') {
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
                         $parser->parentObject = $this;
60 60
                         $parser->parentModel = $model;
61 61
                         $parser->walker = $this->walker;
62
-                        if(defined('mdebug')){
62
+                        if (defined('mdebug')) {
63 63
                             echo " -> objectParse ";
64 64
                         }
65 65
                         $parser->parse();
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
                         $parser->param = $param;
75 75
                         $parser->model = $model;
76 76
                         $parser->object = $this;
77
-                        if(defined('mdebug')){
77
+                        if (defined('mdebug')) {
78 78
                             echo " -> parser ($parserName) ";
79 79
                         }
80 80
                         $parser->parse();
Please login to merge, or discard this patch.
Braces   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 
48 48
             $walked = [];
49 49
             foreach ($this->object->params as $param) {
50
-                if(defined('mdebug')){
50
+                if(defined('mdebug')) {
51 51
                     echo " -> param ($param->id,$param->type,$param->value) ";
52 52
                 }
53 53
                 if ($model && $param->type && $param->type != 'item_key') {
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
                         $parser->parentObject = $this;
60 60
                         $parser->parentModel = $model;
61 61
                         $parser->walker = $this->walker;
62
-                        if(defined('mdebug')){
62
+                        if(defined('mdebug')) {
63 63
                             echo " -> objectParse ";
64 64
                         }
65 65
                         $parser->parse();
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
                         $parser->param = $param;
75 75
                         $parser->model = $model;
76 76
                         $parser->object = $this;
77
-                        if(defined('mdebug')){
77
+                        if(defined('mdebug')) {
78 78
                             echo " -> parser ($parserName) ";
79 79
                         }
80 80
                         $parser->parse();
Please login to merge, or discard this patch.
system/modules/Ecommerce/Ecommerce.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -233,7 +233,7 @@
 block discarded – undo
233 233
      * Getting items with params
234 234
      *
235 235
      * @param array $params
236
-     * @return array
236
+     * @return Model[]
237 237
      */
238 238
     public function getItems($params = []) {
239 239
         $selectOptions = Ecommerce\OptionsParser::parse($params);
Please login to merge, or discard this patch.
system/modules/Money/appControllers/MerchantsController.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -71,8 +71,8 @@
 block discarded – undo
71 71
                 $where[] = ['data', $_GET['data']];
72 72
             }
73 73
             $pays = Money\Pay::getList(['where' => $where, 'order' => ['date_create', 'DESC']]);
74
-            if(count($pays)===1){
75
-                Tools::redirect('/money/merchants/pay/'.current($pays)->id);
74
+            if (count($pays) === 1) {
75
+                Tools::redirect('/money/merchants/pay/' . current($pays)->id);
76 76
             }
77 77
             $this->view->page(['content' => 'pays', 'data' => compact('bread', 'pays')]);
78 78
         } else {
Please login to merge, or discard this patch.
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -71,7 +71,7 @@
 block discarded – undo
71 71
                 $where[] = ['data', $_GET['data']];
72 72
             }
73 73
             $pays = Money\Pay::getList(['where' => $where, 'order' => ['date_create', 'DESC']]);
74
-            if(count($pays)===1){
74
+            if(count($pays)===1) {
75 75
                 Tools::redirect('/money/merchants/pay/'.current($pays)->id);
76 76
             }
77 77
             $this->view->page(['content' => 'pays', 'data' => compact('bread', 'pays')]);
Please login to merge, or discard this patch.
system/modules/Money/objects/MerchantHelper/Paykeeper.php 1 patch
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@
 block discarded – undo
33 33
         ));
34 34
         $context = stream_context_create($options);
35 35
         echo "<style>.tmg {margin:0 auto}</style>";
36
-        echo file_get_contents("http://{$config['domain']}.paykeeper.ru/order/inline/", FALSE, $context);
36
+        echo file_get_contents("http://{$config['domain']}.paykeeper.ru/order/inline/", false, $context);
37 37
 
38 38
     }
39 39
 
Please login to merge, or discard this patch.
system/modules/Users/Users.php 2 patches
Braces   +13 added lines, -10 removed lines patch added patch discarded remove patch
@@ -236,17 +236,20 @@
 block discarded – undo
236 236
      * @return boolean|\Users\User
237 237
      */
238 238
     public function get($idn, $ltype = 'id') {
239
-        if (!$idn)
240
-            return false;
239
+        if (!$idn) {
240
+                    return false;
241
+        }
241 242
 
242
-        if (is_numeric($idn) && $ltype != 'login')
243
-            $user = Users\User::get($idn, 'id');
244
-        elseif ($ltype == 'login')
245
-            $user = Users\User::get($idn, 'login');
246
-        else
247
-            $user = Users\User::get($idn, 'mail');
248
-        if (!$user)
249
-            return [];
243
+        if (is_numeric($idn) && $ltype != 'login') {
244
+                    $user = Users\User::get($idn, 'id');
245
+        } elseif ($ltype == 'login') {
246
+                    $user = Users\User::get($idn, 'login');
247
+        } else {
248
+                    $user = Users\User::get($idn, 'mail');
249
+        }
250
+        if (!$user) {
251
+                    return [];
252
+        }
250 253
 
251 254
         return $user;
252 255
     }
Please login to merge, or discard this patch.
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -257,6 +257,9 @@
 block discarded – undo
257 257
         return $user;
258 258
     }
259 259
 
260
+    /**
261
+     * @param boolean $msg
262
+     */
260 263
     private function msgOrErr($err, $msg) {
261 264
         if ($msg) {
262 265
             Msg::add($err, 'danger');
Please login to merge, or discard this patch.