Test Failed
Push — master ( 045277...23cc6e )
by Alexey
05:33
created
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   +4 added lines patch added patch discarded remove patch
@@ -258,6 +258,10 @@
 block discarded – undo
258 258
         return $user;
259 259
     }
260 260
 
261
+    /**
262
+     * @param string $err
263
+     * @param boolean $msg
264
+     */
261 265
     private function msgOrErr($err, $msg) {
262 266
         if ($msg) {
263 267
             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/Ui/widgets/Form/autocomplete.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
 
38 38
 <div class="form-search-results"></div>
39 39
 <script>
40
-  window.autocomplete<?=$id;?> = function (element, snippet, snippetParams) {
40
+  window.autocomplete<?=$id; ?> = function (element, snippet, snippetParams) {
41 41
     element.element.onkeyup = function () {
42 42
       var inputContainer = element.element.parentNode;
43 43
       var selectedDiv = inputContainer.querySelector('.form-search-cur');
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
   };
91 91
   inji.onLoad(function () {
92 92
     setTimeout(function () {
93
-      new window.autocomplete<?=$id;?>(inji.get('#<?=$id;?>'), '<?=$options['snippet'];?>', <?=json_encode($options['snippetParams']);?>);
93
+      new window.autocomplete<?=$id; ?>(inji.get('#<?=$id; ?>'), '<?=$options['snippet']; ?>', <?=json_encode($options['snippetParams']); ?>);
94 94
     },100);
95 95
   });
96 96
 </script>
Please login to merge, or discard this patch.
system/modules/Ecommerce/models/Cart.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -336,6 +336,9 @@
 block discarded – undo
336 336
         return false;
337 337
     }
338 338
 
339
+    /**
340
+     * @return \Money\Sums
341
+     */
339 342
     public function deliverySum() {
340 343
         $sum = new \Money\Sums([]);
341 344
         if ($this->delivery && $this->needDelivery()) {
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 1 patch
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.
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/Ui/widgets/Form/select.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -9,7 +9,7 @@
 block discarded – undo
9 9
     $primaryValue = isset($options['value']) ? $options['value'] : null;
10 10
     $primaryValue = is_array($primaryValue) && isset($primaryValue['primary']) ? $primaryValue['primary'] : $primaryValue;
11 11
     if (is_numeric($key) && !is_array($primaryValue) && $primaryValue !== '') {
12
-        $primaryValue = (int)$primaryValue;
12
+        $primaryValue = (int) $primaryValue;
13 13
     }
14 14
     if (
15 15
         (!is_array($primaryValue) && ($key === $primaryValue || (isset($form->userDataTree[$name]) && $form->userDataTree[$name] === $key))) ||
Please login to merge, or discard this patch.