ICEfaces
  1. ICEfaces
  2. ICE-10323

ace:gMap - gMapEvent not called when used with gMapAutocomplete

    Details

      Description

      When using the ace:gMapEvent tag to call some script on click, this isn't called when it is used in an ace:gMap that uses an ace:gMapAutocomplete tag.

      This depends on the order that the tags are placed. If the gMapEvent tag is placed before the gMapAutocomplete tag, the events are not fired. If it is placed after the tag, then it works correctly.

        Activity

        Arran Mccullough created issue -
        Hide
        Arran Mccullough added a comment -

        Attached test case that shows the issue.

        • Load welcomeICEfaces.jsf
        • click on the map, the lat and long of where you clicked should be shown.
        Show
        Arran Mccullough added a comment - Attached test case that shows the issue. Load welcomeICEfaces.jsf click on the map, the lat and long of where you clicked should be shown.
        Arran Mccullough made changes -
        Field Original Value New Value
        Attachment Case13083Example.war [ 17515 ]
        Attachment Case13083Example.zip [ 17516 ]
        Ken Fyten made changes -
        Assignee Arturo Zambrano [ artzambrano ]
        Fix Version/s EE-4.0.0.GA [ 11171 ]
        Fix Version/s EE-3.3.0.GA_P03 [ 11572 ]
        Assignee Priority P2 [ 10011 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #43385 Fri Nov 14 15:12:29 MST 2014 art.zambrano ICE-10323 added mechanism to save event instances and re-apply them when recreating the entire map
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/gmap/gmap.js
        Hide
        Arturo Zambrano added a comment -

        r43384: committed fix to the 3.3 EE maintenance branch.
        r43385: committed fix to the 4.0 trunk.

        The problem wasn't specific to gMapAutocomplete. It occurred any time the recreate() method was called, which is necessary to do under certain circumstances. This method creates a new map instance and restores the previous state. The issue was simply the fact that this recreate() method wasn't restoring events, so they were lost whenever this method was called. So, the fix consisted in adding a mechanism to save the event instances and then re-apply them when recreating the entire map.

        Show
        Arturo Zambrano added a comment - r43384: committed fix to the 3.3 EE maintenance branch. r43385: committed fix to the 4.0 trunk. The problem wasn't specific to gMapAutocomplete. It occurred any time the recreate() method was called, which is necessary to do under certain circumstances. This method creates a new map instance and restores the previous state. The issue was simply the fact that this recreate() method wasn't restoring events, so they were lost whenever this method was called. So, the fix consisted in adding a mechanism to save the event instances and then re-apply them when recreating the entire map.
        Arturo Zambrano made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Carmen Cristurean added a comment -

        Verified with IF 4.0 trunk r43553 and 3.3 EE maintenance branch r43548 in IE8, IE9, IE10, FF33, Chrome39.

        Show
        Carmen Cristurean added a comment - Verified with IF 4.0 trunk r43553 and 3.3 EE maintenance branch r43548 in IE8, IE9, IE10, FF33, Chrome39.
        Ken Fyten made changes -
        Fix Version/s 4.1 [ 11375 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Arran Mccullough
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: