Config does not save when Raspi integration disabled

t-oster / VisiCam

A Java application, that turns a webcam into a VisiCut compatible network camera with marker detection and perspective correction. Uses OpenCV through JavaCV

Other

36 stars 12 forks source link

Config does not save when Raspi integration disabled #24

Closed mgmax closed 7 years ago

mgmax commented 8 years ago

saving the settings fails for each of the following:

InactivitySeconds is empty
Raspi integration disabled

In this case, there is no error message visible, the save button just does nothing

suggested fixes

settings HTML page should display an error, not just skip the success message
VisiCam should ignore invalid values, or the JavaScript should disallow an empty value

mgmax commented 8 years ago

my commit 370f84f only partially solves this, one can still enable the raspi setting and set an empty value in InactivitySeconds

mgmax commented 7 years ago

fixed by #26