The field types boolean, decimal, float and integer should have their XML datatype outputed along with their property attribute. Here is an example of output for a price:
<span property="commerce:amount" datatype="xsd:decimal">11.99</span>
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | rdf-datatype_attribute-624974-9.patch | 5.23 KB | linclark |
| #8 | rdf-datatype_attribute-624974-8.patch | 5.01 KB | linclark |
| #7 | rdf-datatype_attribute-624974-7.patch | 2.62 KB | linclark |
| #6 | rdf-datatype_attribute-624974-6.patch | 2.62 KB | linclark |
| #2 | rdf-datatype_attribute-624974-2.patch | 2.62 KB | linclark |
Comments
Comment #1
scor commentedforgot the proper tagging
Comment #2
Anonymous (not verified) commented- For integer, decimal, float, and boolean type fields, adds the datatype to the bundle's mapping in the database when the field is added to the bundle. Because there is no xsd:integer, xsd:decimal is used.
- In rdfa_attributes, makes sure that a predicate is assigned because datatypes should only be output if the data is the subject of a triple.
- Moves the datatype assignment out of the callback conditional because a callback value is not necessary for datatyping.
Comment #3
scor commentedThanks Lin for getting started with this issue :)
xsd:decimal exists and should be used here. see http://www.w3.org/TR/2004/REC-rdf-mt-20040210/#dtype_interp for a list of XSD datatypes valid in RDFS.
EDIT: I meant xsd:integer exists and should be used here.
Comment #4
yched commentedShouldn't this go in the "rdf.module" component ? Feel free to bump back if not.
Comment #5
scor commented@yched: good point. Originally I thought field.module should declare what datatype each field should be mapped to, but we'll do that directly in rdf.module now.
Comment #6
Anonymous (not verified) commentedAh, it turns out I was just looking at the primitive datatypes and didn't make it down to the derived datatypes.
Fixed and rerolled.
Comment #7
Anonymous (not verified) commentedor this, rather (using integer instead of unsignedInt)
Comment #8
Anonymous (not verified) commentedThis now has a test.
Test currently creates a field instance with a type randomly chosen the 4 field types that have datatypes associated with them.
I placed the test function in the RDFa Markup test, but this could easily be moved to another test or its own by copying out the setup information below the comment and the function.
Comment #9
Anonymous (not verified) commentedRerolled to test all 4 fields each time and move the function to its own test.
Comment #10
scor commentedIn fact we don't need that in core since the UI will be in contrib.