> > I noticed that a few of the xml files in clay-plugin and use-cases refer to: > > > "-//Apache Software Foundation//DTD Shale Clay View//EN" > "http://jakarta.apache.org/struts/dtds/clay-config_1_1.dtd";> >
Oh, good catch. I think that one should be "clay-config_1_0.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.dt > > d > 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: > > > > 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. > There are a couple copies of the DTD file: src/conf/view-config.dtd src/java/org/apache/shale/clay/config/resources/clay-config_1_0.dtd I'm not sure why we are keeping the one in src/conf/ around. I believe it was left over. Craig had me add the registration stuff for the digester so that it can load the dtd from the jar. I think it can be deleted (src/conf/view-config.dtd). It looks like the dtd under the resources package need a line or two deleted at the top of the document to make it valid. > 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? > We should fix this and I don't think we have to worry about breaking anything since it's already broken:-) I can hack on it unless you already have it fixed. > -- > Wendy > Gary > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] >
