Skip to content
This repository has been archived by the owner on Jul 28, 2023. It is now read-only.

.setPropertyType() to support multiple types #19

Open
mkovatsc opened this issue Jul 13, 2016 · 1 comment
Open

.setPropertyType() to support multiple types #19

mkovatsc opened this issue Jul 13, 2016 · 1 comment

Comments

@mkovatsc
Copy link

It is valid and in a number of cases required to set multiple types.
Maybe we could change the signature to setPropertyType(String ... propertyType) and make the @type field an array if (propertyType.length>1)?

@danielpeintner
Copy link
Collaborator

Mhh, not sure if I understand the issue. Please let me re-phrase my understanding.

At the moment one can specify @type fields for the "main" thing and for each property, action, and event.

What you are requesting is to allow an array for any @type, correct? e.g.,

"@type": ["Thing", "Sensor"]

IF so I propose to change the signature to setPropertyType(JsonNode propertyType) OR would this be not precise enough?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants