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

MiOS Binding: Issue #3317 Tracking last used pin on locks. #3377

Merged
merged 1 commit into from Nov 6, 2015

Conversation

mrguessed
Copy link
Contributor

Bugfix for Issue #3317 Tracking last used pin on locks.

MiOS Item Generator (XSLT) incorrectly generated the sl_UserCode UPnP attribute as Number type instead of String type.

Fix involves:
a) Emitting original value as String type, which is of the form:

    "UserID=".+" UserName=".*"

b) Emitting new Item declaration *UserCode_userid representing the UserID value encoded in the original value (Number).

c) Emitting new Item declaration *UserCode_username representing the UserName value encoded in the original value (String).

d) When transforming multiply-bound source values, avoid destroying the original/untransformed value.

e) Tighten the parse logic to work better with Transformations (like REGEX) that have complex parameters.

Note: Online docs will be updated once the PR is in the codebase. The fix isn't isolated to the MiOS Item Generator code, but it helped me find an issue found in the MiOS Binding, which needed to be fixed for it all to work together cleanly

Signed-off-by: Mark Clark mr.guessed@gmail.com

MiOS Item Generator (XSLT) incorrectly generated the `sl_UserCode`
UPnP attribute as `Number` type instead of String type.

Fix involves:
a) Emitting original value as String type, which is of the form:
      "UserID=".+" UserName=".*"
b) Emitting new Item declaration `*UserCode_userid` representing the
   _UserID_ value encoded in the original value (`Number`).
c) Emitting new Item declaration `*UserCode_username` representing
   the _UserName_ value encoded in the original value (`String`).
d) When transforming multiply-bound source values, avoid destroying
   the original/untransformed value.

Signed-off-by: Mark Clark <mr.guessed@gmail.com>
@teichsta teichsta added the bug label Nov 6, 2015
@teichsta teichsta added this to the 1.8.0 milestone Nov 6, 2015
teichsta added a commit that referenced this pull request Nov 6, 2015
MiOS Binding: Issue #3317 Tracking last used pin on locks.
@teichsta teichsta merged commit e2a215b into openhab:master Nov 6, 2015
@teichsta
Copy link
Member

teichsta commented Nov 6, 2015

Thanks @mrguessed!

@mrguessed
Copy link
Contributor Author

Thanks @teichsta, appreciate the merge!

@mrguessed mrguessed deleted the mios_issue_3317_lockpin branch November 8, 2015 01:42
@mrguessed mrguessed self-assigned this Feb 1, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants