Struts Menu
  1. Struts Menu
  2. SM-101

DTD URL broken in struts-menu.tld

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 2.4.3
    • Fix Version/s: 2.4.4
    • Component/s: Menu Core
    • Labels:
      None
    • Environment:
      linux, grails 1.1.1 (grails dev. env. and jboss 4.2.3GA)

      Description

      The systemId URL in struts-menu.tld does not exist anymore.

      This issue causes an error in grails 1.1.1 when trying to use struts-menu with gsp (granted, grails probably has some issues with its entity resolver when using jsp tag libraries, but the URL is still wrong).

      The fix is trivial. Replace:
      http://java.sun.com/dtd/web-jsptaglibrary_1_1.dtd
      with
      http://java.sun.com/j2ee/dtds/web-jsptaglibrary_1_1.dtd

      1. navigator.dtd.url.patch
        0.7 kB
        Jason Thrasher
      2. navigator.dtd.url.patch2
        2 kB
        Jason Thrasher

        Activity

        Hide
        Jason Thrasher added a comment -

        Hah, just had this issue, and I see there's no patch here, so...

        Just updates the DTD in:
        navigator/src/conf/META-INF/struts-menu-el.tld

        from the Oracle version:
        http://java.sun.com/j2ee/dtds/web-jsptaglibrary_1_2.dtd
        with contents:
        The file named http://java.sun.com/j2ee/dtds/web-jsptaglibrary_1_2.dtd
        has been renamed to http://java.sun.com/dtd/web-jsptaglibrary_1_2.dtd
        in the most current version of the specification.
        Please update your application to use the new name.

        to:
        http://java.sun.com/dtd/web-jsptaglibrary_1_2.dtd
        with contents of the DTD.

        Why they couldn't do a 302?.... dunno.

        run the patch from navigator/.

        cheers

        Show
        Jason Thrasher added a comment - Hah, just had this issue, and I see there's no patch here, so... Just updates the DTD in: navigator/src/conf/META-INF/struts-menu-el.tld from the Oracle version: http://java.sun.com/j2ee/dtds/web-jsptaglibrary_1_2.dtd with contents: The file named http://java.sun.com/j2ee/dtds/web-jsptaglibrary_1_2.dtd has been renamed to http://java.sun.com/dtd/web-jsptaglibrary_1_2.dtd in the most current version of the specification. Please update your application to use the new name. to: http://java.sun.com/dtd/web-jsptaglibrary_1_2.dtd with contents of the DTD. Why they couldn't do a 302?.... dunno. run the patch from navigator/. cheers
        Hide
        Jason Thrasher added a comment -

        It turns out that this DTD has also moved:
        http://java.sun.com/dtd/web-jsptaglibrary_1_1.dtd

        and it's referenced in:
        navigator/src/conf/META-INF/struts-menu.tld

        Oracle hasn't supplied a nice message for this one, but google's "inurl:web-jsptaglibrary_1_1.dtd" reveals a(n ugly) candidate:
        http://svn.apache.org/repos/asf/tomcat/archive/jasper/tc6.0.x/src/etc/dtd/web-jsptaglibrary_1_1.dtd

        This second patch applies this fix, but use at your own discretion.

        Show
        Jason Thrasher added a comment - It turns out that this DTD has also moved: http://java.sun.com/dtd/web-jsptaglibrary_1_1.dtd and it's referenced in: navigator/src/conf/META-INF/struts-menu.tld Oracle hasn't supplied a nice message for this one, but google's "inurl:web-jsptaglibrary_1_1.dtd" reveals a(n ugly) candidate: http://svn.apache.org/repos/asf/tomcat/archive/jasper/tc6.0.x/src/etc/dtd/web-jsptaglibrary_1_1.dtd This second patch applies this fix, but use at your own discretion.

          People

          • Assignee:
            Matt Raible
            Reporter:
            polivares
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Development