Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Checkout Grand Total amount miscacluation #1149

Closed
alexkaranda opened this issue Mar 30, 2015 · 1 comment
Closed

Checkout Grand Total amount miscacluation #1149

alexkaranda opened this issue Mar 30, 2015 · 1 comment
Assignees
@vpelipenko
Labels
bug report Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development

Comments

@alexkaranda
Copy link

Issue: Total amounts are calculated incorrectly.
Magento Version: 0.74.0-beta1

Steps to reproduce:

  • add an item in cart with price: 10$
  • Fill in billing address and click next button
  • Fill in shipping address and click next button
  • Back to billing step
  • Click next button
  • Back to billing step
  • Click next button
  • open DB tables to check amounts
    -- DB tables are affected: tables quote, quote_address

Expected result:

  • Subtotal: 10$
  • Grand Total: 10$

Actual result:

  • Subtotal: 10$
  • Grand Total: 30$

There is also possible error with shopping cart, when we play with addresses and them go back to the shopping cart page.

If we start sharing total amount in the progress block, it might be injured as well.

Potential reason to the issue are reset methods, which are not executed for some reasons.
$address->setSubtotal(0);
$address->setBaseSubtotal(0);

$address->setGrandTotal(0);
$address->setBaseGrandTotal(0);
@sshrewz
Copy link

sshrewz commented Mar 31, 2015

@alexkaranda, thanks for submitting this issue. I've created an internal ticket (MAGETWO-35676) so this can be looked into.

@sshrewz sshrewz added Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development CS bug report labels Mar 31, 2015
@vpelipenko vpelipenko self-assigned this Mar 31, 2015
magento-team added a commit to abeeskau/magento2-community-edition that referenced this issue Apr 21, 2015
@magento-team
0.74.0-beta5
342fd78 
* Various
    * Added the new methods/fields in the Catalog Product Data Object
    * Improved the Nginx configuration sample file for better web-server responsiveness and security
    * Implemented the new look & feel for Create New Order page
    * Removed the redundant DB constraints for cascade operations related to order management
    * Implemented the mechanism of asynchronous email notifications after creation of Orders, Invoices, Shipments and Credit Memos
    * Moved the join logic on application level in order to make DB separation possible in Reports component
    * Implemented the TTL and event approaches of cache invalidation, introduced the full and the partial Varnish Cache flush
    * Moved all Setup commands to Magento CLI
    * Exposed CMS API as Web API
* Fixed bugs:
    * Unexpected response for API "/V1/customers/password" service
    * Cant include a third-party link to frontend section via layout
    * Specified details for Grouped product are lost after adding to wishlist
    * Impossible to configure products in customer wishlist in Admin Panel
    * Adding the product from wishlist to cart if more than one store view exists
    * Specified product field custom options is not displayed in wishlist in Admin Panel
    * Checkout doesn't work with JS bundling enabled in production mode
    * Issue with price excluding tax when selecting downloadable links
    * Undefined index warning in case the frontend cache information is missing in configuration file
    * "New Order" email is not sent to customer after placing order via API service
    * 503 error when placing order with multiple shipping addresses if mail transport doesn't exist
    * Broken words for fields with long labels all over the Admin Panel
    * Issue with saving 'is_virtual' flag in quote
    * "Void" button available after "Deny Payment" operation
    * Uninstall logic did not clean cache properly
    * Obsolete code tests did not cover Tests folders
    * Random fail of Magento\Log\Test\Unit\Model\VisitorTest
* GitHub issues:
   * [#1149] (magento/magento2#1149) -- Checkout Grand Total amount miscalculation
   * [#1165] (magento/magento2#1165) -- Fix typos
   * [#1182] (magento/magento2#1182) -- Update system.xml for 'fix' sortOrder in adminhtml
   * [#1186] (magento/magento2#1186) -- SalesSequence: Fixed composer installer dependency
magento-team added a commit to abeeskau/magento2-community-edition that referenced this issue Apr 28, 2015
@magento-team
0.74.0-beta5
6cbe7dd 
* Various
    * Added the new methods/fields in the Catalog Product Data Object
    * Improved the Nginx configuration sample file for better web-server responsiveness and security
    * Implemented the new look & feel for Create New Order page
    * Removed the redundant DB constraints for cascade operations related to order management
    * Implemented the mechanism of asynchronous email notifications after creation of Orders, Invoices, Shipments and Credit Memos
    * Moved the join logic on application level in order to make DB separation possible in Reports component
    * Implemented the TTL and event approaches of cache invalidation, introduced the full and the partial Varnish Cache flush
    * Moved all Setup commands to Magento CLI
    * Exposed CMS API as Web API
* Fixed bugs:
    * Unexpected response for API "/V1/customers/password" service
    * Cant include a third-party link to frontend section via layout
    * Specified details for Grouped product are lost after adding to wishlist
    * Impossible to configure products in customer wishlist in Admin Panel
    * Adding the product from wishlist to cart if more than one store view exists
    * Specified product field custom options is not displayed in wishlist in Admin Panel
    * Checkout doesn't work with JS bundling enabled in production mode
    * Issue with price excluding tax when selecting downloadable links
    * Undefined index warning in case the frontend cache information is missing in configuration file
    * "New Order" email is not sent to customer after placing order via API service
    * 503 error when placing order with multiple shipping addresses if mail transport doesn't exist
    * Broken words for fields with long labels all over the Admin Panel
    * Issue with saving 'is_virtual' flag in quote
    * "Void" button available after "Deny Payment" operation
    * Uninstall logic did not clean cache properly
    * Obsolete code tests did not cover Tests folders
    * Random fail of Magento\Log\Test\Unit\Model\VisitorTest
* GitHub issues:
   * [#1149] (magento/magento2#1149) -- Checkout Grand Total amount miscalculation
   * [#1165] (magento/magento2#1165) -- Fix typos
   * [#1182] (magento/magento2#1182) -- Update system.xml for 'fix' sortOrder in adminhtml
   * [#1186] (magento/magento2#1186) -- SalesSequence: Fixed composer installer dependency
magento-team added a commit to abeeskau/magento2-community-edition that referenced this issue Apr 28, 2015
@magento-team
0.74.0-beta5
bfc9c29 
* Various
    * Added the new methods/fields in the Catalog Product Data Object
    * Improved the Nginx configuration sample file for better web-server responsiveness and security
    * Implemented the new look & feel for Create New Order page
    * Removed the redundant DB constraints for cascade operations related to order management
    * Implemented the mechanism of asynchronous email notifications after creation of Orders, Invoices, Shipments and Credit Memos
    * Moved the join logic on application level in order to make DB separation possible in Reports component
    * Implemented the TTL and event approaches of cache invalidation, introduced the full and the partial Varnish Cache flush
    * Moved all Setup commands to Magento CLI
    * Exposed CMS API as Web API
* Fixed bugs:
    * Unexpected response for API "/V1/customers/password" service
    * Cant include a third-party link to frontend section via layout
    * Specified details for Grouped product are lost after adding to wishlist
    * Impossible to configure products in customer wishlist in Admin Panel
    * Adding the product from wishlist to cart if more than one store view exists
    * Specified product field custom options is not displayed in wishlist in Admin Panel
    * Checkout doesn't work with JS bundling enabled in production mode
    * Issue with price excluding tax when selecting downloadable links
    * Undefined index warning in case the frontend cache information is missing in configuration file
    * "New Order" email is not sent to customer after placing order via API service
    * 503 error when placing order with multiple shipping addresses if mail transport doesn't exist
    * Broken words for fields with long labels all over the Admin Panel
    * Issue with saving 'is_virtual' flag in quote
    * "Void" button available after "Deny Payment" operation
    * Uninstall logic did not clean cache properly
    * Obsolete code tests did not cover Tests folders
    * Random fail of Magento\Log\Test\Unit\Model\VisitorTest
* GitHub issues:
   * [#1149] (magento/magento2#1149) -- Checkout Grand Total amount miscalculation
   * [#1165] (magento/magento2#1165) -- Fix typos
   * [#1182] (magento/magento2#1182) -- Update system.xml for 'fix' sortOrder in adminhtml
   * [#1186] (magento/magento2#1186) -- SalesSequence: Fixed composer installer dependency
magento-team pushed a commit that referenced this issue Jan 8, 2018
@ishakhsuvarov
MAGETWO-86343: magento/magento2#NONE: Fix vault_payment_token install…
97add94 
… script type where column defaults were not set (PR #12965 backport 2.1) #1149

 - Merge Pull Request magento-engcom/magento2ce#1149 from p-bystritsky/magento2:PULL-12965-BACKPORT
 - Merged commits:
   1. 3230447
   2. 634afa8
magento-team pushed a commit that referenced this issue Jan 8, 2018
MAGETWO-86343: magento/magento2#NONE: Fix vault_payment_token install…
9da32c7 
… script type where column defaults were not set (PR #12965 backport 2.1) #1149
VitaliyBoyko pushed a commit to VitaliyBoyko/magento2 that referenced this issue Jun 22, 2018
@maghamed
Merge pull request magento#1149 from magento-engcom/inventory-sales-m…
ea401d5 
…odularity

Fix modularity of InventorySales module
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vpelipenko vpelipenko

Labels
bug report Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@vpelipenko @alexkaranda @sshrewz