MAX! Update for new xsd definitions

[marcelrv]

Also includes:
Set boost needs a setpoint temperature #1821
(openhab/openhab1-addons#1821) alternative approach
simplifying the code, eliminating the need for PR #82

new lines at end of all files as suggested by Thomas.

Signed-off-by: marcel [email protected]

[kaikreuzer]

Great, nice to see the XML extensions being used so quickly!
There is just one thing I wonder: Whether it makes sense to have the "max" tag on the channels - what is the idea behind it?
I know we are lacking any kind of guideline on how tags should be used (and we will also have to come up with some default tag library) - so no offence at all.

[marcelrv]

Well,

The thinking behind it was that it would be nice to have all the items
related to the max system in one screen. Hence the max tag.
I made all the regular heating stuff (mode, set temp and actual temp) as
heating reflecting the basic items for the heating system.

I assume we will later in the UI be able to group stuff by the tag

Also I want to see if I can dynamically add tags later onwards, as I would
like to tag with the room names (coming from the max cube definitions) as
well.

2015-01-07 23:08 GMT+01:00 Kai Kreuzer [email protected]:

Great, nice to see the XML extensions being used so quickly!
There is just one thing I wonder: Whether it makes sense to have the "max"
tag on the channels - what is the idea behind it?
I know we are lacking any kind of guideline on how tags should be used
(and we will also have to come up with some default tag library) - so no
offence at all.

—
Reply to this email directly or view it on GitHub
#105 (comment).

[kaikreuzer]

The thinking behind it was that it would be nice to have all the items
related to the max system in one screen. Hence the max tag.

Hm, if this would be a best practice", we could actually adapt the thing infrastructure to add such tags automatically for items - so even in this case there is no need to have them specified here. I would therefore suggest to remove them!

[kaikreuzer]

Besides this, this PR is ready for merging.

[marcelrv]

Okay removed the max tag.

[kaikreuzer]

Thanks!