Skip to content

Commit d8e06d5

Browse files
committed
Merge branch 'MAGETWO-34832-cherry-pick' of github.corp.ebay.com:magento-mpi/magento2ce into pull-request
Conflicts: app/code/Magento/Quote/composer.json
2 parents 1b6c66e + f59297a commit d8e06d5

File tree

9 files changed

+14
-156
lines changed

9 files changed

+14
-156
lines changed

app/code/Magento/Quote/Model/Quote.php

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2243,12 +2243,6 @@ public function reserveOrderId()
22432243
{
22442244
if (!$this->getReservedOrderId()) {
22452245
$this->setReservedOrderId($this->_getResource()->getReservedOrderId($this));
2246-
} else {
2247-
//checking if reserved order id was already used for some order
2248-
//if yes reserving new one if not using old one
2249-
if ($this->_getResource()->isOrderIncrementIdUsed($this->getReservedOrderId())) {
2250-
$this->setReservedOrderId($this->_getResource()->getReservedOrderId($this));
2251-
}
22522246
}
22532247
return $this;
22542248
}

app/code/Magento/Quote/Model/Quote/Address/ToOrder.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,8 @@ public function convert(Address $object, $data = [])
7575
'\Magento\Sales\Api\Data\OrderInterface'
7676
);
7777
$order->setStoreId($object->getQuote()->getStoreId())
78-
->setQuoteId($object->getQuote()->getId());
78+
->setQuoteId($object->getQuote()->getId())
79+
->setIncrementId($object->getQuote()->getReservedOrderId());
7980

8081
$this->objectCopyService->copyFieldsetToTarget('sales_convert_quote', 'to_order', $object->getQuote(), $order);
8182
$this->eventManager->dispatch(

app/code/Magento/Quote/Model/Resource/Quote.php

Lines changed: 8 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -16,22 +16,22 @@
1616
class Quote extends AbstractDb
1717
{
1818
/**
19-
* @var \Magento\Eav\Model\Config
19+
* @var \Magento\SalesSequence\Model\Manager
2020
*/
21-
protected $_config;
21+
protected $sequenceManager;
2222

2323
/**
2424
* @param \Magento\Framework\Model\Resource\Db\Context $context
25-
* @param \Magento\Eav\Model\Config $config
26-
* @param string|null $resourcePrefix
25+
* @param \Magento\SalesSequence\Model\Manager $sequenceManager
26+
* @param null $resourcePrefix
2727
*/
2828
public function __construct(
2929
\Magento\Framework\Model\Resource\Db\Context $context,
30-
\Magento\Eav\Model\Config $config,
30+
\Magento\SalesSequence\Model\Manager $sequenceManager,
3131
$resourcePrefix = null
3232
) {
3333
parent::__construct($context, $resourcePrefix);
34-
$this->_config = $config;
34+
$this->sequenceManager = $sequenceManager;
3535
}
3636

3737
/**
@@ -156,28 +156,8 @@ public function loadByIdWithoutStore($quote, $quoteId)
156156
*/
157157
public function getReservedOrderId($quote)
158158
{
159-
$storeId = (int)$quote->getStoreId();
160-
return $this->_config->getEntityType(\Magento\Sales\Model\Order::ENTITY)->fetchNewIncrementId($storeId);
161-
}
162-
163-
/**
164-
* Check is order increment id use in sales/order table
165-
*
166-
* @param int $orderIncrementId
167-
* @return bool
168-
*/
169-
public function isOrderIncrementIdUsed($orderIncrementId)
170-
{
171-
$adapter = $this->_getReadAdapter();
172-
$bind = [':increment_id' => $orderIncrementId];
173-
$select = $adapter->select();
174-
$select->from($this->getTable('sales_order'), 'entity_id')->where('increment_id = :increment_id');
175-
$entity_id = $adapter->fetchOne($select, $bind);
176-
if ($entity_id > 0) {
177-
return true;
178-
}
179-
180-
return false;
159+
return $this->sequenceManager->getSequence(\Magento\Sales\Model\Order::ENTITY, (int)$quote->getStoreId())
160+
->getNextValue();
181161
}
182162

183163
/**

app/code/Magento/Quote/Test/Unit/Model/Quote/Address/ToOrderTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ public function testConvert()
8484

8585
$object = $this->getMock('Magento\Quote\Model\Quote\Address', [], [], '', false);
8686
$quote = $this->getMock('Magento\Quote\Model\Quote', [], [], '', false);
87-
$object->expects($this->exactly(4))->method('getQuote')->willReturn($quote);
87+
$object->expects($this->exactly(5))->method('getQuote')->willReturn($quote);
8888
$quote->expects($this->once())->method('getId')->willReturn($quoteId);
8989
$quote->expects($this->once())->method('getStoreId')->willReturn($storeId);
9090
$this->objectCopyMock->expects($this->once())->method('getDataFromFieldset')->with(

app/code/Magento/Quote/Test/Unit/Model/Resource/QuoteTest.php

Lines changed: 0 additions & 82 deletions
This file was deleted.

app/code/Magento/Quote/composer.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
"magento/module-payment": "0.74.0-beta4",
1212
"magento/module-sales": "0.74.0-beta4",
1313
"magento/module-shipping": "0.74.0-beta4",
14+
"magento/module-sales-sequence": "0.74.0-beta4",
1415
"magento/module-backend": "0.74.0-beta4",
1516
"magento/module-directory": "0.74.0-beta4",
1617
"magento/module-eav": "0.74.0-beta4",

dev/tests/integration/testsuite/Magento/Sales/Model/Resource/QuoteTest.php

Lines changed: 0 additions & 37 deletions
This file was deleted.

dev/tests/integration/testsuite/Magento/Sales/_files/order_info.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@
4848
'Magento\Store\Model\StoreManagerInterface'
4949
)->getStore()->getId()
5050
)->setReservedOrderId(
51-
'test01'
51+
'100000001'
5252
)->setBillingAddress(
5353
$billingAddress
5454
)->setShippingAddress(

dev/tests/static/testsuite/Magento/Test/Legacy/_files/obsolete_methods.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2155,4 +2155,5 @@
21552155
['_isCacheEnabled', 'Magento\Eav\Model\Config'],
21562156
['_createCustomerAttribute', '\Magento\Customer\Model\Customer'],
21572157
['prepareCatalogProductPriceIndexTable', 'Magento\CatalogRule\Model\Observer'],
2158+
['isOrderIncrementIdUsed', 'Magento\Quote\Model\Resource\Quote'],
21582159
];

0 commit comments

Comments
 (0)