You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@turbine.apache.org by Danny Angus <da...@thought.co.uk> on 2002/01/15 18:19:37 UTC
[PATCH] http://jakarta.apache.org/turbine/dtd/database.dtd
I think this torque DTD needs an app-data !ELEMENT for it to be valid for
project-schema.xml which contains element app-data in the !DOCTYPE tag, and
as the root element.
I was using it with an editor which reads the DTD to populate gui editing
tools, it complained about this, I'm not the expert here ..
So here's a teeny patch.
d.
cvs -z9 diff -u database.dtd
Index: database.dtd
===================================================================
RCS file: /home/cvs/jakarta-turbine-torque/src/dtd/database.dtd,v
retrieving revision 1.10
diff -u -r1.10 database.dtd
--- database.dtd 27 Dec 2001 20:49:32 -0000 1.10
+++ database.dtd 15 Jan 2002 17:15:41 -0000
@@ -31,6 +31,8 @@
to lowercase.
-->
+<!ELEMENT app-data (database+)>
+
<!ELEMENT database (table+)>
<!ATTLIST database
name CDATA #IMPLIED
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>
RE: [PATCH] http://jakarta.apache.org/turbine/dtd/database.dtd
Posted by Danny Angus <da...@thought.co.uk>.
In that case it is project-schema.xml which needs changed..
> -----Original Message-----
> From: jmcnally [mailto:jmcnally]On Behalf Of John McNally
> Sent: Tuesday, January 15, 2002 8:11 PM
> To: Turbine Developers List
> Subject: Re: [PATCH] http://jakarta.apache.org/turbine/dtd/database.dtd
>
>
> I am pretty sure that is how jason intended it. You can specify
> multiple schema files, for example all .xml files in the schema
> directory, as an input to torque.
>
> john mcnally
>
> Danny Angus wrote:
> >
> > But it contains two database elements, and they need to be
> contained in a
> > root element which there seems to be no scope for in the DTD.
> > If you go that way each schema can only have one database.
> > d.
> >
> > > -----Original Message-----
> > > From: jmcnally [mailto:jmcnally]On Behalf Of John McNally
> > > Sent: Tuesday, January 15, 2002 5:55 PM
> > > To: Turbine Developers List
> > > Subject: Re: [PATCH]
> http://jakarta.apache.org/turbine/dtd/database.dtd
> > >
> > >
> > > project-schema.xml should be fixed. not database.dtd
> > >
> > > john mcnally
> > >
> > > Danny Angus wrote:
> > > >
> > > > I think this torque DTD needs an app-data !ELEMENT for it to be
> > > valid for
> > > > project-schema.xml which contains element app-data in the
> > > !DOCTYPE tag, and
> > > > as the root element.
> > > > I was using it with an editor which reads the DTD to populate
> > > gui editing
> > > > tools, it complained about this, I'm not the expert here ..
> > > > So here's a teeny patch.
> > > >
> > > > d.
> > > >
> > > > cvs -z9 diff -u database.dtd
> > > > Index: database.dtd
> > > > ===================================================================
> > > > RCS file: /home/cvs/jakarta-turbine-torque/src/dtd/database.dtd,v
> > > > retrieving revision 1.10
> > > > diff -u -r1.10 database.dtd
> > > > --- database.dtd 27 Dec 2001 20:49:32 -0000 1.10
> > > > +++ database.dtd 15 Jan 2002 17:15:41 -0000
> > > > @@ -31,6 +31,8 @@
> > > > to lowercase.
> > > > -->
> > > >
> > > > +<!ELEMENT app-data (database+)>
> > > > +
> > > > <!ELEMENT database (table+)>
> > > > <!ATTLIST database
> > > > name CDATA #IMPLIED
> > > >
> > > > --
> > > > To unsubscribe, e-mail:
> > <ma...@jakarta.apache.org>
> > > For additional commands, e-mail:
> > <ma...@jakarta.apache.org>
> >
> > --
> > To unsubscribe, e-mail:
> > <ma...@jakarta.apache.org>
> > For additional commands, e-mail:
> > <ma...@jakarta.apache.org>
> >
> > --
> > To unsubscribe, e-mail:
<ma...@jakarta.apache.org>
> For additional commands, e-mail:
<ma...@jakarta.apache.org>
--
To unsubscribe, e-mail:
<ma...@jakarta.apache.org>
For additional commands, e-mail:
<ma...@jakarta.apache.org>
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>
Re: [PATCH] http://jakarta.apache.org/turbine/dtd/database.dtd
Posted by John McNally <jm...@collab.net>.
I am pretty sure that is how jason intended it. You can specify
multiple schema files, for example all .xml files in the schema
directory, as an input to torque.
john mcnally
Danny Angus wrote:
>
> But it contains two database elements, and they need to be contained in a
> root element which there seems to be no scope for in the DTD.
> If you go that way each schema can only have one database.
> d.
>
> > -----Original Message-----
> > From: jmcnally [mailto:jmcnally]On Behalf Of John McNally
> > Sent: Tuesday, January 15, 2002 5:55 PM
> > To: Turbine Developers List
> > Subject: Re: [PATCH] http://jakarta.apache.org/turbine/dtd/database.dtd
> >
> >
> > project-schema.xml should be fixed. not database.dtd
> >
> > john mcnally
> >
> > Danny Angus wrote:
> > >
> > > I think this torque DTD needs an app-data !ELEMENT for it to be
> > valid for
> > > project-schema.xml which contains element app-data in the
> > !DOCTYPE tag, and
> > > as the root element.
> > > I was using it with an editor which reads the DTD to populate
> > gui editing
> > > tools, it complained about this, I'm not the expert here ..
> > > So here's a teeny patch.
> > >
> > > d.
> > >
> > > cvs -z9 diff -u database.dtd
> > > Index: database.dtd
> > > ===================================================================
> > > RCS file: /home/cvs/jakarta-turbine-torque/src/dtd/database.dtd,v
> > > retrieving revision 1.10
> > > diff -u -r1.10 database.dtd
> > > --- database.dtd 27 Dec 2001 20:49:32 -0000 1.10
> > > +++ database.dtd 15 Jan 2002 17:15:41 -0000
> > > @@ -31,6 +31,8 @@
> > > to lowercase.
> > > -->
> > >
> > > +<!ELEMENT app-data (database+)>
> > > +
> > > <!ELEMENT database (table+)>
> > > <!ATTLIST database
> > > name CDATA #IMPLIED
> > >
> > > --
> > > To unsubscribe, e-mail:
> <ma...@jakarta.apache.org>
> > For additional commands, e-mail:
> <ma...@jakarta.apache.org>
>
> --
> To unsubscribe, e-mail:
> <ma...@jakarta.apache.org>
> For additional commands, e-mail:
> <ma...@jakarta.apache.org>
>
> --
> To unsubscribe, e-mail: <ma...@jakarta.apache.org>
> For additional commands, e-mail: <ma...@jakarta.apache.org>
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>
RE: [PATCH] http://jakarta.apache.org/turbine/dtd/database.dtd
Posted by Danny Angus <da...@thought.co.uk>.
But it contains two database elements, and they need to be contained in a
root element which there seems to be no scope for in the DTD.
If you go that way each schema can only have one database.
d.
> -----Original Message-----
> From: jmcnally [mailto:jmcnally]On Behalf Of John McNally
> Sent: Tuesday, January 15, 2002 5:55 PM
> To: Turbine Developers List
> Subject: Re: [PATCH] http://jakarta.apache.org/turbine/dtd/database.dtd
>
>
> project-schema.xml should be fixed. not database.dtd
>
> john mcnally
>
> Danny Angus wrote:
> >
> > I think this torque DTD needs an app-data !ELEMENT for it to be
> valid for
> > project-schema.xml which contains element app-data in the
> !DOCTYPE tag, and
> > as the root element.
> > I was using it with an editor which reads the DTD to populate
> gui editing
> > tools, it complained about this, I'm not the expert here ..
> > So here's a teeny patch.
> >
> > d.
> >
> > cvs -z9 diff -u database.dtd
> > Index: database.dtd
> > ===================================================================
> > RCS file: /home/cvs/jakarta-turbine-torque/src/dtd/database.dtd,v
> > retrieving revision 1.10
> > diff -u -r1.10 database.dtd
> > --- database.dtd 27 Dec 2001 20:49:32 -0000 1.10
> > +++ database.dtd 15 Jan 2002 17:15:41 -0000
> > @@ -31,6 +31,8 @@
> > to lowercase.
> > -->
> >
> > +<!ELEMENT app-data (database+)>
> > +
> > <!ELEMENT database (table+)>
> > <!ATTLIST database
> > name CDATA #IMPLIED
> >
> > --
> > To unsubscribe, e-mail:
<ma...@jakarta.apache.org>
> For additional commands, e-mail:
<ma...@jakarta.apache.org>
--
To unsubscribe, e-mail:
<ma...@jakarta.apache.org>
For additional commands, e-mail:
<ma...@jakarta.apache.org>
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>
Re: [PATCH] http://jakarta.apache.org/turbine/dtd/database.dtd
Posted by John McNally <jm...@collab.net>.
project-schema.xml should be fixed. not database.dtd
john mcnally
Danny Angus wrote:
>
> I think this torque DTD needs an app-data !ELEMENT for it to be valid for
> project-schema.xml which contains element app-data in the !DOCTYPE tag, and
> as the root element.
> I was using it with an editor which reads the DTD to populate gui editing
> tools, it complained about this, I'm not the expert here ..
> So here's a teeny patch.
>
> d.
>
> cvs -z9 diff -u database.dtd
> Index: database.dtd
> ===================================================================
> RCS file: /home/cvs/jakarta-turbine-torque/src/dtd/database.dtd,v
> retrieving revision 1.10
> diff -u -r1.10 database.dtd
> --- database.dtd 27 Dec 2001 20:49:32 -0000 1.10
> +++ database.dtd 15 Jan 2002 17:15:41 -0000
> @@ -31,6 +31,8 @@
> to lowercase.
> -->
>
> +<!ELEMENT app-data (database+)>
> +
> <!ELEMENT database (table+)>
> <!ATTLIST database
> name CDATA #IMPLIED
>
> --
> To unsubscribe, e-mail: <ma...@jakarta.apache.org>
> For additional commands, e-mail: <ma...@jakarta.apache.org>
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>