Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Time picker in display profiles writes 0 instead of 00:00 to the hidden field which breaks Display edit form #994

Closed
PeterMis opened this issue Jan 3, 2017 · 4 comments
Assignees
Labels
Milestone

Comments

@PeterMis
Copy link
Member

PeterMis commented Jan 3, 2017

On a fresh 1.8.0-rc2 the hidden fields for update/download window have value of 0 instead of 00:00, which results in Data missing error on Display edit form.

upload

If you change the time to something else than default 00:00 and save, then change it back to 00:00 it's working correctly though.

@PeterMis PeterMis added the bug label Jan 3, 2017
@PeterMis PeterMis added this to the 1.8.0-rc3 milestone Jan 3, 2017
maurofmferrao added a commit to maurofmferrao/xibo-cms that referenced this issue Jan 6, 2017
@dasgarner dasgarner reopened this Jan 27, 2017
@dasgarner
Copy link
Member

This also needs to be corrected for the windows display profile.

@dasgarner
Copy link
Member

After finding the root cause in the JS control, we could also protect against error when it does have 0 by editing /lib/Controller/Display.php L#615 to:

$profile[$i]['valueString'] = ($profile[$i]['value'] == null || $profile[$i]['value'] == '0' ) ? '00:00' : $this->getDate()->parse($profile[$i]['value'], 'H:i')->format('H:i');

@telemanagain
Copy link

Hello all, first let me tell you that I have had a ball the last few days wrapping my brain around xibo, I'm currently running a docker as recommended and I am SO DANG CLOSE to having this up and running. It comes down to not being able to edit a newly connected android device. I have changes the times on Download Window Start Time and Download Window End Time then changed them back. No luck am I not changing things in the right place? forgive me if its probably something stupid I'm an old admin who's been out of the game for almost 6 years. Thats a lifetime in tech years.

@PeterMis
Copy link
Member Author

PeterMis commented Feb 6, 2017

For future reference, please discuss such issues on https://community.xibo.org.uk/

As you can see this issue here is not yet fixed, but the work around to edit display profile assigned to your device -> change download/update windows to lets say 00:15 | 23:55 -> save -> (you should be able to edit display now) -> edit profile again -> change to 00:00 | 00:00 -> save -> edit display should still work.

If not, please create a topic on Xibo Community site and we can discuss this further.

maurofmferrao added a commit to maurofmferrao/xibo-cms that referenced this issue Feb 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants