https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27032
Jonathan Druart <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart+koha@gmail. | |com --- Comment #15 from Jonathan Druart <[email protected]> --- Joonas, don't you think we should have the same return than CanBookBeRenewed? ($boolean, $reason); The 'return "no";' seems weird to me. We could then replace: $auto_renew = _CanBookBeAutoRenewed($borrowernumber, $itemnumber); return ( 0, $auto_renew ) if $auto_renew =~ 'auto_account_expired'; return ( 0, $auto_renew ) if $auto_renew =~ 'auto_too_late'; return ( 0, $auto_renew ) if $auto_renew =~ 'auto_too_much_oweing'; with: my ( $can_be_auto_renewed, $reason ) = _CanBookBeAutoRenewed($borrowernumber, $itemnumber); return ( 0, $reason ) unless $can_be_auto_renewed; -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
