varamfer / openhab

Automatically exported from code.google.com/p/openhab
0 stars 0 forks source link

Code review request #95

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Purpose of code changes on this branch:

Make the autoupdate binding not require the item registry (as it is best 
practice for all bindings) and use the EventPublisher to update the item state 
(as any other binding does it as well).
Note that this change means that "status change/update" rules are fired as 
well, which was not the case before. I actually assume it was a bug before - I 
definitely expect status rules to be fired.

When reviewing my code changes, please focus on:

Regression testing to make sure that none of your use cases is negatively 
impacted by this change.

Original issue reported on code.google.com by kai.openhab on 2 Aug 2012 at 8:57

GoogleCodeExporter commented 8 years ago
this is done meanwhile

Original comment by teichsta on 22 Aug 2012 at 9:36