OFBiz
  1. OFBiz
  2. OFBIZ-4805

Change shipping order address failed

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Not a Problem
    • Affects Version/s: Trunk
    • Fix Version/s: None
    • Component/s: order
    • Labels:
      None

      Description

      When you select a new shipping address in order view, operation failed because the service that update orderItemShipGroup haven't all information on carrier.

      1. OFBIZ-4805.patch
        1 kB
        Nicolas Malin
      2. OFBIZ-4805.patch
        1 kB
        Nicolas Malin

        Activity

        Nicolas Malin created issue -
        Nicolas Malin made changes -
        Field Original Value New Value
        Attachment OFBIZ-4805.patch [ 12522572 ]
        Hide
        Jacques Le Roux added a comment -

        Sorry Nicolas, this patch does not apply

        Show
        Jacques Le Roux added a comment - Sorry Nicolas, this patch does not apply
        Hide
        Nicolas Malin added a comment -

        Oups, my apologies jacques, I recreate the patch.

        Show
        Nicolas Malin added a comment - Oups, my apologies jacques, I recreate the patch.
        Nicolas Malin made changes -
        Attachment OFBIZ-4805.patch [ 12522597 ]
        Hide
        Jacques Le Roux added a comment -

        Hi Nicolas,

        Before applying your patch on trunk demo I had any problems (but the states dependent dropdown not working at load when creating the new adress in the layered popup, unrelated).
        After having applied your patch locally I have also any issues.

        I just would like to know in which case you encoutered this issue to reproduce. A carrier must be used?

        Note: for both I used DEMO10090 order

        Show
        Jacques Le Roux added a comment - Hi Nicolas, Before applying your patch on trunk demo I had any problems (but the states dependent dropdown not working at load when creating the new adress in the layered popup, unrelated). After having applied your patch locally I have also any issues. I just would like to know in which case you encoutered this issue to reproduce. A carrier must be used? Note: for both I used DEMO10090 order
        Hide
        Nicolas Malin added a comment -

        Hi Jacques,

        Strange, I have the problem on customer production site and I try on my local. The service updateOrderItemShipGroup failed on call-bsh element to decompose shipmentMethod keys. But on demo-trunk, I haven't the problem. I apply this patch to unblock the production site.

        The problem would be present with shipmentMethod NoShipping, NA, CARRIER.

        Sorry for the noise, if I reproduce this problem on demo-trunk, I will complete this issue.

        Thanks jacques for your review !

        Show
        Nicolas Malin added a comment - Hi Jacques, Strange, I have the problem on customer production site and I try on my local. The service updateOrderItemShipGroup failed on call-bsh element to decompose shipmentMethod keys. But on demo-trunk, I haven't the problem. I apply this patch to unblock the production site. The problem would be present with shipmentMethod NoShipping, NA , CARRIER. Sorry for the noise, if I reproduce this problem on demo-trunk, I will complete this issue. Thanks jacques for your review !
        Hide
        Nicolas Malin added a comment -

        Hi Jacques,
        I found the problem.
        The customer data have been an error on OISG. The Order Contact Mech and OISG contact Mech were different, just on the case and when I tested on my local OFBiz trunk, I have embeded bad data.

        Thanks jacques for your check !

        Show
        Nicolas Malin added a comment - Hi Jacques, I found the problem. The customer data have been an error on OISG. The Order Contact Mech and OISG contact Mech were different, just on the case and when I tested on my local OFBiz trunk, I have embeded bad data. Thanks jacques for your check !
        Nicolas Malin made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Not A Problem [ 8 ]
        Jacques Le Roux made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Jacques Le Roux [ jacques.le.roux ]
        Hide
        Paul Foxworthy added a comment -

        Re Jacques's comment of 14/Apr/12: See OFBIZ-5125 for a fix for initial state list.

        Show
        Paul Foxworthy added a comment - Re Jacques's comment of 14/Apr/12: See OFBIZ-5125 for a fix for initial state list.

          People

          • Assignee:
            Jacques Le Roux
            Reporter:
            Nicolas Malin
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development