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

Item with unit can be changed only once in PaperUI #6074

Closed
mvalla opened this issue Aug 17, 2018 · 7 comments
Closed

Item with unit can be changed only once in PaperUI #6074

mvalla opened this issue Aug 17, 2018 · 7 comments

Comments

@mvalla
Copy link

mvalla commented Aug 17, 2018

I have an item in PaperUI connected with a channel of type Number:Temperature (with unit in Celsius).
I can edit the item from PaperUI > Control and change the number (temperature), but once I click the check mark to confirm the change and the item is changed (which is reflected in the binding logs), then the item cannot be changed anymore: the UI does not allow me to change the value.
Only by refreshing the page in the browser (F5) I can then change the value again.
See a small video of the current wrong behavior below:
numberitem
(shown in the video: after the first change the item cannot be changed anymore until page refresh).

Tested with Chrome v68 and Firefox.
No error is reported on the Chrome developer console.

The problem is present in both openHAB 2.3 and 2.4.

The item is attached to the the following channel:

    <channel-type id="setpointTemperature">
        <item-type>Number:Temperature</item-type>
        <label>Setpoint Temperature</label>
        <description>The setpoint temperature</description>
        <category>Temperature</category>
        <tags>
            <tag>TargetTemperature</tag>
        </tags>
        <state pattern="%.1f %unit%" step="0.5" />
    </channel-type>
@kaikreuzer
Copy link
Contributor

I can confirm this, I have experienced the same.
@htreu Is there any easy fix for this?

@htreu
Copy link
Contributor

htreu commented Aug 20, 2018

since I'm about to refactor the control view in Paper UI I put this on the list of issues to fix along the way. stay tuned.

@mvalla
Copy link
Author

mvalla commented Aug 25, 2018

same issue appears even if no state is used in the channel-type definition

htreu pushed a commit to htreu/smarthome that referenced this issue Aug 27, 2018
…onents

This also addresses:
* eclipse-archived#6074
* eclipse-archived#6076

Signed-off-by: Henning Treu <henning.treu@telekom.de>
kaikreuzer pushed a commit that referenced this issue Aug 30, 2018
#6131)

* Paper UI: refactor single control HTML & controllers to separate components

This also addresses:
* #6074
* #6076

Signed-off-by: Henning Treu <henning.treu@telekom.de>
@htreu
Copy link
Contributor

htreu commented Sep 5, 2018

This should be fixed with #6131.

@htreu htreu closed this as completed Sep 5, 2018
@mvalla
Copy link
Author

mvalla commented Sep 5, 2018

it is not fixed for me. I just updated to origin/master and can report same wrong behavior in PaperUI

@htreu
Copy link
Contributor

htreu commented Sep 6, 2018

I will check again, thanks for the feedback.

@htreu htreu reopened this Sep 6, 2018
clinique pushed a commit to clinique/smarthome that referenced this issue Sep 17, 2018
eclipse-archived#6131)

* Paper UI: refactor single control HTML & controllers to separate components

This also addresses:
* eclipse-archived#6074
* eclipse-archived#6076

Signed-off-by: Henning Treu <henning.treu@telekom.de>
@mvalla
Copy link
Author

mvalla commented Sep 22, 2018

it's ok now (the updated code was not being used in my Eclipse env)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants