https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41940 --- Comment #4 from Roman Dolny <roman.dolny@jezuici.pl> --- (In reply to Baptiste Wojtkowski (bwoj) from comment #3)
my $patron_match = $borrowernumber == ( $patron_for_session // 0 ); my @conf_keys = split( /\|/, $session_confirmations // '' );
This line is correct but looks a bit unclear to me. Shouldn't we fix it with ternary syntax like this ?
my $patron_match = $patron_for_session && ($borrowernumber == $patron_for_session); my @conf_keys = $session_confirmations ? split( /\|/, $session_confirmations) : ();
Baptiste, thank you for checking the code. I'm not a programmer, so I don't see the difference. If you think it's necessary, or if someone else suggests it, I can certainly make the change. -- You are receiving this mail because: You are watching all bug changes.