ICEfaces
  1. ICEfaces
  2. ICE-10208

ace:gMap QA test application failures

          Details

          • Assignee Priority:
            P2

            Description

            In the QA test app for ace:gMap, the following tests failed:

            1) ace:gMapServices Dynamic Attribute Test (all browsers): a JS error occurs when selecting ``distance`` option in the ``name`` drop-down and enabling ``rendered``:
            IE10 & Chrome:
            Message: Unable to get property 'charAt' of undefined or null reference
            Line: 316
            Char: 249
            Code: 0
            URI: http://localhost:8080/gMap/javax.faces.resource/util/ace-components.js.jsf?ln=icefaces.ace&v=4_0_0_140829

            FF31 error:
            TypeError: c[1] is undefined
            http://localhost:8080/gMap/javax.faces.resource/util/ace-components.js.jsf?ln=icefaces.ace&v=4_0_0_140829
            Line 316

            Test page: /gMapDynServices.jsf

            2) ace:gMapMarker / ace:gMapInfoWindow Test: setting the ``rendered`` checkboxes on or off does not make the markers appear or disappear.

            Test page: /gMapMarkerInfoWindow.jsf

              Activity

              Ascending order - Click to sort in descending order
              Carmen Cristurean created issue -
              Ken Fyten made changes -
              Field Original Value New Value
              Assignee Arturo Zambrano [ artzambrano ]
              Security Private [ 10001 ]
              Fix Version/s 4.0 [ 11382 ]
              Assignee Priority P2 [ 10011 ]
              Carmen Cristurean made changes -
              Environment ICEfaces 4.0.Beta release code; IF4 trunk rev.42336
              Browsers: all
              Server: Tomcat 7.0.42               		ICEfaces 4.0.Beta release code; IF4 trunk rev.42336
              Browsers: all
              Server: Tomcat 7.0.42
              QA test app: http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Manual/gMap
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #42486 Tue Sep 02 11:57:16 MDT 2014 art.zambrano ICE-10208 fix to avoid processing distance service if not enough points are provided; improved tld documentation
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/gmap/GMapServicesMeta.java
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/gmap/gmap.js
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #42487 Tue Sep 02 13:12:30 MDT 2014 art.zambrano ICE-10208 added onElementRemove callback for ace:gMapMarker to properly remove markers when the component is switched to rendered=false
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/gmap/GMapMarkerRenderer.java
              Hide
              Permalink
              Arturo Zambrano added a comment -

              Committed fixes to 4.0 trunk at revision 42487.

              The error in the first issue was caused because the points weren't supplied in the test app, so a check to see if at least two points were supplied was added in order to prevent the error.

              The second issue occurred because the gmap instance needed a explicit call to remove the marker (since the gmap main object stays in memory across requests), so this call was added in an onElementRemove callback.

              Show
              Arturo Zambrano added a comment - Committed fixes to 4.0 trunk at revision 42487. The error in the first issue was caused because the points weren't supplied in the test app, so a check to see if at least two points were supplied was added in order to prevent the error. The second issue occurred because the gmap instance needed a explicit call to remove the marker (since the gmap main object stays in memory across requests), so this call was added in an onElementRemove callback.
              Arturo Zambrano made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Carmen Cristurean added a comment -

              Verified with IF4 trunk rev. 42487 in Firefox31.

              Show
              Carmen Cristurean added a comment - Verified with IF4 trunk rev. 42487 in Firefox31.
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Arturo Zambrano
                  Reporter:
                  Carmen Cristurean
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: