ICEmobile
  1. ICEmobile
  2. MOBI-511

Geolocation Tag uses id for rendering name, and name attribute has no effect

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.1 Final
    • Fix Version/s: 1.2 Beta
    • Component/s: Spring
    • Labels:
      None
    • Environment:
      n/a

      Description


      The geolocation tag has both an id and name attribute. The name is supposed to be used to submit the value in the post, and can be used for bean binding. The tag is rendering the id for the name and not using the name at all.

        Activity

        Migration created issue -
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #32251 Tue Nov 20 10:08:43 MST 2012 greg.dick MOBI-511 The id is necessary for the GeolocationTag, in order for the js to find the hidden input field.
        Files Changed
        Commit graph MODIFY /icemobile/trunk/icemobile/jsp/src/main/resources/META-INF/icemobile.tld
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #32252 Tue Nov 20 10:09:39 MST 2012 greg.dick MOBI-511 Separate the rendering of the name attribute from the id
        Files Changed
        Commit graph MODIFY /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/tags/GeolocationTag.java
        Hide
        Greg Dick added a comment - - edited

        A bit of legacy from the JSF javascript. The id is required by the javascript for finding the rendered attribute and the name should be supported for Spring JSP. I was experimenting with trying to remove one of these attributes and forgot to redo it. Unfortunately, at this point they're both required.

        Show
        Greg Dick added a comment - - edited A bit of legacy from the JSF javascript. The id is required by the javascript for finding the rendered attribute and the name should be supported for Spring JSP. I was experimenting with trying to remove one of these attributes and forgot to redo it. Unfortunately, at this point they're both required.
        Hide
        Greg Dick added a comment - - edited

        Fixed

        Show
        Greg Dick added a comment - - edited Fixed
        Migration made changes -
        Field Original Value New Value
        Reporter Migration [ remote ] User Philip.breau [ philip.breau ]
        Migration made changes -
        Fix Version/s 1.2 Beta [ 10344 ]
        Migration made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Steve Maryka
            Reporter:
            Philip Breau
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: