227307 - 'about' and 'resource' attrs need namespace prefix

Status: RESOLVED FIXED

(Core Graveyard :: RDF, defect)

Description

[Bob DuCharme]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.5) Gecko/20031007 Firebird/0.7
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.5) Gecko/20031007 Firebird/0.7

The use of these attributes without namespace prefixes is deprecated, so they
need the prefix that goes with the http://www.w3.org/1999/02/22-rdf-syntax-ns#
URL. In general, when arp (http://www-uk.hpl.hp.com/people/jjc/arp/) finds no
problems with your RDF, your RDF is ready for consumption. (Instead of
downloading and installing arp, you can just paste your RDF into the W3C's front
end to it at http://www.w3.org/RDF/Validator/).

Reproducible: Always

Steps to Reproduce:
1. I used Mozwho's new feature to save bookmark RDF to disk
2. validate RDF using arp (see Details)


Actual Results:  
lots of warning messages

Expected Results:  
Added RDF namespace prefix to 'about' and 'resource' attributes in RDF.

Comment 1

[Jo Hermans]

Since Mozilla isn't capable of exporting the bookmarks in an RDF file (see bug
177886), isn't this a bug in mozwho ?

Comment 2

[Andy Edmonds]

MozWho just invokes Components.interfaces.nsIRDFXMLSerializer, so certainly one
fix for the issue is to manually generate an RDF from a more manual js process.

However, my guess is that the *issue* is either with the representation of the
bookmarks internally or the serializer.

The point is a long term one, looking to the future of RDF as a common storage
format, and inter-operability among applications.

Confirming, adjusting OS and severity -- but it wouldn't be the first time I've
screwed up in b.m.o.

Comment 3

[Chase Tingley]

Andy is correct, this is a serializer issue.  In the early days of the RDF spec,
the namespace was optional for certain attributes (about/resource/ID), but this
is no longer the case.

Comment 4

[Chase Tingley]

Attachment: fix

This puts everything into the RDF namespace, and also cleans up a few duplicate
string constants that were scattered throughout the file.

Comment 5

[Chase Tingley]

*** Bug 189854 has been marked as a duplicate of this bug. ***

Comment 6

[Robert John Churchill]

Comment on attachment 139008 [details] [diff] [review]
fix

r=rjc

Comment 7

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 139008 [details] [diff] [review]
fix

sr=bzbarsky

Comment 8

[Chase Tingley]

Fix checked in.