Khronos Public Bugzilla
Bug 9 - allow spaces in names
: allow spaces in names
Status: CLOSED FIXED
Product: COLLADA
Classification: Unclassified
Component: Schema
: 1.4.1
: All All
: P3 enhancement
: Release 1.5.0
Assigned To: Mark Barnes
: COLLADA Work Group email alias
https://collada.org/public_forum/view...
:
Depends on:
Blocks: 64
  Show dependency treegraph
 
Reported: 2006-08-07 19:40 PDT by Erwin Coumans
Modified: 2009-04-18 23:22 PDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Erwin Coumans 2006-08-07 19:40:39 PDT
See discussion here: https://collada.org/public_forum/viewtopic.php?t=514

At the moment, name attribute in node and rigid_body is of type NCName. This
means it cannot contain spaces. This requirement is too strict, and not
necessary in my opinion.

Can we change the name to be generic string?
Comment 1 Mark Barnes 2006-08-07 19:58:54 PDT
In XML Schema, strings can have all manner of characters and whitespace including newlines, carriage returns, vertical tabs, etc. etc..

Perhaps the best data type for name attributes is xs:token? A token is a normalized string that may contain a single space between words.
Comment 2 Mark Barnes 2006-08-11 18:56:07 PDT
Consider loosening the constraints for 1.5 release.
Comment 3 Ellen Finch 2007-08-29 21:22:18 PDT
Reopening for consideration in 1.5.0 per comment 2.

(Same as internal bug 532: https://cvs.khronos.org/bugzilla/show_bug.cgi?id=532 )
Comment 4 Mark Barnes 2008-03-09 21:29:04 PDT
Ellen what is the impact on the spec editing if we change all name attribute types in the schema from xs:NCName to xs:token? We need a spec bug to block on this too.
Comment 5 Ellen Finch 2008-03-10 17:34:44 PDT
I can do it pretty quickly. I will do so.
Comment 6 Ellen Finch 2008-03-10 17:48:34 PDT
Note that the schema contains a redefinition:

<xs:simpleType name="token">
		<xs:restriction base="xs:token"/>
	</xs:simpleType>

so we should use "token" instead of "xs:token".

The larger question is, "xs:token" is widely used within the schema. Should any others (or all) be changed to "token"? (If so, maybe this should be a different bug.)
Comment 7 Mark Barnes 2008-05-10 19:27:21 PDT
Fixed.
Comment 8 Ellen Finch 2009-04-18 23:22:32 PDT
Verified that all "name" attributes are xs:token in 1.5.0 except in unit. Opening separate Bug 146 for next release.