fhem / fhem/AutoShuttersControl

Fehlede Eingabe Validierung führt zu Warninungs

Open
#95 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Perl
Stars
9
Forks
7
PR merge metrics
No merged PRs in 30d

Description

Ich habe 0:20 - 20 Minuten - für BlockingTimeAfterManual hinterlegt.

Das führ zu einem Warning:
PERL WARNING: Argument "0:20" isn't numeric in numeric lt (<) at lib/FHEM/Automation/ShuttersControl/Helper.pm line 924.
In Version # $Id: Helper.pm 23934 2021-03-12 12:19:50Z CoolTux $

Ich nehme an, im Master branch entspricht das folgender Zeile:

https://github.com/fhem/AutoShuttersControl/blob/2012ec1fc303d7330dfa22f14353642fe1b1f35d/lib/FHEM/Automation/ShuttersControl/Helper.pm#L929

Offensichtlich ist nicht vogesehen, dass in dem Feld eine Stunden:Minuten Angabe getätigt wird, dennoch wird der Parameter aber beim Eingeben angenommen.
Ich würde eine Validierung der Parameter empfehlen, dann könnten zumindest keine Fehlerhaften eingegeben werden.

Aus Sicht der Lib, wäre es aber auch wünschenswert, dass es eine Verifizierung gibt,

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.