ICEpush
  1. ICEpush
  2. PUSH-104

JSP integration's icepush.tld is not valid for JBoss 6.0.0.M5

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Alpha3
    • Fix Version/s: 2.0.0
    • Component/s: Integrations
    • Labels:
      None
    • Environment:
      JBoss 6.0.0.M5, ICEpush, icepush-jsp-push.war

      Description

      When using JBoss 6.0.0.M5 and trying to deploy the icepush-jsp-push.war, JBoss doesn't like the plain taglib:

      Caused by: org.jboss.xb.binding.JBossXBException: Failed to parse source: Failed to resolve schema nsURI= location=taglib
              at org.jboss.xb.binding.parser.sax.SaxJBossXBParser.parse(SaxJBossXBParser.java:195) [jbossxb.jar:2.0.2.Beta7]
              at org.jboss.xb.binding.UnmarshallerImpl.unmarshal(UnmarshallerImpl.java:168) [jbossxb.jar:2.0.2.Beta7]
              at org.jboss.xb.util.JBossXBHelper.parse(JBossXBHelper.java:229) [jbossxb.jar:2.0.2.Beta7]
              at org.jboss.xb.util.JBossXBHelper.parse(JBossXBHelper.java:206) [jbossxb.jar:2.0.2.Beta7]
              at org.jboss.deployers.vfs.spi.deployer.SchemaResolverDeployer.parse(SchemaResolverDeployer.java:137) [:2.2.0.Alpha7]
              at org.jboss.deployment.TldParsingDeployer.parse(TldParsingDeployer.java:64) [:6.0.0.20100911-M5]
              at org.jboss.deployment.TldParsingDeployer.parse(TldParsingDeployer.java:38) [:6.0.0.20100911-M5]
              at org.jboss.deployers.vfs.spi.deployer.SchemaResolverDeployer.parse(SchemaResolverDeployer.java:121) [:2.2.0.Alpha7]
              at org.jboss.deployers.vfs.spi.deployer.AbstractVFSParsingDeployer.handleMultipleFiles(AbstractVFSParsingDeployer.java:446) [:2.2.0.Alpha7]
              at org.jboss.deployers.vfs.spi.deployer.AbstractVFSParsingDeployer.parse(AbstractVFSParsingDeployer.java:319) [:2.2.0.Alpha7]
              at org.jboss.deployers.spi.deployer.helpers.AbstractParsingDeployerWithOutput.createMetaData(AbstractParsingDeployerWithOutput.java:376) [:2.2.0.Alpha7]
              ... 41 more

        Activity

        Hide
        Jack Van Ooststroom added a comment -

        I changed the plain <taglib> definition to:

        <taglib
        xmlns="http://java.sun.com/xml/ns/javaee"
        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
        xsi:schemaLocation="http://java.sun.com/xml/ns/javaee http://java.sun.com/xml/ns/javaee/web-jsptaglibrary_2_1.xsd"
        version="2.1">

        After making this change there were other things I had to change in the icepush.tld in order to get it to work:

        • <description> must be the first child of <taglib>
        • <tlibversion> must be <tlib-version>
        • I removed <jspversion>
        • <shortname> must be <short-name>
        • I removed <display-name> as the child of <taglib> (though this one is optional I only got it to work after removing it)
        • <description> must be the first child of <tag>
        • I removed <display-name> as the child of <tag> (though this one is optional I only got it to work after removing it)
        • <tagclass> must be <tag-class>
        • I added <body-content> as a child of <tag> (this seems mandatory)
        • <description> must be the first child of <attribute>
        • I removed <type> as a child of <attribute> (though this one is optional I only got it to work after removing it)

        I'm not familiar with taglibs, so some if these change might need to be changed again.

        Show
        Jack Van Ooststroom added a comment - I changed the plain <taglib> definition to: <taglib xmlns="http://java.sun.com/xml/ns/javaee" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://java.sun.com/xml/ns/javaee http://java.sun.com/xml/ns/javaee/web-jsptaglibrary_2_1.xsd " version="2.1"> After making this change there were other things I had to change in the icepush.tld in order to get it to work: <description> must be the first child of <taglib> <tlibversion> must be <tlib-version> I removed <jspversion> <shortname> must be <short-name> I removed <display-name> as the child of <taglib> (though this one is optional I only got it to work after removing it) <description> must be the first child of <tag> I removed <display-name> as the child of <tag> (though this one is optional I only got it to work after removing it) <tagclass> must be <tag-class> I added <body-content> as a child of <tag> (this seems mandatory) <description> must be the first child of <attribute> I removed <type> as a child of <attribute> (though this one is optional I only got it to work after removing it) I'm not familiar with taglibs, so some if these change might need to be changed again.
        Hide
        Jack Van Ooststroom added a comment -

        I went over the schema and ended up with the following changes:

        • <description> must be the first child of <taglib>
        • <tlibversion> must be <tlib-version>
        • I removed <jspversion>
        • <shortname> must be <short-name>
        • <description> must be the first child of <tag>
        • <tagclass> must be <tag-class>
        • I added <body-content> as a child of <tag> (this seems mandatory) !NOTE! for now I've chosen "empty" as the value !NOTE!
        • <description> must be the first child of <attribute>
        • I added <rtexprvalue> right before <type> (though <rtexprvalue> is optional I added it with a value of "false", which is the default, in order to get the <type> working)

        Marking this one as FIXED.

        Show
        Jack Van Ooststroom added a comment - I went over the schema and ended up with the following changes: <description> must be the first child of <taglib> <tlibversion> must be <tlib-version> I removed <jspversion> <shortname> must be <short-name> <description> must be the first child of <tag> <tagclass> must be <tag-class> I added <body-content> as a child of <tag> (this seems mandatory) !NOTE! for now I've chosen "empty" as the value !NOTE! <description> must be the first child of <attribute> I added <rtexprvalue> right before <type> (though <rtexprvalue> is optional I added it with a value of "false", which is the default, in order to get the <type> working) Marking this one as FIXED.

          People

          • Assignee:
            Unassigned
            Reporter:
            Jack Van Ooststroom
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: