fix: dbUpdate calls now check if it is 0 or above #4996

Merged
merged 1 commit into from Nov 16, 2016

Projects

None yet

4 participants

@laf
Member
laf commented Nov 15, 2016

Please note

Please read this information carefully. You can run ./scripts/pre-commit.php to check your code before submitting.

If the dbUpdate call works but doesn't change anything then 0 is returned which would have previously shown as a fail for these forms so just updating to show the correct message now.

@laf laf fix: dbUpdate calls now check if it is 0 or above
3fa8fde
@laf laf added Bug WebUI labels Nov 15, 2016
@scrutinizer-notifier

The inspection completed: No new issues

@murrant murrant merged commit 7061acf into librenms:master Nov 16, 2016

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@laf laf deleted the laf:forms-dbupdate branch Nov 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment