Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[deconz] Fixed wrong channel type for extended color light #10132

Merged
merged 1 commit into from
Feb 13, 2021
Merged

[deconz] Fixed wrong channel type for extended color light #10132

merged 1 commit into from
Feb 13, 2021

Conversation

cweitkamp
Copy link
Contributor

  • Fixed wrong channel type for extended color light

Unfortunately I missed to revert one Color Temperature channel type in #10126 because it was written in a wrong way anyways.

2021-02-13 10:33:31.133 [ERROR] [ng.xml.internal.ThingTypeXmlProvider] - Could not register ThingType: deconz:extendedcolorlight
java.lang.IllegalArgumentException: ID segment 'system.color-temperature-abs' contains invalid characters. Each segment of the ID must match the pattern [\w-]*.
	at org.openhab.core.common.AbstractUID.validateSegment(AbstractUID.java:98) ~[bundleFile:?]
	at org.openhab.core.common.AbstractUID.<init>(AbstractUID.java:76) ~[bundleFile:?]
	at org.openhab.core.common.AbstractUID.<init>(AbstractUID.java:49) ~[bundleFile:?]
	at org.openhab.core.thing.UID.<init>(UID.java:48) ~[bundleFile:?]
	at org.openhab.core.thing.type.ChannelTypeUID.<init>(ChannelTypeUID.java:42) ~[bundleFile:?]
	at org.openhab.core.thing.xml.internal.ChannelXmlResult.toChannelDefinition(ChannelXmlResult.java:135) ~[bundleFile:?]
	at org.openhab.core.thing.xml.internal.ThingTypeXmlResult.toChannelDefinitions(ThingTypeXmlResult.java:98) ~[bundleFile:?]
	at org.openhab.core.thing.xml.internal.ThingTypeXmlResult.getBuilder(ThingTypeXmlResult.java:148) ~[bundleFile:?]
	at org.openhab.core.thing.xml.internal.ThingTypeXmlResult.toThingType(ThingTypeXmlResult.java:156) ~[bundleFile:?]
	at org.openhab.core.thing.xml.internal.ThingTypeXmlProvider.addingFinished(ThingTypeXmlProvider.java:137) [bundleFile:?]
	at org.openhab.core.config.xml.osgi.XmlDocumentBundleTracker.addingFinished(XmlDocumentBundleTracker.java:247) [bundleFile:?]
	at org.openhab.core.config.xml.osgi.XmlDocumentBundleTracker.parseDocuments(XmlDocumentBundleTracker.java:411) [bundleFile:?]
	at org.openhab.core.config.xml.osgi.XmlDocumentBundleTracker.processBundle(XmlDocumentBundleTracker.java:382) [bundleFile:?]
	at org.openhab.core.config.xml.osgi.XmlDocumentBundleTracker$2.run(XmlDocumentBundleTracker.java:347) [bundleFile:?]
	at java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source) [?:?]
	at java.util.concurrent.FutureTask.run(Unknown Source) [?:?]
	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(Unknown Source) [?:?]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source) [?:?]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source) [?:?]
	at java.lang.Thread.run(Unknown Source) [?:?]

Signed-off-by: Christoph Weitkamp github@christophweitkamp.de

@cweitkamp
Fixed wrong channel type for extended color light
386a66e 
Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
@J-N-K J-N-K merged commit 6829ff6 into openhab:main Feb 13, 2021
@cweitkamp cweitkamp deleted the bugfix-deconz-use-system-default-channel-types branch February 13, 2021 11:54
@cpmeister cpmeister added this to the 3.1 milestone Feb 16, 2021
@cpmeister cpmeister added the bug An unexpected problem or unintended behavior of an add-on label Feb 16, 2021
themillhousegroup pushed a commit to themillhousegroup/openhab2-addons that referenced this pull request May 10, 2021
@cweitkamp @themillhousegroup
[deconz] Fixed wrong channel type for extended color light (openhab#1…
bba18c2 
…0132)

Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
Signed-off-by: John Marshall <john.marshall.au@gmail.com>
thinkingstone pushed a commit to thinkingstone/openhab-addons that referenced this pull request Nov 7, 2021
@cweitkamp @thinkingstone
[deconz] Fixed wrong channel type for extended color light (openhab#1…
d0912f7 
…0132)

Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
@cweitkamp
[deconz] Fixed wrong channel type for extended color light (openhab#1…
ca76762 
…0132)

Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@J-N-K J-N-K J-N-K approved these changes

Assignees
No one assigned
Labels
bug An unexpected problem or unintended behavior of an add-on
Projects
None yet
Milestone
3.1
Development

Successfully merging this pull request may close these issues.
3 participants
@cweitkamp @J-N-K @cpmeister