Wicket
  1. Wicket
  2. WICKET-4241

Modal Window respond 404 with Internet Explorer.

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5.3
    • Fix Version/s: 1.5.4, 6.0.0-beta1
    • Component/s: wicket-extensions
    • Labels:
      None
    • Environment:
      Internet Explorer (IE)

      Description

      Modal Window respond 404 with Internet Explorer.
      Stack trace is displayed on the console.
      This issue does not occur in chrome.

      the stack trace below.
      WARN - WicketObjects - Could not resolve class [wicket]
      java.lang.ClassNotFoundException: wicket
      at org.apache.catalina.loader.WebappClassLoader.loadClass(WebappClassLoader.java:1680)
      at org.apache.catalina.loader.WebappClassLoader.loadClass(WebappClassLoader.java:1526)
      at java.lang.ClassLoader.loadClassInternal(ClassLoader.java:320)
      at java.lang.Class.forName0(Native Method)
      at java.lang.Class.forName(Class.java:242)
      at org.apache.wicket.application.AbstractClassResolver.resolveClass(AbstractClassResolver.java:108)
      at org.apache.wicket.util.lang.WicketObjects.resolveClass(WicketObjects.java:68)
      at org.apache.wicket.request.mapper.AbstractComponentMapper.getPageClass(AbstractComponentMapper.java:138)
      at org.apache.wicket.request.mapper.BookmarkableMapper.parseRequest(BookmarkableMapper.java:110)
      at org.apache.wicket.request.mapper.AbstractBookmarkableMapper.mapRequest(AbstractBookmarkableMapper.java:263)
      at org.apache.wicket.request.mapper.CompoundRequestMapper.mapRequest(CompoundRequestMapper.java:130)
      at org.apache.wicket.request.cycle.RequestCycle.resolveRequestHandler(RequestCycle.java:181)
      at org.apache.wicket.request.cycle.RequestCycle.processRequest(RequestCycle.java:206)
      at org.apache.wicket.request.cycle.RequestCycle.processRequestAndDetach(RequestCycle.java:280)
      at org.apache.wicket.protocol.http.WicketFilter.processRequest(WicketFilter.java:162)
      at org.apache.wicket.protocol.http.WicketFilter.doFilter(WicketFilter.java:218)
      at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:235)
      at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:206)
      at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:233)
      at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:191)
      at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:127)
      at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:102)
      at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:109)
      at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:298)
      at org.apache.coyote.http11.Http11Processor.process(Http11Processor.java:859)
      at org.apache.coyote.http11.Http11Protocol$Http11ConnectionHandler.process(Http11Protocol.java:588)
      at org.apache.tomcat.util.net.JIoEndpoint$Worker.run(JIoEndpoint.java:489)
      at java.lang.Thread.run(Thread.java:595)

      1. WICKET-4241_with_patch1_404_error.png
        24 kB
        Masaya Seko
      2. WICKET-4241_with_patch1_debug.png
        12 kB
        Masaya Seko
      3. WICKET-4241_with_patch1.zip
        1.72 MB
        Masaya Seko
      4. WICKET-4241.patch
        1 kB
        Andrea Del Bene
      5. WICKET-4241.zip
        39 kB
        Masaya Seko

        Activity

        Hide
        Masaya Seko added a comment - - edited

        attach quickstart

        Show
        Masaya Seko added a comment - - edited attach quickstart
        Hide
        Andrea Del Bene added a comment - - edited

        Inside modal.js there is this if block:

        if(Wicket.Browser.isIELessThan9())

        { this.content.contentWindow.location.replace(this.settings.ie8_src); }

        else

        { this.content.contentWindow.location.replace(this.settings.src); }

        I might be wrong but It seems that 'ie8_src' is never initialized. Removing if block (leaving just 'this.content.contentWindow.location.replace(this.settings.src);') seems to fix the issue.

        Show
        Andrea Del Bene added a comment - - edited Inside modal.js there is this if block: if(Wicket.Browser.isIELessThan9()) { this.content.contentWindow.location.replace(this.settings.ie8_src); } else { this.content.contentWindow.location.replace(this.settings.src); } I might be wrong but It seems that 'ie8_src' is never initialized. Removing if block (leaving just 'this.content.contentWindow.location.replace(this.settings.src);') seems to fix the issue.
        Hide
        Martin Grigorov added a comment -

        This has been introduced by a similar ticket in 1.5.2 I think by Pedro.
        Use svn blame to find the other ticket and if we find a solution for both quickstarts it'll be perfect.

        Show
        Martin Grigorov added a comment - This has been introduced by a similar ticket in 1.5.2 I think by Pedro. Use svn blame to find the other ticket and if we find a solution for both quickstarts it'll be perfect.
        Hide
        Andrea Del Bene added a comment -

        The parameter 'settings.ie8_src' is not initialize inside ModalWindow if page is stateless. I've added initialization also for this case.

        Tested with IE 6 and 8.

        Show
        Andrea Del Bene added a comment - The parameter 'settings.ie8_src' is not initialize inside ModalWindow if page is stateless. I've added initialization also for this case. Tested with IE 6 and 8.
        Hide
        Masaya Seko added a comment -

        Thank you.
        But, in my environment do not seem to have fixed.

        I builded wicket-extensions with patch. And test.
        Modal Window respond 404 with Internet Explorer(6 and 8).

        Patch does not seem to run.

        Attached quickstart and screenshot.
        WICKET-4241_with_patch1.zip
        WICKET-4241_with_patch1_404_error.png
        WICKET-4241_with_patch1_debug.png

        Show
        Masaya Seko added a comment - Thank you. But, in my environment do not seem to have fixed. I builded wicket-extensions with patch. And test. Modal Window respond 404 with Internet Explorer(6 and 8). Patch does not seem to run. Attached quickstart and screenshot. WICKET-4241 _with_patch1.zip WICKET-4241 _with_patch1_404_error.png WICKET-4241 _with_patch1_debug.png
        Hide
        Masaya Seko added a comment -

        attached WICKET-4241_with_patch1.zip

        Show
        Masaya Seko added a comment - attached WICKET-4241 _with_patch1.zip
        Hide
        Masaya Seko added a comment -

        attached WICKET-4241_with_patch1_404_error.png

        Show
        Masaya Seko added a comment - attached WICKET-4241 _with_patch1_404_error.png
        Hide
        Masaya Seko added a comment -

        WICKET-4241_with_patch1_debug.png

        Show
        Masaya Seko added a comment - WICKET-4241 _with_patch1_debug.png
        Hide
        Andrea Del Bene added a comment -

        I've updated the patch and tested it with your quickstart and with Wicket modal window examples (with IE 6 and Firefox). I look forward to your feedbacks.

        Show
        Andrea Del Bene added a comment - I've updated the patch and tested it with your quickstart and with Wicket modal window examples (with IE 6 and Firefox). I look forward to your feedbacks.
        Hide
        Martin Grigorov added a comment -

        Looks good to me.
        Let's see whether it fixes the problem in Masaya's application and then we will apply it.

        Show
        Martin Grigorov added a comment - Looks good to me. Let's see whether it fixes the problem in Masaya's application and then we will apply it.
        Hide
        Masaya Seko added a comment -

        Modal window runs properly.
        To running with Internet explorer(6 and 8) and Chrome.

        I'm glad.
        Thank you.

        Show
        Masaya Seko added a comment - Modal window runs properly. To running with Internet explorer(6 and 8) and Chrome. I'm glad. Thank you.

          People

          • Assignee:
            Igor Vaynberg
            Reporter:
            Masaya Seko
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development