OFBiz
  1. OFBiz
  2. OFBIZ-4972

NumberFormateException while update order items.

    Details

      Description

      If order item price or quantity is greater then 999 then update order items service throw NumberFormateException.
      Steps to regenerate:

      • Create an order(SO/PO).
      • Edit order items, set order price 1000 and update order items.
      • Again edit order items and try to update price from (1,000 to 1,002).
        System will throw NPE and if same will perform for quantity then it will throw NumberFormateException on console.
      1. OFBIZ-4972.patch
        3 kB
        Deepak Dixit
      2. OFBIZ-4972-10.04.patch
        3 kB
        Deepak Dixit

        Activity

        Hide
        Deepak Dixit added a comment -

        Here is the patch to fix the issue.

        Cause: Update order items service not able to parse the price/quantity that are in comma format.

        Fix: Used ObjectType.simpleTypeConvert method to convert price/quantity string to BigDecimal instead of using BigDecimal constructor.

        Show
        Deepak Dixit added a comment - Here is the patch to fix the issue. Cause: Update order items service not able to parse the price/quantity that are in comma format. Fix: Used ObjectType.simpleTypeConvert method to convert price/quantity string to BigDecimal instead of using BigDecimal constructor.
        Hide
        Ashish Vijaywargiya added a comment -

        Committed code at trunk at r1385014, R12.04 at r1385015, R11.04 at r1385016 & R10.04 at r1385017.

        Thanks Deepak for the contribution.


        Ashish Vijaywargiya

        Show
        Ashish Vijaywargiya added a comment - Committed code at trunk at r1385014, R12.04 at r1385015, R11.04 at r1385016 & R10.04 at r1385017. Thanks Deepak for the contribution. – Ashish Vijaywargiya
        Hide
        Deepak Dixit added a comment -

        Thanks Ashish.

        Show
        Deepak Dixit added a comment - Thanks Ashish.

          People

          • Assignee:
            Ashish Vijaywargiya
            Reporter:
            Deepak Dixit
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development