Uploaded image for project: 'Tapestry'
  1. Tapestry
  2. TAPESTRY-1214

@Submit with submitType="refresh" : JavaScript syntax error

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 4.1.2
    • Fix Version/s: 4.1.2
    • Component/s: None
    • Labels:
      None
    • Environment:
      FF 1.5

      Description

      syntax error
      tapestry.form.refresh("myForm", "myButton", );

        Activity

        Hide
        mrowekg Grzegorz Mrowka added a comment -

        Occures only when onClick binding is used and dojo.event.connect with anonymous function is generated:
        tapestry.event1340173574=function(e)

        { dojo.event.browser.stopEvent(e); tapestry.form.refresh("pageForm", "importantDates_addButton", ); }

        ;
        dojo.event.connect(dojo.byId("importantDates_addButton"), "onclick", tapestry, "event1340173574");

        There is one more troublesome issue.
        When I use onclick="ognl:onClickEvent" binding on @Submit component and ognl expression is returning null, then there is no onclick binding in generated html element but tapestry is connecting anonymous function by dojo. In this situation it could use standard onclick method.

        Show
        mrowekg Grzegorz Mrowka added a comment - Occures only when onClick binding is used and dojo.event.connect with anonymous function is generated: tapestry.event1340173574=function(e) { dojo.event.browser.stopEvent(e); tapestry.form.refresh("pageForm", "importantDates_addButton", ); } ; dojo.event.connect(dojo.byId("importantDates_addButton"), "onclick", tapestry, "event1340173574"); There is one more troublesome issue. When I use onclick="ognl:onClickEvent" binding on @Submit component and ognl expression is returning null, then there is no onclick binding in generated html element but tapestry is connecting anonymous function by dojo. In this situation it could use standard onclick method.

          People

          • Assignee:
            jkuhnert Jesse Kuhnert
            Reporter:
            mrowekg Grzegorz Mrowka
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development