169197 - Bugzilla RDF format has errors and deprecated style

Status: VERIFIED FIXED

(Bugzilla :: Query/Bug List, defect)

Description

[Heikki Toivonen (remove -bugzilla when emailing directly)]

The RDF format sent out by Bugzilla has errors and deprecated style according to
the W3C RDF validator. Python RDFlib can not deal with the data because of
these, which is blocking me & Gagan from upgrading the http://status/bugs tools. 

I do not understand the errors at the moment (luckily RDFlib does not seem to
care about these it seems), but the warnings with attributes happen because they
do not inherit the default namespace (per the namespaces Recommendation). The
simple fix is to make the RDF namespace use a prefix.

Below is a report from the W3C RDF validator
(http://www.w3.org/RDF/Validator/ARPServlet):

The original RDF/XML document

1: <?xml version="1.0"?>
2: <!-- [e-mail] -->
3: <RDF xmlns="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
4:      xmlns:bz="http://www.bugzilla.org/rdf#"
5:      xmlns:nc="http://home.netscape.com/NC-rdf#">
6: 
7: <bz:result
about="http://bugzilla.mozilla.org/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&bug_status=UNCONFIRMED&email1=heikki%40netscape.com&emailtype1=substring&emailassigned_to1=1&short_desc=&short_desc_type=substring&long_desc=&long_desc_type=substring&bug_file_loc=&bug_file_loc_type=substring&field0-0-0=short_desc&type0-0-0=substring&value0-0-0=&format=rdf">
8:   <bz:installation resource="http://bugzilla.mozilla.org/" />
9:   <bz:bugs>
10:     <Seq>
11:       <li>
12:         
13:         <bz:bug about="http://bugzilla.mozilla.org/show_bug.cgi?id=164344">
14:           
15:           <bz:id nc:parseType="Integer">164344</bz:id>
16:         
17:           <bz:severity>critical</bz:severity>
18:           <bz:priority>--</bz:priority>
19:           <bz:owner>[e-mail]</bz:owner>
20:           <bz:os>Linux</bz:os>
21:           <bz:target_milestone>---</bz:target_milestone>
22:           <bz:status_whiteboard></bz:status_whiteboard>
23:           <bz:keywords>hang, regression</bz:keywords>
24:           <bz:summaryfull>Hang after XMLHttpRequest call, trying to dismiss
alert</bz:summaryfull>
25:         
26:         </bz:bug>
27:       
28:       </li>
29: 
30: 
31:     </Seq>
32: 
33:   </bz:bugs>
34: 
35: </bz:result>
36: 
37: </RDF> 

Error Messages
Error: {E201} Syntax error when processing "164344". Encountered "164344" Was
expecting one of: XML comment processing instruction end element tag property
attributes attribute rdf:type [Line = 15, Column = 41]

Warning Messages
Warning: {W101} Unqualified use of rdf:about has been deprecated.[Line = 7,
Column = 453]
Warning: {W101} Unqualified use of rdf:resource has been deprecated.[Line = 8,
Column = 62]
Warning: {W101} Unqualified use of rdf:about has been deprecated.[Line = 13,
Column = 76]

Comment 1

[Myk Melez [:myk] [@mykmelez]]

Attachment: patch v1: fixes warnings

Here's a patch that fixes the warnings.  It adds an "rdf" prefix for the RDF
namespace and uses it for the "about" and "resource" attributes.  It retains
the default RDF namespace for RDF tags, however.

The error may have something to do with the nc:parseType attribute.  Does it
still appear if you remove it?

Comment 2

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Uh, don't really like the double namespace declaration, but it is syntactically
correct and saves a little bandwidth so if you want to go with that its fine.

The nc:parseType attribute seems also to cause problems. With that I can get my
python app to spew out strange data and eventually crash when it tries to handle
the bug numbers (could be I don't know what I am doing as this is my first
Python app). Removing the attribute makes the validator happy and my program
runs as expected. 

Of course it would be nice to be able to say in the RDF the type of data, but I
am not yet sure how to do that. I'll do some reading and experimenting.

Comment 3

[Heikki Toivonen (remove -bugzilla when emailing directly)]

I think that datatype can be saved for another bug. All we need to status/bugs
to work is the bug count, and that works ok with these changes.

Comment 4

[Gervase Markham [:gerv]]

Comment on attachment 99529 [details] [diff] [review]
patch v1: fixes warnings

r=gerv. As far as I understand it, this makes no difference in XML terms, and
if it makes Python happy, then great.

Gerv

Comment 5

[Myk Melez [:myk] [@mykmelez]]

Fixed, and fix applied to b.m.o.

Checking in template/en/default/list/list.rdf.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/list/list.rdf.tmpl,v  <--
 list.rdf.tmpl
new revision: 1.2; previous revision: 1.1
done

Comment 6

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Verified, the deprecated attribute warnings are gone
(http://www.w3.org/RDF/Validator/)