Skip to content

Commit b1dd1c6

Browse files
committed
MAGETWO-52079: [GITHUB] Order Repository GetList Method always returns the same shipping address for all orders #4019
1 parent c1b7c35 commit b1dd1c6

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

app/code/Magento/Sales/Model/OrderRepository.php

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
use Magento\Sales\Model\Order\ShippingAssignmentBuilder;
1111
use Magento\Sales\Api\Data\OrderSearchResultInterfaceFactory as SearchResultFactory;
1212
use Magento\Sales\Api\Data\OrderExtensionInterface;
13-
use Magento\Sales\Api\Data\OrderExtension;
13+
use Magento\Sales\Api\Data\OrderExtensionFactory;
1414
use Magento\Sales\Api\Data\OrderInterface;
1515
use Magento\Sales\Api\Data\ShippingAssignmentInterface;
1616
use Magento\Framework\Exception\NoSuchEntityException;
@@ -34,9 +34,9 @@ class OrderRepository implements \Magento\Sales\Api\OrderRepositoryInterface
3434
protected $searchResultFactory = null;
3535

3636
/**
37-
* @var OrderExtension
37+
* @var OrderExtensionFactory
3838
*/
39-
private $orderExtension;
39+
private $orderExtensionFactory;
4040

4141
/**
4242
* @var ShippingAssignmentBuilder
@@ -173,7 +173,7 @@ private function setShippingAssignments(OrderInterface $order)
173173
$extensionAttributes = $order->getExtensionAttributes();
174174

175175
if ($extensionAttributes === null) {
176-
$extensionAttributes = $this->getOrderExtensionDependency()->create();
176+
$extensionAttributes = $this->getOrderExtensionDependencyFactory()->create();
177177
} elseif ($extensionAttributes->getShippingAssignments() !== null) {
178178
return;
179179
}
@@ -189,14 +189,14 @@ private function setShippingAssignments(OrderInterface $order)
189189
* @return OrderExtension
190190
* @deprecated
191191
*/
192-
private function getOrderExtensionDependency()
192+
private function getOrderExtensionDependencyFactory()
193193
{
194-
if (!$this->orderExtension instanceof OrderExtension) {
195-
$this->orderExtension = \Magento\Framework\App\ObjectManager::getInstance()->get(
194+
if (!$this->orderExtensionFactory instanceof OrderExtensionFactory) {
195+
$this->orderExtensionFactory = \Magento\Framework\App\ObjectManager::getInstance()->get(
196196
'\Magento\Sales\Api\Data\OrderExtensionFactory'
197197
);
198198
}
199-
return $this->orderExtension;
199+
return $this->orderExtensionFactory;
200200
}
201201

202202
/**

0 commit comments

Comments
 (0)