Merge pull request #130 from magento-fearless-kiwis/FearlessKiwis-MAG…

…ETWO-MAGETWO-45024-Shipping-Amount-Incorrect-Currency [FearlessKiwis] Bug fix
magento · Nov 5, 2015 · acd35f4 · acd35f4
2 parents 869f987 + 938156a
commit acd35f4
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/app/code/Magento/Quote/Model/Quote/Address/Total/Shipping.php b/app/code/Magento/Quote/Model/Quote/Address/Total/Shipping.php
@@ -171,7 +171,8 @@ public function collect(
                     $total->setBaseTotalAmount($this->getCode(), $rate->getPrice());
                     $shippingDescription = $rate->getCarrierTitle() . ' - ' . $rate->getMethodTitle();
                     $address->setShippingDescription(trim($shippingDescription, ' -'));
-                    $total->setShippingAmount($rate->getPrice());
+                    $total->setBaseShippingAmount($rate->getPrice());
+                    $total->setShippingAmount($amountPrice);
                     $total->setShippingDescription($address->getShippingDescription());
                     break;
                 }

diff --git a/app/code/Magento/Quote/Test/Unit/Model/Quote/Address/Total/ShippingTest.php b/app/code/Magento/Quote/Test/Unit/Model/Quote/Address/Total/ShippingTest.php
@@ -70,6 +70,7 @@ protected function setUp()
             'Magento\Quote\Model\Quote\Address\Total',
             [
                 'setShippingAmount',
+                'setBaseShippingAmount',
                 'setBaseTotalAmount',
                 'setTotalAmount',
                 'setShippingDescription',
@@ -229,9 +230,12 @@ public function testCollect()
         $this->priceCurrency->expects($this->once())
             ->method('convert')
             ->with(5, $this->store)
-            ->willReturn(5);
+            ->willReturn(10);
         $this->total->expects($this->once())
             ->method('setShippingAmount')
+            ->with(10);
+        $this->total->expects($this->once())
+            ->method('setBaseShippingAmount')
             ->with(5);
         $this->rate->expects($this->once())
             ->method('getCarrierTitle')