308221 - Update bugzilla.dtd to comply with schema

Status: RESOLVED FIXED

(Bugzilla :: Bug Import/Export & Moving, defect)

Description

[Greg Hendricks]

Add such things as flags, isprivate, and attachment encoding

Comment 1

[Greg Hendricks]

Attachment: v1

I am not sure if flags need to be defined twice, once for bugs and again for
attachments, or if it can be done only once. I was unable to find any examples
where an element is defined more than in one element. To be safe I defined it
in both places.

Comment 2

[Greg Hendricks]

Attachment: test xml file

Exported from updated show.xml.tmpl

Comment 3

[Colin Ogilvie [:cso]]

Comment on attachment 195789 [details] [diff] [review]
v1

There is no need to define flag twice (dupe ELEMENT and ATTLIST) and there's no
definition for the filename part of attachment.

Comment 4

[Greg Hendricks]

Attachment: v2

fixed the problems outlined.

Comment 5

[Colin Ogilvie [:cso]]

Comment on attachment 195801 [details] [diff] [review]
v2

>Index: bugzilla.dtd

>-<!ELEMENT bug (bug_id, (alias?, creation_ts, short_desc, delta_ts, reporter_accessible, cclist_accessible, classification_id, classification, product, component, version, rep_platform, op_sys, bug_status, resolution?, bug_file_loc?, status_whiteboard?, keywords*, priority, bug_severity, target_milestone?, dependson*, blocked*, votes?, reporter, assigned_to, qa_contact?, cc*, (estimated_time, remaining_time, actual_time)?, group*, long_desc*, attachment*)?)>
>+<!ELEMENT bug (bug_id, (alias?, creation_ts, short_desc, delta_ts, reporter_accessible, cclist_accessible, classification_id, classification, product, component, version, rep_platform, op_sys, bug_status, resolution?, bug_file_loc?, status_whiteboard?, keywords*, priority, bug_severity, target_milestone?, dependson*, blocked*, votes?, reporter, assigned_to, qa_contact?, cc*, (estimated_time, remaining_time, actual_time)?, group*, long_desc*, attachment*, flag*)?)>

Your element order is a little off here ... flag comes before longdesc, so
should be:

group*, flag*, long_desc*

>+<!ELEMENT attachment (attachid, date, desc, filename?, type?, data?, flag*)>

type comes before filename...

Comment 6

[Greg Hendricks]

Attachment: v3

Moved flag to be in the proper order.
I will change the location of filename in show.xml.tmpl so that it matches
here.

Comment 7

[Greg Hendricks]

Attachment: v4

Added another missing element: deadline

Comment 8

[Greg Hendricks]

Attachment: test xml file

Comment 9

[Colin Ogilvie [:cso]]

Comment on attachment 195967 [details] [diff] [review]
v4

r=me

I've tested the previous version of this patch, and am doing the deadline
addition by inspection.

Comment 10

[Frédéric Buclin]

Checking in bugzilla.dtd;
/cvsroot/mozilla/webtools/bugzilla/bugzilla.dtd,v  <--  bugzilla.dtd
new revision: 1.12; previous revision: 1.11
done