I noticed that a few of the xml files in clay-plugin and use-cases refer to:

<!DOCTYPE view PUBLIC
  "-//Apache Software Foundation//DTD Shale Clay View//EN"
  "http://jakarta.apache.org/struts/dtds/clay-config_1_1.dtd";>

This doesn't seem to exist... the only thing I can find is
/clay-plugin/src/java/org/apache/struts/clay/config/resources/clay-config_1_0.dtd
and that has no "example" doctype declaration such as is usually found in our dtds.

So, taking a cue from core-library/src/conf/dialog.dtd, I'd like to add the following to the clay-config dtd:

<!--
   DTD for view definitions used by the "org.apache.shale.clay" package.
   To support validation of your configuration file, include the following
   DOCTYPE element at the beginning (after the "xml" declaration):

   <!DOCTYPE view PUBLIC
"-//Apache Software Foundation//DTD Shale Clay View Configuration 1.0//EN"
     "http://struts.apache.org/dtds/shale-clay-config-1_0.dtd";>
-->

Because of the different 'label', this requires a change in ClayXmlParser where it registers the local DTD (line 96). And of course changes to the xml files that refer to the clay-config dtd.

I think the label change will break existing apps, unless we just add a second registration for the same dtd. But I'd rather make a clean break since there hasn't been a release yet.

Thoughts?

--
Wendy


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to