Test Failed
Push — master ( 04fcd4...836f53 )
by Evgenii
05:13
created
src/OrderStatusDispatcher.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -57,10 +57,10 @@
 block discarded – undo
57 57
 
58 58
             foreach ($this->xml->order->items->item as $item) {
59 59
                 $orderItem = new Item();
60
-                $orderItem->setBarcode((string)$item['barcode'])
61
-                    ->setQuantity((int)$item['quantity'])
60
+                $orderItem->setBarcode((string) $item['barcode'])
61
+                    ->setQuantity((int) $item['quantity'])
62 62
                     ->setReturn(boolval($item['returns']))
63
-                    ->setRetprice((float)$item['retprice']);
63
+                    ->setRetprice((float) $item['retprice']);
64 64
                 $this->items[] = $orderItem;
65 65
             }
66 66
         }
Please login to merge, or discard this patch.
Braces   +12 added lines, -8 removed lines patch added patch discarded remove patch
@@ -43,8 +43,9 @@  discard block
 block discarded – undo
43 43
     public function __construct(string $xmlBody)
44 44
     {
45 45
         $this->xml = simplexml_load_string($xmlBody);
46
-        if ($this->xml === false)
47
-            throw new Exception('XML body is not valid.');
46
+        if ($this->xml === false) {
47
+                    throw new Exception('XML body is not valid.');
48
+        }
48 49
         if ($this->xml->order) {
49 50
 
50 51
             foreach ($this->xml->order->statushistory->status as $statusItem) {
@@ -72,8 +73,9 @@  discard block
 block discarded – undo
72 73
      */
73 74
     public function isItemReturned($barcode): ?bool
74 75
     {
75
-        if (isset($this->items[$barcode]))
76
-            return $this->items[$barcode]->isReturned();
76
+        if (isset($this->items[$barcode])) {
77
+                    return $this->items[$barcode]->isReturned();
78
+        }
77 79
         return null;
78 80
     }
79 81
 
@@ -87,8 +89,9 @@  discard block
 block discarded – undo
87 89
 
88 90
     public function getStatusName(): ?string
89 91
     {
90
-        if ($this->getStatusId())
91
-            DalliOrderStatus::getLabel($this->getStatusId());
92
+        if ($this->getStatusId()) {
93
+                    DalliOrderStatus::getLabel($this->getStatusId());
94
+        }
92 95
         return null;
93 96
     }
94 97
 
@@ -112,8 +115,9 @@  discard block
 block discarded – undo
112 115
     {
113 116
         $returned = [];
114 117
         foreach ($this->items as $item) {
115
-            if ($item->isReturned())
116
-                $returned[] = $item;
118
+            if ($item->isReturned()) {
119
+                            $returned[] = $item;
120
+            }
117 121
         }
118 122
         return $returned;
119 123
     }
Please login to merge, or discard this patch.