Opened 13 years ago

Closed 11 years ago

#90 closed enhancement (fixed)

Allow xs:integer in ValueScheme

Reported by: dietuyt Owned by: olhsha
Priority: minor Milestone:
Component: ComponentRegistry Version:
Keywords: Cc: dieuyt

Description

The types you can select for the valueScheme does not allow integers. Gertjan Postma of Meertens requested this during the cmdi workshop.

I checked the schema and it seems that integers are not a primitive type but a derrived type not sure if this makes any difference. Consult Dieter on this.

Change History (13)

comment:1 Changed 13 years ago by patdui

Thorsten also would like to have xs:id added. Is there a good reason why we just don't add all xml-types?

comment:2 Changed 13 years ago by twagoo

Owner: changed from patdui to twagoo
Status: newassigned

comment:3 Changed 13 years ago by dietuyt

I contacted Gertjan to ask why float or digital is not enough. In theory we could indeed support all xml types, so far we took the KISS approach.

comment:4 Changed 12 years ago by twagoo

What's the status of this? I added dateTime (see #163) to the development branch (soon te be deployed I hope), perhaps a good time to revisit.

comment:5 Changed 12 years ago by dietuyt

We never got an answer from Gertjan, so I consider this request to be obsolete/not so important. I would give this ticket a WONTFIX status.

comment:6 Changed 12 years ago by twagoo

Resolution: wontfix
Status: assignedclosed

comment:7 Changed 12 years ago by twagoo

Resolution: wontfix
Status: closedreopened

There was a discussion today at a CMDI tutorial about this.
I think we should reconsider adding the int type, since it potentially has many valid use cases. Also, it is in fact supported by CMDI (any standard xml type is) and many people will be looking for it.

We should perhaps also consider having the positive and negative integer distinction.

comment:8 Changed 12 years ago by dietuyt

Ok, if there is a real demand for this we can add it.

comment:9 Changed 12 years ago by twagoo

Owner: changed from twagoo to olhsha
Status: reopenedassigned

comment:10 Changed 11 years ago by dietuyt

We've had several requests to add integer now, so I would suggest to add xs:int (http://www.w3.org/TR/xmlschema-2/#int)

comment:11 Changed 11 years ago by dietuyt

Reporter: changed from patdui to dietuyt

comment:12 Changed 11 years ago by olhsha

fixed at Ola's branch

comment:13 Changed 11 years ago by twagoo

Resolution: fixed
Status: assignedclosed

Changed in general component schema in r2517.
ComponentRegistry-1.13 will take values in editor from schema, so no further work to be done; closing as fixed.

Last edited 11 years ago by twagoo (previous) (diff)
Note: See TracTickets for help on using tickets.