@@ -61,7 +61,9 @@ discard block |
||
| 61 | 61 | $product = $orderProduct->getProduct(); |
| 62 | 62 | $productType = $product->getType(); |
| 63 | 63 | |
| 64 | - if (!in_array($productType, ['product', 'service']) || $orderProduct->getId()) return $orderProduct; |
|
| 64 | + if (!in_array($productType, ['product', 'service']) || $orderProduct->getId()) { |
|
| 65 | + return $orderProduct; |
|
| 66 | + } |
|
| 65 | 67 | |
| 66 | 68 | |
| 67 | 69 | $existOrderProduct = $this->manager->getRepository(OrderProduct::class)->findOneBy([ |
@@ -82,7 +84,9 @@ discard block |
||
| 82 | 84 | public function postPersist(OrderProduct $orderProduct) |
| 83 | 85 | { |
| 84 | 86 | |
| 85 | - if (!self::$mainProduct) return; |
|
| 87 | + if (!self::$mainProduct) { |
|
| 88 | + return; |
|
| 89 | + } |
|
| 86 | 90 | self::$mainProduct = false; |
| 87 | 91 | |
| 88 | 92 | $json = json_decode($this->request->getContent(), true); |
@@ -100,7 +104,9 @@ discard block |
||
| 100 | 104 | public function preRemove(OrderProduct $orderProduct) |
| 101 | 105 | { |
| 102 | 106 | |
| 103 | - if (!self::$mainProduct) return; |
|
| 107 | + if (!self::$mainProduct) { |
|
| 108 | + return; |
|
| 109 | + } |
|
| 104 | 110 | self::$mainProduct = false; |
| 105 | 111 | $order = $orderProduct->getOrder(); |
| 106 | 112 | $this->manager->persist($order->setPrice(0)); |
@@ -109,8 +115,9 @@ discard block |
||
| 109 | 115 | 'parentProduct' => $orderProduct->getProduct(), |
| 110 | 116 | ]); |
| 111 | 117 | |
| 112 | - foreach ($parentProducts as $parentProduct) |
|
| 113 | - $this->manager->remove($parentProduct); |
|
| 118 | + foreach ($parentProducts as $parentProduct) { |
|
| 119 | + $this->manager->remove($parentProduct); |
|
| 120 | + } |
|
| 114 | 121 | $this->manager->flush(); |
| 115 | 122 | |
| 116 | 123 | self::$calculateBefore[] = $order; |