612347 - Syntax errors in bugzilla.dtd prevent parsing of XML-formatted bug descriptions

Status: VERIFIED DUPLICATE of bug 538428

(Bugzilla :: Query/Bug List, defect)

Description

[Jens Alfke]

User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_6_4; en-US) AppleWebKit/534.12 (KHTML, like Gecko) Chrome/9.0.572.1 Safari/534.12
Build Identifier: 3.6.2

There are two errors in bugzilla.dtd. These cause XML parse errors when trying to read XML-formatted bug descriptions. Specifically, this is breaking FogBugz's Bugzilla import script.

(1) On line 85. there is a missing #REQUIRED or #IMPLIED keyword:
          type_id CDATA
(2) There are two declarations of the delta_ts element:
<!ELEMENT delta_ts (#PCDATA)>

Adding an #IMPLIED keyword and removing the second delta_ts declaration makes xmllint happy.

Reproducible: Always

Steps to Reproduce:
1. Fetch some bug in XML form, e.g. from <https://bugzilla.mozilla.org/show_bug.cgi?ctype=xml&id=1>
2. Run the xmllint tool on the downloaded XML
Actual Results:  
DTD-related errors are reported.

Expected Results:  
Should have parsed correctly.

Comment 1

[Frédéric Buclin]

(In reply to comment #0)
> (2) There are two declarations of the delta_ts element:
> <!ELEMENT delta_ts (#PCDATA)>

Except that the first one is for the bug, and the 2nd one for the attachment.

Comment 2

[Kent Rogers [:mrbball]]

After upgrading to 3.6.2 yesterday, we hit this problem almost immediately.
This is a BIG problem as it apparently blows IE completely out of the water.

Can someone confirm that #IMPLIED is the correct keyword to add to the
type_id line?

Comment 3

[Frédéric Buclin]

IE9 doesn't complain when I view a bug as XML.

Comment 4

[David Lawrence [:dkl]]

LpSolit, is this related to the work I have done on bug 538428 and can be closed as duplicate? bug 538428 should address the xmllint issues as I used that for testing my changes.

dkl

Comment 5

[Frédéric Buclin]

Ah yes, definitely.

Comment 6

[Serge Gautherie (:sgautherie)]

V.Duplicate