| commit | 5bd9482550cee62fde36c5a74d73714ec000dc0b | |
| author | Jonathan Druart <jonathan.druart@biblibre.com> | |
| Wed, 12 Nov 2014 11:23:38 +0000 (12 12:23 +0100) | ||
| committer | Tomas Cohen Arazi <tomascohen@gmail.com> | |
| Tue, 10 Feb 2015 17:07:59 +0000 (10 14:07 -0300) | ||
| tree | 83f61ec7c9c4a8bca6ebe425fe92dcd7e52d4598 | treesnapshot (tar.gz zip) |
| parent | 87bcc8b303e506514318814f63c430a3a54f1181 | commitdiff |
Bug 13240: Remove some code obfuscation
my $bar;
my $foo = $bar->{borrowernumber} ||= {};
$foo->{one} ||= 'something';
$foo->{two}++;
What does $bar contain?
$VAR1 = {
'borrowernumber' => {
'two' => 1,
'bar' => 'something'
}
};
Not really obvious.
Maybe something I did not see is hidden.
Test plan:
Verify the digest for DUE and PREDUE work as before.
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
my $bar;
my $foo = $bar->{borrowernumber} ||= {};
$foo->{one} ||= 'something';
$foo->{two}++;
What does $bar contain?
$VAR1 = {
'borrowernumber' => {
'two' => 1,
'bar' => 'something'
}
};
Not really obvious.
Maybe something I did not see is hidden.
Test plan:
Verify the digest for DUE and PREDUE work as before.
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
| misc/cronjobs/advance_notices.pl | diffblobblamehistory |