Tapestry 5
  1. Tapestry 5
  2. TAP5-1721

A JavaScript typo prevents the error icon for fields containing validation errors from appearing

    Details

      Description

      Tapestry is unable to find the icon and put it in the error message.

      In FieldEventManager in tapestry.js the local variable to store the icon should be "icon" and not "com".

      Tapestry.FieldEventManager = Class.create({
      ...
      getIcon : function() {
      if (!this.icon)

      { this.com = $(this.field.id + "_icon"); }

      return this.icon;
      },
      ...

        Activity

        Pedro Ayala created issue -
        Howard M. Lewis Ship made changes -
        Field Original Value New Value
        Assignee Howard M. Lewis Ship [ hlship ]
        Howard M. Lewis Ship made changes -
        Summary javascript error: ( this.com != this.icon ) A JavaScript typo prevents the error icon for fields containing validation errors from appearing
        Howard M. Lewis Ship made changes -
        Status Open [ 1 ] Closed [ 6 ]
        Fix Version/s 5.3 [ 12316024 ]
        Resolution Fixed [ 1 ]
        Hide
        Hudson added a comment -

        Integrated in tapestry-trunk-freestyle #607 (See https://builds.apache.org/job/tapestry-trunk-freestyle/607/)
        TAP5-1721: A JavaScript typo prevents the error icon for fields containing validation errors from appearing

        hlship : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1188802
        Files :

        • /tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js
        Show
        Hudson added a comment - Integrated in tapestry-trunk-freestyle #607 (See https://builds.apache.org/job/tapestry-trunk-freestyle/607/ ) TAP5-1721 : A JavaScript typo prevents the error icon for fields containing validation errors from appearing hlship : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1188802 Files : /tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js

          People

          • Assignee:
            Howard M. Lewis Ship
            Reporter:
            Pedro Ayala
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development