Struts Menu

DTD URL broken in struts-menu.tld

Details

  • Type: Bug Bug
  • Status: Open 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. Text File
    navigator.dtd.url.patch
    15/Feb/10 10:19 PM
    0.7 kB
    Jason Thrasher
  2. File
    navigator.dtd.url.patch2
    15/Feb/10 10:43 PM
    2 kB
    Jason Thrasher

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
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
Permalink
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
Vote (1)
Watch (2)

Dates

  • Created:
    Updated: