Revision 3685018f
Von Bernd Bleßmann vor mehr als 3 Jahren hinzugefügt
| SL/BackgroundJob/ConvertTimeRecordings.pm | ||
|---|---|---|
| 21 | 21 |
); |
| 22 | 22 |
|
| 23 | 23 |
# valid parameters -> better as class members with rose generic set/get |
| 24 |
my %params = ( |
|
| 25 |
fromdate => '', |
|
| 26 |
todate => '', |
|
| 24 |
my %valid_params = (
|
|
| 25 |
from_date => '',
|
|
| 26 |
to_date => '',
|
|
| 27 | 27 |
customernumbers => '', |
| 28 | 28 |
part_id => '', |
| 29 | 29 |
rounding => 1, |
| ... | ... | |
| 48 | 48 |
|
| 49 | 49 |
# check user input param names |
| 50 | 50 |
foreach my $param (keys %{ $data }) {
|
| 51 |
croak "Not a valid key: $param" unless $params{$param};
|
|
| 51 |
die "Not a valid parameter: $param" unless exists $valid_params{$param};
|
|
| 52 | 52 |
} |
| 53 | 53 |
|
| 54 | 54 |
# TODO check user input param values - (defaults are assigned later) |
Auch abrufbar als: Unified diff
Zeiterfassung: richtige Parameter f. Hintergrund-Job prüfen …
und
- Hash mit validen Parametern umbenannt
- exists statt trueish
- "die" statt "croak"