ICEfaces
  1. ICEfaces
  2. ICE-3304

Javascript Errors when inputRichText is used in a panelPopup

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.7, 1.7.1
    • Fix Version/s: 1.7.2
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      IE7

      Description

      There seems to be a few Javascript errors that are thrown when a rich text component is included in a panelPopup component. The errors are only thrown in IE7 and most likely thrown in IE6 also. The first error is when the visibility of the popup is set by using rendered attribute. The error is not thrown when the popup is first displayed but happen every time after the popup is displayed. Attached is a screenshot of the error (ie7-rendered-error.jpg). The details of the error are:
      Line: 50
      Char:945
      Error:Unspecified Error
      Code:0
      URL:http://locathost:8084/Bug4942Example/block/resource/LTQ5MTYyMDg1Mw==/editor/fckeditor.html?InstanceName=_id10%3ArichTextTwo&Toolbar=Default

      This error could be caused by using the rendered attribute to show/hide the popup.

      The other error I noticed was when the visible attribute was used to show/hide the popup. The error code is 800a025e. This is a know issue with the FCKEditor api (http://www.fckeditor.net/forums/viewtopic.php?f=6&t=8700&p=22724&hilit=800a025e#p22724).

        Activity

        Arran Mccullough created issue -
        Arran Mccullough made changes -
        Field Original Value New Value
        Attachment ie7-rendered-error.JPG [ 11121 ]
        Arran Mccullough made changes -
        Attachment main.jspx [ 11122 ]
        Hide
        Tyler Johnson added a comment -

        The issue is due to the FCKeditor being used within a hidden div. We should look at adding this workaround to our code base.

        Show
        Tyler Johnson added a comment - The issue is due to the FCKeditor being used within a hidden div. We should look at adding this workaround to our code base.
        Tyler Johnson made changes -
        Ken Fyten made changes -
        Fix Version/s 1.7.2 [ 10130 ]
        Assignee Priority P2
        Assignee Adnan Durrani [ adnan.durrani ]
        Adnan Durrani made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Adnan Durrani added a comment -

        I have created an app using the attached JSP page, and tested it with IE 6 and 7 and didn't see any problem. Arran can you please retest it with the trunk head?

        Show
        Adnan Durrani added a comment - I have created an app using the attached JSP page, and tested it with IE 6 and 7 and didn't see any problem. Arran can you please retest it with the trunk head?
        Hide
        Arran Mccullough added a comment -

        Tested with the trunk and I'm still seeing the exceptions. Possibly you can send me your war file and I can test yours out.

        Show
        Arran Mccullough added a comment - Tested with the trunk and I'm still seeing the exceptions. Possibly you can send me your war file and I can test yours out.
        Hide
        Adnan Durrani added a comment -

        War file attached

        Show
        Adnan Durrani added a comment - War file attached
        Adnan Durrani made changes -
        Attachment ICe-3304.war [ 11192 ]
        Hide
        Arran Mccullough added a comment -

        Tested the war file and I can still see the exceptions. The Javascript error appears after the popup has been displayed once. I suspect this is because of using the rendered attribute to show/hide the popup. If you change it to visible you will get the other java script exception (the one related to the FCKEditor).

        Show
        Arran Mccullough added a comment - Tested the war file and I can still see the exceptions. The Javascript error appears after the popup has been displayed once. I suspect this is because of using the rendered attribute to show/hide the popup. If you change it to visible you will get the other java script exception (the one related to the FCKEditor).
        Hide
        Adnan Durrani added a comment -

        Did you clear the cache? As I can't see any javascript error on my PC using the IE7, and I don't see on test machine using IE6.

        May be its my browser error exception setting, I'll try to change my browser setting.

        Does this exception breaks the component functionality?

        Show
        Adnan Durrani added a comment - Did you clear the cache? As I can't see any javascript error on my PC using the IE7, and I don't see on test machine using IE6. May be its my browser error exception setting, I'll try to change my browser setting. Does this exception breaks the component functionality?
        Hide
        Arran Mccullough added a comment -

        Cleared the cache and still see the messages, I don't think its breaking functionality, just very very annoying, seems to happen when the popup is closed and opened.

        Show
        Arran Mccullough added a comment - Cleared the cache and still see the messages, I don't think its breaking functionality, just very very annoying, seems to happen when the popup is closed and opened.
        Hide
        Adnan Durrani added a comment -

        Hi Mandeep,

        Arran can reproduce this error on his PC, and it doesn't happen on my PC. Can you please ask someone to deploy the attached war and run the demo under IE6 or 7 and let us know if you see any javascript exception.

        Thanks,

        Show
        Adnan Durrani added a comment - Hi Mandeep, Arran can reproduce this error on his PC, and it doesn't happen on my PC. Can you please ask someone to deploy the attached war and run the demo under IE6 or 7 and let us know if you see any javascript exception. Thanks,
        Adnan Durrani made changes -
        Assignee Adnan Durrani [ adnan.durrani ] Mandeep Hayher [ mandeep.hayher ]
        Hide
        Mandeep Hayher added a comment -

        Tested attached war file on tomcat6 with IE7, could not reproduce the problem.

        Show
        Mandeep Hayher added a comment - Tested attached war file on tomcat6 with IE7, could not reproduce the problem.
        Mandeep Hayher made changes -
        Assignee Mandeep Hayher [ mandeep.hayher ] Adnan Durrani [ adnan.durrani ]
        Hide
        Adnan Durrani added a comment -

        Arran can you please test your app on some other PC?

        Thanks,

        Show
        Adnan Durrani added a comment - Arran can you please test your app on some other PC? Thanks,
        Hide
        Arran Mccullough added a comment -

        Tested my app on Tylers machine and he was able to see the JavaScript errors. If the errors don't popup it should say Errors on page in the bottom left corner of IE.

        Show
        Arran Mccullough added a comment - Tested my app on Tylers machine and he was able to see the JavaScript errors. If the errors don't popup it should say Errors on page in the bottom left corner of IE.
        Hide
        Adnan Durrani added a comment -

        Didn't even see the error on the bottom left corner as well.

        Show
        Adnan Durrani added a comment - Didn't even see the error on the bottom left corner as well.
        Hide
        Adnan Durrani added a comment -

        Yip, I am not able to reproduce this error on my machine, so I am unable to fix it. Can you please run this attached war, and see if you get this error?

        Thanks,

        Show
        Adnan Durrani added a comment - Yip, I am not able to reproduce this error on my machine, so I am unable to fix it. Can you please run this attached war, and see if you get this error? Thanks,
        Adnan Durrani made changes -
        Assignee Adnan Durrani [ adnan.durrani ] Yip Ng [ yip.ng ]
        Hide
        yip.ng added a comment -

        .war file doesn't run. All I get is:

        HTTP Status 404 - /ICe-3304/
        description The requested resource (/ICe-3304/) is not available.

        I accessed the app. thru. Tomcat Manager. And the manager showed that it had been deployed successfully and was up and running. No errors either in the Tomcat console.

        Need to create an Intellij project to run?

        Show
        yip.ng added a comment - .war file doesn't run. All I get is: HTTP Status 404 - /ICe-3304/ description The requested resource (/ICe-3304/) is not available. I accessed the app. thru. Tomcat Manager. And the manager showed that it had been deployed successfully and was up and running. No errors either in the Tomcat console. Need to create an Intellij project to run?
        made changes -
        Attachment screenshot-1.jpg [ 11199 ]
        Hide
        yip.ng added a comment -

        Problem reproduced on my machine. See screenshot-1. The error is popped up by first clicking Show, then clicking Close, then clicking Show again.

        Show
        yip.ng added a comment - Problem reproduced on my machine. See screenshot-1. The error is popped up by first clicking Show, then clicking Close, then clicking Show again.
        Hide
        Arran Mccullough added a comment -

        This is the unspecified error. If you change how the popup is shown/removed to using visible instead of rendered it will through the 800a025e error which is a known issue with the FCKEditor. I have an example if you need one.

        Show
        Arran Mccullough added a comment - This is the unspecified error. If you change how the popup is shown/removed to using visible instead of rendered it will through the 800a025e error which is a known issue with the FCKEditor. I have an example if you need one.
        made changes -
        Attachment screenshot-2.jpg [ 11224 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #17505 Fri Sep 05 16:13:34 MDT 2008 yip.ng ICE-3304
        Applied fix for fckeditor error 800a025e.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/inputrichtext/fckeditor.zip
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #17506 Fri Sep 05 16:13:52 MDT 2008 yip.ng ICE-3304
        Applied fix for fckeditor error 800a025e.
        Files Changed
        Commit graph MODIFY /icefaces/branches/icefaces-1.7/icefaces/component/src/com/icesoft/faces/component/inputrichtext/fckeditor.zip
        Hide
        yip.ng added a comment -

        Suggested fix applied to fix error 800a025e.
        Don't know how to fix the other (Unspecified) error. The error message is useless, and when I say Yes to Debug, the script debugger just comes up totally blank.

        Show
        yip.ng added a comment - Suggested fix applied to fix error 800a025e. Don't know how to fix the other (Unspecified) error. The error message is useless, and when I say Yes to Debug, the script debugger just comes up totally blank.
        Ken Fyten made changes -
        Fix Version/s 1.7.2RC1 [ 10140 ]
        Fix Version/s 1.7.2 [ 10130 ]
        made changes -
        Attachment screenshot-3.jpg [ 11243 ]
        Hide
        yip.ng added a comment -

        Finally got the error line to display in the Script Debugger. See screenshot-3. Now we need to figure out how to debug and fix this. The FCKeditor source code is actually scrambled and then zipped, so to debug and change it requires a few extra tedious steps.

        Show
        yip.ng added a comment - Finally got the error line to display in the Script Debugger. See screenshot-3. Now we need to figure out how to debug and fix this. The FCKeditor source code is actually scrambled and then zipped, so to debug and change it requires a few extra tedious steps.
        Ken Fyten made changes -
        Fix Version/s 1.7.2 [ 10130 ]
        Fix Version/s 1.7.2RC1 [ 10140 ]
        Hide
        yip.ng added a comment -

        Suggested fix applied for error 800a025e. Issue of "unspecified error" split into ICE-3572.

        Show
        yip.ng added a comment - Suggested fix applied for error 800a025e. Issue of "unspecified error" split into ICE-3572 .
        yip.ng made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Priority P2
        Assignee Yip Ng [ yip.ng ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Arran Mccullough
          • Votes:
            4 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: