Merge pull request #1181 from plentymarkets/fix/undefined_in_localize…

…dorder fix potential undefined#
plentymarkets · Jul 19, 2023 · 4ff44e4 · 4ff44e4
2 parents 2bbf816 + 34721a8
commit 4ff44e4
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/src/Models/LocalizedOrder.php b/src/Models/LocalizedOrder.php
@@ -209,19 +209,19 @@ public static function wrap($order, ...$data)
 
         $wrappedOrderitemTypes = $order->typeId === OrderType::RETURNS ? self::WRAPPED_ORDERITEM_TYPES_FOR_RETURN : self::WRAPPED_ORDERITEM_TYPES;
         foreach ($order->orderItems as $key => $orderItem) {
+            // unset purchase price
+            foreach($order->orderItems[$key]->amounts as &$amount) {
+                unset($amount['purchasePrice']);
+            }
+            unset($amount);
+
             if (in_array((int)$orderItem->typeId, $wrappedOrderitemTypes)) {
                 if ($orderItem->itemVariationId !== 0) {
                     $orderVariationIds[] = $orderItem->itemVariationId;
                 }
             } else {
                 unset($order->orderItems[$key]);
             }
-
-            // unset purchase price
-            foreach($order->orderItems[$key]->amounts as &$amount) {
-                unset($amount['purchasePrice']);
-            }
-            unset($amount);
         }
 
         $resultFields = ResultFieldTemplate::load(ResultFieldTemplate::TEMPLATE_LIST_ITEM);