Issue Details (XML | Word | Printable)

Key: OFFICE-1806
Type: Bug Bug
Status: Applied Applied
Resolution: Fixed
Priority: Major Major
Assignee: Svante Schubert
Reporter: Robert Weir
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
OASIS Open Document Format for Office Applications (OpenDocument) TC

ISO/IEC JTC 1/SC 34 N 1078 : DEFECT REPORT NUMBER JP2-14

Created: 28/May/09 04:24 PM   Updated: 10/Nov/10 08:33 PM
Component/s: None
Affects Version/s: ODF 1.0, ODF 1.0 (second edition), ODF 1.0 Errata 02
Fix Version/s: ODF 1.0 Errata CD 5

Proposal: Consistently use either value type or data type. ISO prefers data type.
Resolution:

8.1.3
Value Type
Replace:
"The table:value-type attribute specifies the type of value that can appear in a cell."
with
"The office:value-type attribute specifies the type of value that can appear in a cell."

8.1.3
Cell Current Numeric Value
Replace:
"This attribute is only evaluated for cells that contain the following data types:"
with
"This attribute is present only whenever there is an office:value-type attribute with one of the following values:"

8.1.3
Cell Current Currency
Replace:
"The tableoffice:currency attribute specifies the current currency value of a cell. The value of this attribute is usually currency information such as DEM or EUR. This attribute is only evaluated for cells whose data type is currency."
with
"The office:currency attribute specifies the currency system in which the value of the office:value attribute is expressed. The value of this attribute is usually a currency identifier, such as "DEM" or "EUR". This attribute may be present only when the office:value-type attribute value is "currency"."

8.1.3
Cell Current Date Value
Replace:
"This attribute is only evaluated for cells whose data type is date."
with
"This attribute is present only whenever the office:value-type attribute valueis "date"."

8.1.3
Cell Current Time Value
Replace:
"This attribute is only evaluated for cells whose data type is time."
with
"This attribute is present only whenever the office:value-type attribute is "time"."

8.1.3
Cell Current Boolean Value
Replace:
"This attribute is only evaluated for cells whose data type is boolean."
with
"This attribute is present only whenever the office:value-type attribute value is "boolean"."

8.1.3
Cell Current String Value
Replace:
"This attribute is only evaluated for cells whose data type is string."
with
"This attribute may be present only whenever the office:value-type attribute value is "string"."

Sub-Tasks  All   Open   
 Sub-Task Progress: 

 Description  « Hide
Transcribed from http://www.itscj.ipsj.or.jp/sc34/open/1078.htm

Original author: "MURATA Makoto (FAMILY Given)" <eb2m-mrt@asahi-net.or.jp>
DEFECT REPORT NUMBER JP2-14

QUALIFIER error

REFERENCES IN DOCUMENT "Cell Current Currency", Clause 8.1.3

NATURE OF DEFECT Replace "data type" in "This attribute is only evaluated for cells whose data type is currency." by "value type". The same error is repeated in the following subclauses.

SOLUTION PROPOSED BY THE SUBMITTER


 All   Comments   Change History      Sort Order: Ascending order - Click to sort in descending order
Patrick Durusau added a comment - 06/Jul/09 01:06 PM
We have been inconsistent in our usage. I suggest the TC choose data type and I will sweep the draft for all instances. That does leave us with "value-type" in attribute names but there isn't anything I can do about that now.

Patrick Durusau added a comment - 06/Jul/09 02:01 PM
Proposal filed.

Patrick Durusau added a comment - 03/Aug/09 08:27 AM
On re-reading this I think I see what Makoto means. We called these "value types" in the prose and then proceeded to refer to them as data types. Ok, that is inconsistent, even though data type is more correct. Will correct to be consistent in this version but 1.2 should still follow the correct usage.

Dennis Hamilton added a comment - 22/Mar/10 12:03 PM
I think that we should reconsider this and also keep 1.2 consistent with 1.0, not vice versa.

Based on the typography, it is clear that these sections are making reference to the value of the office:value-type attribute. It would be more appropriate to say, for "Cell Current Currency" under 8.1.3 that "This attribute is only evaluated for cells whose <tt>office:value-type</tt> value is <tt>currency</tt>."

The same rewording is applicable in the additional subsections of 8.1.3 where the use of "data type" is inapplicable. The schema establishes a complex relationship between the different cases of office:value-type values and the other attributes which can be present.

For example, for the three "numeric" office:value-type attribute values float, percentage, and currency, the office:value attribute must be present and the data type of its value is double. So value type and data type do not correspond at all.

RECOMMENDATIONS

1. Make the change to refer to the "office:value-type value" in all of the places that are changed to mention "value type".

2. Make the same change in the preceding 8.1.3 "Cell Current Numeric Value" where the problem also exists, even though it was not specifically called out in the defect report. It is clearly part of the defect and the same correction is called for.

ADDITIONAL

3. Consider whether the form of expression that "This attribute is only evaluated when ..." makes any sense. Under the schema, the attribute is only allowed to be present under those conditions. Also, there is no attribute tableoffice:currency as mentioned in the 8.1.3 subsection "Cell Current Currency." It is office:currency, and office:currency is optional when the office:value-type value is "currency". Note further that office:currency does not hold the value (the value is carried in the office:value attribute that is required by the office:value-type="currency" case). office:currency holds a code that determines what the currency system is (USD, GBP, EUR, etc.).

Dennis Hamilton added a comment - 22/Mar/10 12:08 PM
Re-opened to resolve confusion between data type and the dependency on the values of the office:value-type attribute and the additional attributes that can occur depending on the office:value-type value.

Dennis Hamilton added a comment - 22/Mar/10 12:44 PM
I note that ODF 1.2 Part 1 CD04 section 19.387 on office:value-type does a better job of making these distinctions clear, although it could be tightened a little.

Svante Schubert added a comment - 17/May/10 03:09 PM
Dennis, I did the following:

1) Expanded the fix to the previous sub-chapter
2) Included the fix of the wrong spelled attribute name
3) Adapted different font usage for the value type

What I did not was:
4) Changed the wording of "only evaluated" as it specifies what the application does, ignoring the invalid ODF and make easy to read. I share your feelings that ODF 1.2 has an improved wording, but would like to keep the fix for the errata only to the necessary, keeping it simple.

Nevertheless thanks for the good review, spotting the above topics, we had overseen!
Svante

Dennis Hamilton added a comment - 12/Jul/10 01:57 PM
I disagree with regard to the "only evaluated." According to the schema, those attributes are only permitted when the particular office:value-type attribute is present. It is invalid for the particular attribute to occur any other time. Furthermore, "evaluated" seems to be the wrong word in any case. For all but string, these attributes are required to be present if the office:value-type fits.

To say that an invalid attribute occurrence is simply not evaluated is misleading and not part of the definition of behavior for implementations, although that could certainly be implementation-dependent behavior, just as for any attribute that appears where the schema has no provision for it.

Svante Schubert added a comment - 19/Jul/10 03:20 PM
Dennis pointed out correctly that on previous occurance was not covered. Added to the resolution:

8.1.3
Value Type
Replace:
"The table:value-type attribute specifies the type of value that can appear in a cell."
with
"The office:value-type attribute specifies the type of value that can appear in a cell."

the other sections were changed accordingly to his suggestion.

Dennis Hamilton added a comment - 10/Nov/10 08:33 PM
Errata 02 - ODF 1.2 Reconciliation. There are a few deviations in ODF 1.2 that apparently arose through the re-organization of the material. That is being handled by one or more tasks here, based on the attributes involved.