https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647
Tomás Cohen Arazi <tomasco...@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |tomasco...@gmail.com
--- Comment #7 from Tomás Cohen Arazi <tomasco...@gmail.com> ---
(In reply to Marc Véron from comment #6)
> (In reply to Jonathan Druart from comment #3)
> > No, it is definitely not the way to fix it, the borrower variable comes from
> > GetMember most of the time (from other script of members/)
>
> Sorry then to have tried to fix it. Leaving it to others to fix the issue.
Your patch clearly fixes the issue by making patron-title.inc aware of the fact
that borrower is now a K::Patron object. The problem is that the rest of the
code is written in terms of the hashref returned by GetMember and we could be
missing some edge cases. So we better move it back to the previous behaviour,
and let bug 17829 take care of the C4 > Koha shift.
We still need Marcel's comment on why he put the TODO, because we might as well
miss some other edge case Marcel noticed.
--
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/
