From b989a6ff158d5b506ded4ae04dfa5fb891086875 Mon Sep 17 00:00:00 2001 From: NiLuJe Date: Sat, 13 May 2023 20:45:47 +0200 Subject: [PATCH] NaturalLightWidget: Fallback to previous values instead of `nil` (#10425) Happens in case the InputText field is emptied by the user. The backend code makes rather strong assumptions that it'll *always* get a number of of it ;). Fix #10352 --- frontend/ui/widget/naturallightwidget.lua | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/frontend/ui/widget/naturallightwidget.lua b/frontend/ui/widget/naturallightwidget.lua index 397a5da64..c94947efd 100644 --- a/frontend/ui/widget/naturallightwidget.lua +++ b/frontend/ui/widget/naturallightwidget.lua @@ -115,6 +115,13 @@ function NaturalLightWidget:adaptableNumber(initial, step) table.insert(minus_number_plus, button_minus) table.insert(minus_number_plus, input_text) table.insert(minus_number_plus, button_plus) + + -- Sanitize the returned value so as not to upset sysfs_light... + function input_text:getText() + -- Also, while we're here, make sure we actually return a number, because InputText doesn't for... reasons. + return tonumber(self.text) or initial + end + return minus_number_plus end