https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29684

--- Comment #23 from Andrew Nugged <nug...@gmail.com> ---
I found that it looks that $borrower in /members/paycollect.pl:

    my $borrower       = $patron->unblessed;

used only once even in old code:

    $template->param(%$borrower);

and there all around "patron." used in paycollect.tt like:

    <input type="hidden" name="borrowernumber" id="borrowernumber"
     value="[% patron.borrowernumber | html %]" />

but for some reason it was only ONE place (two times) where it was:

    <input type="hidden" name="title" id="title" value="[% title | html %]" />

how hell it can be so? :) So complicated solution to bring $borrower unblessed
through a such number of code lines just to get ... title? :)

I assume these are some oooold times refactoring remnants, but - 

... so please mates check if my patch to paycollect part is the correct
solution
(and I see the strong hivemind we have here :) ).

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

Reply via email to