Skip to content

Commit 6a6e732

Browse files
author
Stanislav Idolov
committed
MAGETWO-32501: Implement Cart Service interfaces
1 parent 22db2a3 commit 6a6e732

File tree

2 files changed

+8
-20
lines changed

2 files changed

+8
-20
lines changed

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

Lines changed: 8 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -319,14 +319,9 @@ class Quote extends AbstractExtensibleModel implements \Magento\Quote\Api\Data\C
319319
protected $customerRepository;
320320

321321
/**
322-
* @var \Magento\Quote\Api\Data\TotalsDataBuilder
322+
* @var Cart\CurrencyFactory
323323
*/
324-
protected $totalsBuilder;
325-
326-
/**
327-
* @var \Magento\Quote\Api\Data\CurrencyDataBuilder
328-
*/
329-
protected $currencyBuilder;
324+
protected $currencyFactory;
330325

331326
/**
332327
* @param \Magento\Framework\Model\Context $context
@@ -359,8 +354,7 @@ class Quote extends AbstractExtensibleModel implements \Magento\Quote\Api\Data\C
359354
* @param \Magento\Customer\Api\Data\CustomerDataBuilder $customerBuilder
360355
* @param \Magento\Customer\Api\CustomerRepositoryInterface $customerRepository
361356
* @param \Magento\Framework\Api\ExtensibleDataObjectConverter $extensibleDataObjectConverter
362-
* @param \Magento\Quote\Api\Data\TotalsDataBuilder $totalsBuilder
363-
* @param \Magento\Quote\Api\Data\CurrencyDataBuilder $currencyBuilder
357+
* @param Cart\CurrencyFactory $currencyFactory
364358
* @param \Magento\Framework\Model\Resource\AbstractResource $resource
365359
* @param \Magento\Framework\Data\Collection\Db $resourceCollection
366360
* @param array $data
@@ -396,8 +390,7 @@ public function __construct(
396390
\Magento\Customer\Api\Data\CustomerDataBuilder $customerBuilder,
397391
\Magento\Customer\Api\CustomerRepositoryInterface $customerRepository,
398392
\Magento\Framework\Api\ExtensibleDataObjectConverter $extensibleDataObjectConverter,
399-
\Magento\Quote\Api\Data\TotalsDataBuilder $totalsBuilder,
400-
\Magento\Quote\Api\Data\CurrencyDataBuilder $currencyBuilder,
393+
\Magento\Quote\Model\Cart\CurrencyFactory $currencyFactory,
401394
\Magento\Framework\Model\Resource\AbstractResource $resource = null,
402395
\Magento\Framework\Data\Collection\Db $resourceCollection = null,
403396
array $data = []
@@ -428,8 +421,7 @@ public function __construct(
428421
$this->customerBuilder = $customerBuilder;
429422
$this->customerRepository = $customerRepository;
430423
$this->extensibleDataObjectConverter = $extensibleDataObjectConverter;
431-
$this->totalsBuilder = $totalsBuilder;
432-
$this->currencyBuilder = $currencyBuilder;
424+
$this->currencyFactory = $currencyFactory;
433425
parent::__construct(
434426
$context,
435427
$registry,
@@ -452,13 +444,15 @@ protected function _construct()
452444
}
453445

454446
/**
447+
* @codeCoverageIgnoreStart
448+
*
455449
* {@inheritdoc}
456450
*/
457451
public function getCurrency()
458452
{
459453
$currency = $this->getData('currency');
460454
if (!$currency) {
461-
$this->currencyBuilder
455+
$currency = $this->currencyFactory->create()
462456
->setGlobalCurrencyCode($this->getGlobalCurrencyCode())
463457
->setBaseCurrencyCode($this->getBaseCurrencyCode())
464458
->setStoreCurrencyCode($this->getStoreCurrencyCode())
@@ -467,14 +461,11 @@ public function getCurrency()
467461
->setStoreToQuoteRate($this->getStoreToQuoteRate())
468462
->setBaseToGlobalRate($this->getBaseToGlobalRate())
469463
->setBaseToQuoteRate($this->getBaseToQuoteRate());
470-
$currency = $this->currencyBuilder->create();
471464
}
472465
return $currency;
473466
}
474467

475468
/**
476-
* @codeCoverageIgnoreStart
477-
*
478469
* {@inheritdoc}
479470
*/
480471
public function getItems()

dev/tests/api-functional/testsuite/Magento/Quote/Api/CartManagementInterfaceTest.php

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -336,9 +336,6 @@ public function testGetCartForCustomer()
336336

337337
$this->assertContains('customer', $cartData);
338338
$this->assertEquals(false, $cartData['customer_is_guest']);
339-
$this->assertContains('totals_object', $cartData);
340-
$this->assertEquals($cart->getSubtotal(), $cartData['totals_object']['subtotal']);
341-
$this->assertEquals($cart->getGrandTotal(), $cartData['totals_object']['grand_total']);
342339
$this->assertContains('currency', $cartData);
343340
$this->assertEquals($cart->getGlobalCurrencyCode(), $cartData['currency']['global_currency_code']);
344341
$this->assertEquals($cart->getBaseCurrencyCode(), $cartData['currency']['base_currency_code']);

0 commit comments

Comments
 (0)