Uploaded image for project: 'Blesta Core'
  1. Blesta Core
  2. CORE-652

Order: Need to display service prices with any tax inclusive amounts

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.0.b7
    • Fix Version/s: 4.1.1, 4.2.0-b1
    • Component/s: Plugins
    • Labels:
      None

      Description

      When adding services (through order form or when adding an addon service through the client interface) we need to display service prices to include any tax inclusive amounts.

        Issue Links

          Activity

          Hide
          tyson Tyson Phillips (Inactive) added a comment -

          I think the new pricing presenters should already handle this behavior if they are being used in those locations, otherwise they will probably need to be used for it.

          Show
          tyson Tyson Phillips (Inactive) added a comment - I think the new pricing presenters should already handle this behavior if they are being used in those locations, otherwise they will probably need to be used for it.
          Hide
          tyson Tyson Phillips (Inactive) added a comment -

          Need to check whether this issue is already resolved by this functionality using the pricing presenters.

          Show
          tyson Tyson Phillips (Inactive) added a comment - Need to check whether this issue is already resolved by this functionality using the pricing presenters.
          Hide
          jonathan Jonathan Reissmueller added a comment -

          So it seems that the service presenter treats all taxes as inclusive. It always includes the tax amounts in the overall total. This can be seen in the tax functions of the ItemPrice class, there is no reference to type in those methods or the TaxPrice method that it calls.

          Show
          jonathan Jonathan Reissmueller added a comment - So it seems that the service presenter treats all taxes as inclusive. It always includes the tax amounts in the overall total. This can be seen in the tax functions of the ItemPrice class, there is no reference to type in those methods or the TaxPrice method that it calls.
          Hide
          tyson Tyson Phillips (Inactive) added a comment -

          You may have to show this to me. I believe the item price collection should already handle this, or make it possible to fetch the totals with and without tax included, in which case the tax inclusivity/exclusivity would be determined later, wherever it is necessary to be displayed.

          Show
          tyson Tyson Phillips (Inactive) added a comment - You may have to show this to me. I believe the item price collection should already handle this, or make it possible to fetch the totals with and without tax included, in which case the tax inclusivity/exclusivity would be determined later, wherever it is necessary to be displayed.

            People

            • Assignee:
              jonathan Jonathan Reissmueller
              Reporter:
              cody Cody Phillips (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                27/Sep/17

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Time Spent - 2 hours, 57 minutes Remaining Estimate - 5 hours, 40 minutes
                5h 40m
                Logged:
                Time Spent - 2 hours, 57 minutes Remaining Estimate - 5 hours, 40 minutes
                2h 57m

                  Agile