OFBiz
  1. OFBiz
  2. OFBIZ-5125

No states displayed for default country in New Shipping Address for order

    Details

      Description

      On the demo site, go to https://demo-trunk.ofbiz.apache.org/ordermgr/control/orderview?orderId=WSCO10001 and click on New Shipping Address. No states are displayed for the default country, USA. Once you choose a country, states are displayed properly.

      Have a look at the pattern used elsewhere in OFBiz in framework/common/webcommon/includes/setDependentDropdownValuesJs.ftl (https://fisheye6.atlassian.com/browse/ofbiz/trunk/framework/common/webcommon/includes/setDependentDropdownValuesJs.ftl). A change event calls a function, in this case getDependentDropdownValues, and there's a call to the function to set an initial value for the dependent value based on the default value of the source data.

      Compare the code in https://fisheye6.atlassian.com/browse/ofbiz/trunk/applications/order/webapp/ordermgr/images/js/OrderShippingInfo.js, which sets up the change event but does not set a default value. The patch includes a call to getAssociatedStateList to fix the problem.

        Issue Links

          Activity

          Hide
          Jacques Le Roux added a comment -

          Hi Paul,

          Thanks to take setDependentDropdownValuesJs as a reference . I agree on the fix.

          Show
          Jacques Le Roux added a comment - Hi Paul, Thanks to take setDependentDropdownValuesJs as a reference . I agree on the fix.
          Hide
          Paul Foxworthy added a comment -

          Committed to trunk in rev 1451195
          11.04 1451198
          12.04 1451203

          10.04 predates jQuery and doesn't have this problem

          Show
          Paul Foxworthy added a comment - Committed to trunk in rev 1451195 11.04 1451198 12.04 1451203 10.04 predates jQuery and doesn't have this problem
          Hide
          Jacques Le Roux added a comment -

          Please Paul, close issues once done (it's our policy) thanks :)0

          Show
          Jacques Le Roux added a comment - Please Paul, close issues once done (it's our policy) thanks :)0

            People

            • Assignee:
              Paul Foxworthy
              Reporter:
              Paul Foxworthy
            • Votes:
              1 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development