Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 14.12.01, 16.11.01
    • Component/s: framework
    • Labels:
      None

      Description

      When a from widget has the attribute of skip-start="true", the javascript creation references the incorrect DOM object.
      ie
      <form name="beginForm" type="single" skip-end="true>

      </form>
      <form name="middleForm" type="single" skip-start="true">
      <field name="date"><date-time/></field>
      </form>

      will result in the creation of the following
      javascript call: javascript:call_cal(document.middleForm.date,''); should instead be:
      javascript call: javascript:call_cal(document.beginForm.date,'');

      or even better the super form name should be a variable.

      The second option would require an additional attribute in the form definition.

        Issue Links

          Activity

          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          I believe this has been fixed while ago.

          SinceI only tested in an environment where trunk and R14.12 are "included", I put only them as fixed, though I believe all current living releases behave correctly.

          Show
          jacques.le.roux Jacques Le Roux added a comment - I believe this has been fixed while ago. SinceI only tested in an environment where trunk and R14.12 are "included", I put only them as fixed, though I believe all current living releases behave correctly.
          Hide
          deepak.dixit Deepak Dixit added a comment -

          IMO this is related to OFBIZ-6139

          Show
          deepak.dixit Deepak Dixit added a comment - IMO this is related to OFBIZ-6139
          Hide
          deepak.dixit Deepak Dixit added a comment -

          IMO we can close this issue, as I am not able to regenerate it for layered lookup. As window lookups are deprecated so its safe to close this issue.

          Show
          deepak.dixit Deepak Dixit added a comment - IMO we can close this issue, as I am not able to regenerate it for layered lookup. As window lookups are deprecated so its safe to close this issue.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development