Test Failed
Push — master ( f1b65c...c8a2bf )
by Alexey
07:14
created
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.
system/modules/Chats/models/Chat.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -16,11 +16,11 @@
 block discarded – undo
16 16
     public static $cols = [
17 17
         'name' => ['type' => 'text'],
18 18
         'code' => ['type' => 'text'],
19
-        'message' => ['type' => 'dataManager','relation'=>'messages'],
19
+        'message' => ['type' => 'dataManager', 'relation'=>'messages'],
20 20
     ];
21 21
     public static $dataManagers = [
22 22
         'manager' => [
23
-            'cols' => ['name', 'code','message']
23
+            'cols' => ['name', 'code', 'message']
24 24
         ]
25 25
     ];
26 26
     public static $forms = [
Please login to merge, or discard this patch.
system/modules/Ecommerce/objects/Vendor/CalculatePriceDeliveryCdek.php 1 patch
Upper-Lower-Casing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -185,7 +185,7 @@  discard block
 block discarded – undo
185 185
      */
186 186
     public function getGoodslist() {
187 187
         if (!isset($this->goodsList)) {
188
-            return NULL;
188
+            return null;
189 189
         }
190 190
         return $this->goodsList;
191 191
     }
@@ -213,7 +213,7 @@  discard block
 block discarded – undo
213 213
      */
214 214
     private function _getTariffList() {
215 215
         if (!isset($this->tariffList)) {
216
-            return NULL;
216
+            return null;
217 217
         }
218 218
         return $this->tariffList;
219 219
     }
Please login to merge, or discard this patch.
system/modules/Ecommerce/migrations/cdekCities.php 2 patches
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@
 block discarded – undo
23 23
         $handle = fopen(__DIR__ . '/../vendor/CdekCity_RUS_20170729.csv', 'r');
24 24
         $row = 1;
25 25
         $cols = [];
26
-        while (($data = fgetcsv($handle, 10000, ";")) !== FALSE) {
26
+        while (($data = fgetcsv($handle, 10000, ";")) !== false) {
27 27
             if ($row === 1) {
28 28
                 $cols = $data;
29 29
             } else {
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -2,7 +2,7 @@
 block discarded – undo
2 2
 
3 3
 return [
4 4
     'up' => function () {
5
-        \Ecommerce\Delivery\Field\Item::get(1,'data');
5
+        \Ecommerce\Delivery\Field\Item::get(1, 'data');
6 6
         $field = new \Ecommerce\Delivery\Field(
7 7
             [
8 8
                 'name' => 'Город',
Please login to merge, or discard this patch.
system/modules/Sliders/widgets/slider.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@
 block discarded – undo
8 8
 if (!empty($alias)) {
9 9
     $slider = Sliders\Slider::get($alias, 'alias');
10 10
 }
11
-if(empty($slider)){
11
+if (empty($slider)) {
12 12
     echo 'Slider not found';
13 13
     return;
14 14
 }
Please login to merge, or discard this patch.
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@
 block discarded – undo
8 8
 if (!empty($alias)) {
9 9
     $slider = Sliders\Slider::get($alias, 'alias');
10 10
 }
11
-if(empty($slider)){
11
+if(empty($slider)) {
12 12
     echo 'Slider not found';
13 13
     return;
14 14
 }
Please login to merge, or discard this patch.
system/modules/UserForms/UserForms.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -14,13 +14,13 @@
 block discarded – undo
14 14
         \App::$cur->view->customAsset('js', '/static/moduleAsset/UserForms/js/formCatcher.js');
15 15
         if (!empty($_POST['UserForms'])) {
16 16
             foreach ($_POST['UserForms'] as $form_id => $inputs) {
17
-                $form = \UserForms\Form::get((int)$form_id);
17
+                $form = \UserForms\Form::get((int) $form_id);
18 18
                 if (!$form) {
19 19
                     continue;
20 20
                 }
21 21
                 $formRecive = new \UserForms\Recive();
22
-                $formRecive->user_id = (int)\Users\User::$cur->id;
23
-                $formRecive->form_id = (int)$form_id;
22
+                $formRecive->user_id = (int) \Users\User::$cur->id;
23
+                $formRecive->form_id = (int) $form_id;
24 24
                 $data = [];
25 25
                 $error = false;
26 26
                 foreach ($form->inputs as $input) {
Please login to merge, or discard this patch.
system/modules/Ecommerce/objects/DeliveryProvider/PickPoint.php 3 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@
 block discarded – undo
16 16
     static $name = 'PickPoint - курьерская служба';
17 17
 
18 18
     /**
19
-     * @param \Ecommerce\Cart $cart
19
+     * @param string $URL
20 20
      * @return \Money\Sums
21 21
      */
22 22
     static function curl_get_file_contents($URL, $data) {
Please login to merge, or discard this patch.
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -35,8 +35,11 @@
 block discarded – undo
35 35
         $contents = curl_exec($c);
36 36
         curl_close($c);
37 37
 
38
-        if ($contents) return $contents;
39
-        else return FALSE;
38
+        if ($contents) {
39
+            return $contents;
40
+        } else {
41
+            return FALSE;
42
+        }
40 43
     }
41 44
 
42 45
     static function deliveryTime($cart) {
Please login to merge, or discard this patch.
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@
 block discarded – undo
36 36
         curl_close($c);
37 37
 
38 38
         if ($contents) return $contents;
39
-        else return FALSE;
39
+        else return false;
40 40
     }
41 41
 
42 42
     static function deliveryTime($cart) {
Please login to merge, or discard this patch.