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

        Deepak Dixit created issue -
        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.
        Deepak Dixit made changes -
        Field Original Value New Value
        Attachment OFBIZ-4972.patch [ 12537650 ]
        Attachment OFBIZ-4972-10.04.patch [ 12537651 ]
        Deepak Dixit made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Ashish Vijaywargiya made changes -
        Assignee Ashish Vijaywargiya [ toashishvijay ]
        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
        Ashish Vijaywargiya made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ashish Vijaywargiya made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        Deepak Dixit added a comment -

        Thanks Ashish.

        Show
        Deepak Dixit added a comment - Thanks Ashish.
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        3m 9s 1 Deepak Dixit 24/Jul/12 07:26
        Patch Available Patch Available Resolved Resolved
        52d 23h 7m 1 Ashish Vijaywargiya 15/Sep/12 06:33
        Resolved Resolved Closed Closed
        45s 1 Ashish Vijaywargiya 15/Sep/12 06:34

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development