Review Request: fix writing of text:time values
Friedrich W. H. Kossebau
kossebau at kde.org
Sat Jul 7 03:05:27 BST 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105463/
-----------------------------------------------------------
(Updated July 7, 2012, 2:05 a.m.)
Review request for Calligra and Thorsten Zachmann.
Changes
-------
Okay, was not that simple as first thought. New patch should be more complete.
Description (updated)
-------
The file interoperability/wordprocessing/oowriter/oow_insert_fields.odt from calligratests currently fails on this bad value:
text:time-value="0-00-00T07:37:38"
Reason is that ODF 19.881 text:time-value specifies
"The text:time-value attribute has the data type timeOrDateTime."
and ODF 18.3.36 timeOrDateTime says
"A timeOrDateTime value is either an [xmlschema-2] time or an [xmlschema-2] dateTime value."
Now, [xmlschema-2] demands for the year part a "four-or-more digit optionally negative-signed numeral that represents the year". Also, "There is no year 0, and '0000' is not a valid lexical representation.", see http://www.w3.org/TR/xmlschema-2/#year-zero
Thus the validator fails on this (good boy).
Looking more at the code (also after sending a premature patch here) I see the DateVariable supports both time and date variables (or a combination).
Just, the storage format for both date or time can be simple (just date or time) or combined (datetime). As this info makes a difference (e.g. having just a time without date (so relative) is different from a given datetime (which is absolute)), it should be at least supported in the save and load code, even if the UI does not yet expose all that.
Attached patch fixes the code to do that, both the loading/saving of the complete data (without adding/removing info) as well as the broken "0-00-00T" stuff.
I am not satisfied with the names of the new enum ValueType, proposals welcome.
Okay to backport to 2.5?
Diffs (updated)
-----
plugins/variables/DateVariable.h 9d06879
plugins/variables/DateVariable.cpp cf5ef8b
Diff: http://git.reviewboard.kde.org/r/105463/diff/
Testing
-------
Thanks,
Friedrich W. H. Kossebau
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120707/c00d9340/attachment.htm>
More information about the calligra-devel
mailing list