http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9809

Kyle M Hall <k...@bywatersolutions.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |k...@bywatersolutions.com

--- Comment #10 from Kyle M Hall <k...@bywatersolutions.com> ---
(In reply to Jonathan Druart from comment #9)
> Kyle,
> Thanks for these patches, it's a much more complete solution than the one I
> have proposed.
> 
> I have a doubt with the changes in GetReserveFee:
> Before the patches:
>     if ( $const eq "a" ) {
>         push @biblioitems, $data1;
>     }
>     else {
>         some code;
>     }
> 
> now you always execute some code.
> 
> But before the patch, $const was sometime equal to "a".
> I am not sure about the consequences of this change.

Can you add a splinter review? I'm not sure exactly where the code you are
referencing is. You can also add a followup to fix it if that's easier.

Thanks!

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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/

Reply via email to