ICEfaces
  1. ICEfaces
  2. ICE-7899

REGRESSION: EE Composite Showcase fails with JS error on IE7/8

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.0.0.BETA
    • Fix Version/s: 3.0.1, EE-3.0.0.GA
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      ICEfaces EE 3.0.0, regression as of commit svn rvn # 28277

      Description

      The commit for ICE-7781 is resulting in JS errors being thrown by the EE Composite Showcase application when run with MyFaces (Mojarra is fine).

      ICEfaces-ee/trunk/icefaces-ee rev. 30763
      Server: Tomcat 7.0.0 (icepc13)/7.0.8 (local)
      Browsers: IE7/8

      There is a JS error (IE8) or Runtime Error (IE7 /icepc13) when clicking on any of the left side links on the EE Component Showcase main page.
      No errors in IE9.

      IE8:
      Webpage error details
      User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; WOW64; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; MDDS; .NET4.0C)
      Timestamp: Wed, 14 Mar 2012 17:40:54 UTC

      Message: Object doesn't support this property or method
      Line: 1
      Char: 39412
      Code: 0
      URI: http://localhost:8080/ee-composite-showcase/javax.faces.resource/jsf.js.jsf?ln=javax.faces&v=766874567


      The offending commit is:

      28277 3/12/12 12:31 PM 1 mircea.toma ICE-7781 Changed PanelPopupRenderer to use JavascriptContext.addJavascriptCall method to send the JS code for auto-centering instead rendering the JS code in the component's markup. Using this approach we make sure the auto-centering is sent along every update of the popup.

        Issue Links

          Activity

          Repository Revision Date User Message
          ICEsoft Public SVN Repository #29452 Tue Jun 12 11:34:54 MDT 2012 judy.guglielmin add my faces modified 2.1.7-ice-7899 jars and poms to snapshots repo for testing samples builds
          Files Changed
          Commit graph ADD /maven2/snapshots/org/apache/myfaces/core/myfaces-impl/2.1.7-ice-7899/myfaces-impl-2.1.7-ice-7899.pom
          Commit graph ADD /maven2/snapshots/org/apache/myfaces/core/myfaces-api/2.1.7-ice-7899
          Commit graph ADD /maven2/snapshots/org/apache/myfaces
          Commit graph ADD /maven2/snapshots/org/apache/myfaces/core/myfaces-impl/2.1.7-ice-7899
          Commit graph ADD /maven2/snapshots/org/apache
          Commit graph ADD /maven2/snapshots/org/apache/myfaces/core/myfaces-impl/2.1.7-ice-7899/myfaces-impl-2.1.7-ice-7899.jar
          Commit graph ADD /maven2/snapshots/org/apache/myfaces/core/myfaces-impl
          Commit graph ADD /maven2/snapshots/org/apache/myfaces/core/myfaces-api/2.1.7-ice-7899/myfaces-api-2.1.7-ice-7899.jar
          Commit graph ADD /maven2/snapshots/org/apache/myfaces/core/myfaces-api/2.1.7-ice-7899/myfaces-api-2.1.7-ice-7899.pom
          Commit graph ADD /maven2/snapshots/org/apache/myfaces/core
          Commit graph ADD /maven2/snapshots/org/apache/myfaces/core/myfaces-api
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #28397 Mon Mar 19 10:33:07 MDT 2012 deryk.sinotte ICE-7899: added patch that fixes the condition for loading cleint-side script for defining non-native Array.prototype.forEach function
          Files Changed
          Commit graph DEL /icefaces3/trunk/icefaces/lib/myfaces/myfaces-impl-2.1.6-ice-7855.jar
          Commit graph ADD /icefaces3/trunk/icefaces/lib/myfaces/myfaces-impl-2.1.6-ice-7926.jar
          Commit graph DEL /icefaces3/trunk/icefaces/lib/myfaces/myfaces-api-2.1.6-ice-7855.jar
          Commit graph ADD /icefaces3/trunk/icefaces/lib/myfaces/myfaces-api-2.1.6-ice-7926.jar
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #28396 Mon Mar 19 10:32:06 MDT 2012 deryk.sinotte ICE-7899: added patch that fixes the condition for loading cleint-side script for defining non-native Array.prototype.forEach function
          Files Changed
          Commit graph ADD /icefaces3/branches/icefaces-3.0.x-maintenance/icefaces/lib/myfaces/myfaces-api-2.1.6-ice-7926.jar
          Commit graph ADD /icefaces3/branches/icefaces-3.0.x-maintenance/icefaces/lib/myfaces/myfaces-impl-2.1.6-ice-7926.jar
          Commit graph DEL /icefaces3/branches/icefaces-3.0.x-maintenance/icefaces/lib/myfaces/myfaces-impl-2.1.6-ice-7855.jar
          Commit graph DEL /icefaces3/branches/icefaces-3.0.x-maintenance/icefaces/lib/myfaces/myfaces-api-2.1.6-ice-7855.jar
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #28324 Wed Mar 14 12:30:42 MDT 2012 philip.breau ICE-7899
          Files Changed
          Commit graph MODIFY /icefaces3/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/panelpopup/PanelPopupRenderer.java

            People

            • Assignee:
              Deryk Sinotte
              Reporter:
              Ken Fyten
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: