From 3898172d521c614fee832d70d1601e2acb44fab6 Mon Sep 17 00:00:00 2001 From: Lewis Juggins Date: Sun, 12 Nov 2017 15:37:22 +0000 Subject: [PATCH] Data validation (#109) --- pytradfri/device.py | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/pytradfri/device.py b/pytradfri/device.py index 1ce11c62..27807487 100644 --- a/pytradfri/device.py +++ b/pytradfri/device.py @@ -191,11 +191,8 @@ def set_dimmer(self, dimmer, *, index=0, transition_time=None): dimmer: Integer between 0..254 transition_time: Integer representing tenth of a second (default None) """ - # Set bounds for safety. - if dimmer < 0: - dimmer = 0 - elif dimmer > 254: - dimmer = 254 + if dimmer < 0 or dimmer > 254: + raise ValueError('Dimmer value must be between 0 and 254.') values = { ATTR_LIGHT_DIMMER: dimmer