Test Failed
Push — master ( 245cca...9fb147 )
by Alexey
04:20
created
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.
system/modules/Exchange1c/objects/Mode/File.php 1 patch
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@
 block discarded – undo
45 45
             $pathinfo = pathinfo($filename);
46 46
             if ($pathinfo['extension'] === 'zip') {
47 47
                 $zip = new \ZipArchive;
48
-                if ($zip->open($dir . '/' . $filename) === TRUE) {
48
+                if ($zip->open($dir . '/' . $filename) === true) {
49 49
                     $zip->extractTo($dir);
50 50
                     $zip->close();
51 51
                 } else {
Please login to merge, or discard this patch.
system/modules/Exchange1c/objects/Mode/Init.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
             $query = \App::$cur->db->newQuery();
29 29
             $query->operation = 'select';
30 30
             $query->table = \Exchange1c\Exchange::table();
31
-            $query->cols = \Exchange1c\Exchange::index().','.\Exchange1c\Exchange::colPrefix() . 'path';
31
+            $query->cols = \Exchange1c\Exchange::index() . ',' . \Exchange1c\Exchange::colPrefix() . 'path';
32 32
             $queryArr = $query->buildQuery();
33 33
             $queryArr['query'] .= ' where `' . \Exchange1c\Exchange::colPrefix() . 'cleared` = 0 AND  `' . \Exchange1c\Exchange::colPrefix() . 'date_create` < NOW() - INTERVAL ' . \App::$cur->exchange1c->config['maxSaveFilesInterval'];
34 34
 
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
                 \Tools::delDir($exchangeArr[\Exchange1c\Exchange::colPrefix() . 'path']);
38 38
                 $query = \App::$cur->db->newQuery();
39 39
                 $query->where([\Exchange1c\Exchange::index(), $exchangeArr[\Exchange1c\Exchange::index()]]);
40
-                $query->update(\Exchange1c\Exchange::table(),[\Exchange1c\Exchange::colPrefix() . 'cleared' => 1]);
40
+                $query->update(\Exchange1c\Exchange::table(), [\Exchange1c\Exchange::colPrefix() . 'cleared' => 1]);
41 41
             }
42 42
         }
43 43
     }
Please login to merge, or discard this patch.
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -13,11 +13,9 @@
 block discarded – undo
13 13
 
14 14
 use Exchange1c\Exchange;
15 15
 
16
-class Init extends \Exchange1c\Mode
17
-{
16
+class Init extends \Exchange1c\Mode {
18 17
 
19
-    public function process()
20
-    {
18
+    public function process() {
21 19
         echo "zip=yes\n";
22 20
         echo 'file_limit=' . \Tools::toBytes(ini_get('upload_max_filesize'));
23 21
         $this->end();
Please login to merge, or discard this patch.
system/modules/Ecommerce/widgets/items/showOptions.php 1 patch
Indentation   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -38,25 +38,25 @@
 block discarded – undo
38 38
             <span class="caption">Вид:</span>
39 39
             <span class="group">
40 40
               <?php
41
-              for ($i = 2; $i < 5; $i++) {
42
-                  $curLimit = $limit * $i;
43
-                  $curQuery = http_build_query(array_merge($query, ['limit' => $curLimit]));
44
-                  echo " <a rel=\"nofollow\" href='{$path}?{$curQuery}'>";
45
-                  echo !empty($_GET['limit']) && $_GET['limit'] == $curLimit ? '<b>' : '';
46
-                  echo $curLimit;
47
-                  echo !empty($_GET['limit']) && $_GET['limit'] == $curLimit ? '</b>' : '';
48
-                  echo "</a> ";
49
-              }
50
-              if (!empty(App::$cur->ecommerce->config['list_all'])) {
51
-                  $curLimit = 'all';
52
-                  $curQuery = http_build_query(array_merge($query, ['limit' => $curLimit]));
53
-                  echo " <a rel=\"nofollow\" href='{$path}?{$curQuery}'>";
54
-                  echo !empty($_GET['limit']) && $_GET['limit'] == $curLimit ? '<b>' : '';
55
-                  echo 'Все';
56
-                  echo !empty($_GET['limit']) && $_GET['limit'] == $curLimit ? '</b>' : '';
57
-                  echo "</a> ";
58
-              }
59
-              ?>
41
+                for ($i = 2; $i < 5; $i++) {
42
+                    $curLimit = $limit * $i;
43
+                    $curQuery = http_build_query(array_merge($query, ['limit' => $curLimit]));
44
+                    echo " <a rel=\"nofollow\" href='{$path}?{$curQuery}'>";
45
+                    echo !empty($_GET['limit']) && $_GET['limit'] == $curLimit ? '<b>' : '';
46
+                    echo $curLimit;
47
+                    echo !empty($_GET['limit']) && $_GET['limit'] == $curLimit ? '</b>' : '';
48
+                    echo "</a> ";
49
+                }
50
+                if (!empty(App::$cur->ecommerce->config['list_all'])) {
51
+                    $curLimit = 'all';
52
+                    $curQuery = http_build_query(array_merge($query, ['limit' => $curLimit]));
53
+                    echo " <a rel=\"nofollow\" href='{$path}?{$curQuery}'>";
54
+                    echo !empty($_GET['limit']) && $_GET['limit'] == $curLimit ? '<b>' : '';
55
+                    echo 'Все';
56
+                    echo !empty($_GET['limit']) && $_GET['limit'] == $curLimit ? '</b>' : '';
57
+                    echo "</a> ";
58
+                }
59
+                ?>
60 60
             </span>
61 61
             <span class="group">
62 62
                 <a rel="nofollow" href='#' onclick="inji.onLoad(function () {
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
@@ -338,6 +338,9 @@
 block discarded – undo
338 338
         return false;
339 339
     }
340 340
 
341
+    /**
342
+     * @return \Money\Sums
343
+     */
341 344
     public function deliverySum() {
342 345
         $sum = new \Money\Sums([0 => 0]);
343 346
         if ($this->delivery && $this->needDelivery()) {
Please login to merge, or discard this patch.
system/modules/Ecommerce/migrations/cdekPVZ.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/CdekPVZ.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/Ui/appAdminControllers/content/dataManager/view.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
     $type = !empty($colInfo['colParams']['type']) ? $colInfo['colParams']['type'] : 'string';
20 20
     if ($type != 'dataManager') {
21 21
         $table->addRow([
22
-            str_replace(' ','&nbsp;',!empty($modelName::$labels[$colName]) ? $modelName::$labels[$colName] : $colName),
22
+            str_replace(' ', '&nbsp;', !empty($modelName::$labels[$colName]) ? $modelName::$labels[$colName] : $colName),
23 23
             \Model::resloveTypeValue($item, $colName, true)
24 24
         ]);
25 25
     }
Please login to merge, or discard this patch.