Skip to content
This repository has been archived by the owner on May 7, 2020. It is now read-only.

Hue group issue since latest snapshot #2471

Closed
rene54321 opened this issue Nov 16, 2016 · 11 comments · Fixed by #2504
Closed

Hue group issue since latest snapshot #2471

rene54321 opened this issue Nov 16, 2016 · 11 comments · Fixed by #2504

Comments

@rene54321
Copy link

rene54321 commented Nov 16, 2016

Hi,

I have upgraded to latest snapshot build from today. Now the Hue groups are not working anymore. The old release from 11.11.16 was working fine. Has something changed since than?

Following error in the log when I turn off and on an light of that group:

[pse.smarthome.core.items.GenericItem] - failed notifying listener '[gHueLights_Wohnzimmer_Color (Type=GroupItem, BaseType=ColorItem, Members=8, State=null, Label=Wohnzimmer, Category=colorwheel), Licht_Farbe_WZ_TV_Rack_Rechts, null]' about state update of item java.lang.NullPointerException: {}

Also the Paper UI control tab Show internal error 500. I am also not able anymore to switch an Hue Color Group on and off. As suggested by Kai I tried to remove the (:Color:Max) from my goup but still the same behaviour. If I remove the groups from the item definition than all is working correctly also the paper ui control tab is working and showing all things with values again.

item config:

Group:Color:MAX gHueLights_Alle_Color "Alle Lampen" 

Color Licht_Farbe_WZ_TV_Rack_Oben       "TV-Rack Oben"       <colorwheel> (gHueLights_Alle_Color)     [ "Lighting" ] { channel="hue:0200:00178816e03f:1:color" }
Color Licht_Farbe_WZ_TV_Rack_Links      "TV-Rack Links"      <colorwheel> (gHueLights_Alle_Color)     [ "Lighting" ] { channel="hue:0200:00178816e03f:2:color" }

sitemap:

Colorpicker item=gHueLights_Alle_Color label="Alle" 

Regards René

@kaikreuzer
Copy link
Contributor

If it worked before, it might be related to #2393 or a similar change. @SJKA WDYT?

@rene54321
Copy link
Author

Hi,

still the same with the build from today:

2016-11-17 14:13:08.515 [WARN ] [pse.smarthome.core.items.GenericItem] - failed notifying listener '[gHueLights_TV_Rack_Color (Type=GroupItem, BaseType=ColorItem, Members=3, State=null, Label=TV Rack, Category=colorwheel), Licht_Farbe_WZ_TV_Rack_Links, The argument 'state' must not be null or empty.]' about state update of item java.lang.IllegalArgumentException: The argument 'state' must not be null or empty.: {}

@kaikreuzer
Copy link
Contributor

Nobody claimed that anything has been fixed meanwhile.

@rene54321
Copy link
Author

Hi,

any news here? Seems I am the only one with this problem :-(

@kaikreuzer kaikreuzer changed the title Openhab2 Hue group issue since latest snapshot Hue group issue since latest snapshot Nov 18, 2016
@kaikreuzer
Copy link
Contributor

Did you notice that there are 210 open issues here?

@rene54321
Copy link
Author

You are right sorry.

@maggu2810
Copy link
Contributor

Do we have any information about the ref used by the latest known working version?

@rene54321
Copy link
Author

Hi,

thats an good question. I think the old version was around the 11.11. But I am not 100% sure as I do the updates often in the last time.

@kaikreuzer
Copy link
Contributor

@maggu2810 : I am pretty sure that it is related to the changes done in the context of #2334.

@sjsf
Copy link
Contributor

sjsf commented Nov 21, 2016

at least the timing matches. will have a look.

sjsf pushed a commit to sjsf/smarthome that referenced this issue Nov 21, 2016
fixes eclipse-archived#2471
Signed-off-by: Simon Kaufmann <simon.kfm@googlemail.com>
@sjsf
Copy link
Contributor

sjsf commented Nov 21, 2016

I think I found the root cause: #2504
As there are no tests so far covering group items, I would ask everybody following this issue to double-check that I didn't miss anything else in that area...

kaikreuzer pushed a commit that referenced this issue Nov 22, 2016
* fix state conversion between HSBType and DecimalType

fixes #2471
Signed-off-by: Simon Kaufmann <simon.kfm@googlemail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants