Copied to clipboard

Flag this post as spam?

This post will be reported to the moderators as potential spam to be looked at


These support forums are now closed for new topics and comments.
Please head on over to http://eureka.ucommerce.net/ for support.

  • Robert Mulder 79 posts 272 karma points c-trib
    Dec 04, 2014 @ 20:58
    Robert Mulder
    0

    uCommerce_Discount AmountOffTotal decimal places

    I've noticed that the AmountOffTotal column of the uCommerce_Discount table is a decimal(18,2).

    Because of the way we want to display the discounts in the basket I'm using this field to do some calculations. This causes a problem with the rounding of the discounts.

    When loading the basket from the database the rounding can be different than from a 'freshly calculated' basket. That is, when performing the calculations on a basket that has already has TransactionLibrary.ExecuteBasketPipeline() called on it will have many decimal places, but performing the same calculations on a basket retrieved from the database suddenly introduces rounding issues...

    If at all possible it'd be great if the AmountOffTotal column of the uCommerce_Discount would be changed to either a MONEY datatype or simply a decimal(18,4) or something.

  • Thomas 30 posts 163 karma points
    Mar 06, 2015 @ 09:48
    Thomas
    0

    Hey Robert.

    Thank you for reporting the bug, it has been added to our backlog and will be fixed in a upcoming release of ucommerce.

    Kind regards
    Thomas Arvidsen.

  • Thomas 30 posts 163 karma points
    Apr 14, 2015 @ 13:38
    Thomas
    100

    Hey Robert

    The data type has been change in version 6.6.3 of uCommerce.

    Kind regards

    Thomas Arvidsen.

  • Robert Mulder 79 posts 272 karma points c-trib
    Apr 14, 2015 @ 13:44
    Robert Mulder
    0

    Thanks, I'll try to find a moment to do an upgrade. Do you happen to know if it's safe to upgrade to the latest Umbraco version aswell (7.2.4 at this point)

  • Thomas 30 posts 163 karma points
    Apr 14, 2015 @ 16:03
    Thomas
    0

    Hey Robert

    I have upgraded Umbraco from 7.2.0 to 7.2.4 with uCommerce and the demo store installed, and then upgrade uCommerce to the newest version. It still works.

    I recommend upgrading Umbraco and then upgrade uCommerce to a newer version.

    Kind regards

    Thomas Arvidsen.

Please Sign in or register to post replies

Write your reply to:

Draft