[shale] Clay DTD

[Wendy Smoak <ja...@wendysmoak.com>]

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: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org