Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-5697

Add a drop-down for Tax Auth Combined Id in Product Price Creation screen

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Implemented
    • Affects Version/s: Trunk
    • Fix Version/s: 16.11.01
    • Component/s: product
    • Labels:
      None
    • Sprint:
      Bug Crush Event - 21/2/2015

      Description

      Currently in Product Price Creation screen there are 3 fields:

      • Tax Auth Geo ID
      • Tax In Price
      • Tax Auth Combined Id

      We should either show only

      • Tax Auth Geo ID
      • Tax In Price
        using lookups like in Product Store screen and remove the Tax Auth Combined Id field
        or keep it and use a combined Tax Auth Geo ID+Tax In Price drop-down for it. Then we could use the same in Product Store screen to be consistent

        Activity

        Hide
        pawan.verma Pawan Verma added a comment -

        Providing patch for the improvements. Removed "Tax Auth Combined Id" field and display lookups for "Tax Authority Party ID" & "Tax Auth Geo ID" fields as showing on the product store screen.

        Show
        pawan.verma Pawan Verma added a comment - Providing patch for the improvements. Removed "Tax Auth Combined Id" field and display lookups for "Tax Authority Party ID" & "Tax Auth Geo ID" fields as showing on the product store screen.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks Pawan,

        Your modified patch is in trunk at revision: 1743254

        Ialso modified the createProductPrice service implementation (by removing taxAuthCombinedId handling in inlineHandlePriceWithTaxIncuded) and the documentation (comments about taxAuthCombinedId in the service definition)

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks Pawan, Your modified patch is in trunk at revision: 1743254 Ialso modified the createProductPrice service implementation (by removing taxAuthCombinedId handling in inlineHandlePriceWithTaxIncuded) and the documentation (comments about taxAuthCombinedId in the service definition)

          People

          • Assignee:
            jacques.le.roux Jacques Le Roux
            Reporter:
            jacques.le.roux Jacques Le Roux
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development

                Agile