QuailAutomation / openhab2-addons

The next-generation open Home Automation Bus (openHAB)
Eclipse Public License 2.0
8 stars 7 forks source link

BaseThingHandler.initialize() will be removed soon #128

Closed swamiller closed 5 years ago

swamiller commented 6 years ago

I am getting the following error repeatedly during restart:

2018-03-31 22:06:04.387 [WARN ] [.core.thing.binding.BaseThingHandler] - BaseThingHandler.initialize() will be removed soon, ThingStatus can be set manually via updateStatus(ThingStatus.ONLINE)

The message seems associated with ominlink thing status updates during restart.

craigham commented 6 years ago

yes, there is a call to a inherited method around initialize that has been deprecated in openhab. Someone will have to remove that call.

boc-tothefuture commented 5 years ago

@craigham I am doing a few things in OpenHab. My time will be limited, but I may be able to make some progress here and there. What version of OpenHab are we currently targeting? I am currently on 2.2, have we tested this driver on 2.3?

craigham commented 5 years ago

@boc-tothefuture which version of openhab is a good question. About a month ago I tried to get the repository caught up to openhab/openhab and didn't get very far, and gave up.

I think ideally we want the branch current to the latest master. I'm not sure if we have to squash all the commits first and rebase, or what. Have gotten out of my comfort zone with git unfortunately.

What do you think?

swamiller commented 5 years ago

Looks like this one is fixed.