Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0.0-b5
    • Component/s: None
    • Labels:
      None

      Description

      Create Blesta\Core\Pricing\Presenter.

      The pricing presenter should perform all of the necessary actions that the PricingPresenter component does, but should use MetaItemPrice from CORE-2207.

      The presenter should also support functionality currently implemented in Invoices::getItemTotals, i.e. methods should exist that allow the items in the ItemPriceCollection, a set of totals (i.e. subtotal, total, total_after_tax, total_after_discount, tax_amount, discount_amount), a set of discounts, and a set of taxes to be fetched.

        Issue Links

          Activity

          tyson Tyson Phillips (Inactive) created issue -
          Automated transition triggered when Tyson Phillips (Inactive) created a branch in Stash -
          Field Original Value New Value
          Status Open [ 1 ] In Progress [ 3 ]
          admin Paul Phillips made changes -
          Fix Version/s 4.0.0-b2 [ 11001 ]
          Fix Version/s 4.0.0-b1 [ 10603 ]
          admin Paul Phillips made changes -
          Fix Version/s 4.0.0-b3 [ 11002 ]
          Fix Version/s 4.0.0-b2 [ 11001 ]
          admin Paul Phillips made changes -
          Fix Version/s 4.0.0-b4 [ 11003 ]
          Fix Version/s 4.0.0-b3 [ 11002 ]
          admin Paul Phillips made changes -
          Fix Version/s 4.0.0-b5 [ 11004 ]
          Fix Version/s 4.0.0-b4 [ 11003 ]
          Hide
          jonathan Jonathan Reissmueller added a comment - - edited

          Blesta\Core\Pricing\Presenter\Build\ServiceChange\ServiceChangeBuilder has a function called pairCollections(). This function helps to determine whether new items have been added in a service change using keys that are created to identify each item. Currently setup fees and cancellation fees for configurable options have their own keys (service-##-option-##-setup/ service-##-option-##-cancel). This means that when a configurable option is is changed from a value without a setup fee to one with a setup fee, that setup fee is included in though total even though there was no new configurable option added. This applies to terms changes as well.

          Show
          jonathan Jonathan Reissmueller added a comment - - edited Blesta\Core\Pricing\Presenter\Build\ServiceChange\ServiceChangeBuilder has a function called pairCollections(). This function helps to determine whether new items have been added in a service change using keys that are created to identify each item. Currently setup fees and cancellation fees for configurable options have their own keys (service-##-option-##-setup/ service-##-option-##-cancel). This means that when a configurable option is is changed from a value without a setup fee to one with a setup fee, that setup fee is included in though total even though there was no new configurable option added. This applies to terms changes as well.
          Hide
          tyson Tyson Phillips (Inactive) added a comment -

          Can you include a truth table illustrating the cases?

          Show
          tyson Tyson Phillips (Inactive) added a comment - Can you include a truth table illustrating the cases?
          Hide
          jonathan Jonathan Reissmueller added a comment - - edited
          From Option Value To Option Value Included In Total Box
          None w/ setup yes
          w/o setup w/ setup yes
          w/ setup w/ setup no
          Show
          jonathan Jonathan Reissmueller added a comment - - edited From Option Value To Option Value Included In Total Box None w/ setup yes w/o setup w/ setup yes w/ setup w/ setup no
          jonathan Jonathan Reissmueller made changes -
          Link This issue blocks CORE-2327 [ CORE-2327 ]
          Automated transition triggered when Tyson Phillips (Inactive) created pull request #220 in Stash -
          Status In Progress [ 3 ] In Review [ 5 ]
          Resolution Fixed [ 1 ]
          Automated transition triggered when Tyson Phillips (Inactive) merged pull request #220 in Stash -
          Status In Review [ 5 ] Closed [ 6 ]
          tyson Tyson Phillips (Inactive) made changes -
          Link This issue relates to CORE-1685 [ CORE-1685 ]
          tyson Tyson Phillips (Inactive) made changes -
          Link This issue relates to CORE-1667 [ CORE-1667 ]

            People

            • Assignee:
              tyson Tyson Phillips (Inactive)
              Reporter:
              tyson Tyson Phillips (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                10/Feb/17

                Agile