Passed
Push — master ( 7fbd35...c2390d )
by Luiz Kim
02:31
created
src/Service/OrderProductService.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
         $this->request = $this->requestStack->getCurrentRequest();
33 33
     }
34 34
 
35
-    public function addProduct(Order $order, Product $product, $quantity, $price, ?ProductGroup  $productGroup = null, ?Product $parentProduct = null, ?OrderProduct $orderProductParent =  null): OrderProduct
35
+    public function addProduct(Order $order, Product $product, $quantity, $price, ?ProductGroup  $productGroup = null, ?Product $parentProduct = null, ?OrderProduct $orderProductParent = null): OrderProduct
36 36
     {
37 37
         $OProduct = new OrderProduct();
38 38
         $OProduct->setOrder($order);
@@ -74,10 +74,10 @@  discard block
 block discarded – undo
74 74
         $this->orderProductQueueService->addProductToQueue($OProduct);
75 75
     }
76 76
 
77
-    private function checkInventory(OrderProduct &$orderProduct)
77
+    private function checkInventory(OrderProduct & $orderProduct)
78 78
     {
79 79
         $order = $orderProduct->getOrder();
80
-        $product =  $orderProduct->getProduct();
80
+        $product = $orderProduct->getProduct();
81 81
 
82 82
         if ($order->getOrderType() == 'sale' && !$orderProduct->getOutInventory())
83 83
             $orderProduct->setOutInventory($product->getDefaultOutInventory());
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
         $this->checkInventory($orderProduct);
98 98
         foreach ($subProducts as $subproduct) {
99 99
             $product = $this->manager->getRepository(Product::class)->find($subproduct['product']);
100
-            $productGroup =  $this->manager->getRepository(ProductGroup::class)->find($subproduct['productGroup']);
100
+            $productGroup = $this->manager->getRepository(ProductGroup::class)->find($subproduct['productGroup']);
101 101
             $this->addSubproduct($orderProduct, $product, $productGroup, $subproduct['quantity']);
102 102
         }
103 103
         $this->orderProductQueueService->addProductToQueue($orderProduct);
Please login to merge, or discard this patch.
Braces   +15 added lines, -8 removed lines patch added patch discarded remove patch
@@ -79,17 +79,21 @@  discard block
 block discarded – undo
79 79
         $order = $orderProduct->getOrder();
80 80
         $product =  $orderProduct->getProduct();
81 81
 
82
-        if ($order->getOrderType() == 'sale' && !$orderProduct->getOutInventory())
83
-            $orderProduct->setOutInventory($product->getDefaultOutInventory());
82
+        if ($order->getOrderType() == 'sale' && !$orderProduct->getOutInventory()) {
83
+                    $orderProduct->setOutInventory($product->getDefaultOutInventory());
84
+        }
84 85
 
85
-        if ($order->getOrderType() == 'purchase' && !$orderProduct->getInInventory())
86
-            $orderProduct->setInInventory($product->getDefaultInInventory());
86
+        if ($order->getOrderType() == 'purchase' && !$orderProduct->getInInventory()) {
87
+                    $orderProduct->setInInventory($product->getDefaultInInventory());
88
+        }
87 89
     }
88 90
 
89 91
     public function postPersist(OrderProduct $orderProduct)
90 92
     {
91 93
 
92
-        if (!self::$mainProduct || !$this->request) return;
94
+        if (!self::$mainProduct || !$this->request) {
95
+          return;
96
+        }
93 97
         self::$mainProduct = false;
94 98
 
95 99
         $json = json_decode($this->request->getContent(), true);
@@ -107,7 +111,9 @@  discard block
 block discarded – undo
107 111
     public function preRemove(OrderProduct $orderProduct)
108 112
     {
109 113
 
110
-        if (!self::$mainProduct) return;
114
+        if (!self::$mainProduct) {
115
+          return;
116
+        }
111 117
         self::$mainProduct = false;
112 118
         $order = $orderProduct->getOrder();
113 119
         $this->manager->persist($order->setPrice(0));
@@ -116,8 +122,9 @@  discard block
 block discarded – undo
116 122
             'parentProduct'  => $orderProduct->getProduct(),
117 123
         ]);
118 124
 
119
-        foreach ($parentProducts as $parentProduct)
120
-            $this->manager->remove($parentProduct);
125
+        foreach ($parentProducts as $parentProduct) {
126
+                    $this->manager->remove($parentProduct);
127
+        }
121 128
         $this->manager->flush();
122 129
 
123 130
         self::$calculateBefore[] = $order;
Please login to merge, or discard this patch.