File tree Expand file tree Collapse file tree 4 files changed +5
-3
lines changed
QuoteGraphQl/Model/Resolver
dev/tests/integration/testsuite/Magento/SalesRule
Model/Rule/Action/Discount Expand file tree Collapse file tree 4 files changed +5
-3
lines changed Original file line number Diff line number Diff line change @@ -42,7 +42,7 @@ private function getDiscountValues(Quote $quote)
4242 $ discountValues =[];
4343 $ address = $ quote ->getShippingAddress ();
4444 $ totalDiscounts = $ address ->getExtensionAttributes ()->getDiscounts ();
45- if ($ totalDiscounts ) {
45+ if ($ totalDiscounts && is_array ( $ totalDiscounts ) ) {
4646 foreach ($ totalDiscounts as $ value ) {
4747 $ discount = [];
4848 $ amount = [];
Original file line number Diff line number Diff line change 99
1010use Magento \Framework \Api \ExtensionAttributesInterface ;
1111use Magento \SalesRule \Api \Data \RuleDiscountInterface ;
12+ use Magento \Framework \Api \AbstractExtensibleObject ;
1213
1314/**
1415 * Data Model for Rule Discount
1516 */
16- class RuleDiscount extends \ Magento \ Framework \ Api \ AbstractExtensibleObject implements RuleDiscountInterface
17+ class RuleDiscount extends AbstractExtensibleObject implements RuleDiscountInterface
1718{
1819 const KEY_DISCOUNT_DATA = 'discount ' ;
1920 const KEY_RULE_LABEL = 'rule ' ;
Original file line number Diff line number Diff line change @@ -164,6 +164,7 @@ public function testDiscountsOnQuoteWithFixedDiscount(): void
164164 /** @var CartItemInterface $item */
165165 $ item = $ quote ->getItems ()[0 ];
166166 $ quoteItemDiscounts = $ item ->getExtensionAttributes ()->getDiscounts ();
167+ $ this ->assertArrayHasKey ('0 ' , $ quoteItemDiscounts );
167168 $ discountData = $ quoteItemDiscounts [0 ]->getDiscountData ();
168169 $ ruleLabel = $ quoteItemDiscounts [0 ]->getRuleLabel ();
169170 $ this ->assertEquals (5 , $ discountData ->getAmount ());
@@ -173,6 +174,7 @@ public function testDiscountsOnQuoteWithFixedDiscount(): void
173174 $ this ->assertEquals ('TestRule_Coupon ' , $ ruleLabel );
174175
175176 $ quoteAddressItemDiscount = $ quote ->getShippingAddressesItems ()[0 ]->getExtensionAttributes ()->getDiscounts ();
177+ $ this ->assertArrayHasKey ('0 ' , $ quoteAddressItemDiscount );
176178 $ discountData = $ quoteAddressItemDiscount [0 ]->getDiscountData ();
177179 $ ruleLabel = $ quoteAddressItemDiscount [0 ]->getRuleLabel ();
178180 $ this ->assertEquals (5 , $ discountData ->getAmount ());
Original file line number Diff line number Diff line change 4141
4242 'store_id ' => 0 ,
4343 'store_label ' => 'TestRule_Coupon ' ,
44-
4544 ]
4645 ]
4746);
You can’t perform that action at this time.
0 commit comments