[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Katrin Fischerchanged: What|Removed |Added CC||katrin.fisc...@bsz-bw.de Status|Pushed to Master|Pushed to Stable --- Comment #19 from Katrin Fischer --- This patch has been pushed to 16.11.x and will be in 16.11.09. -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Kyle M Hallchanged: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #18 from Kyle M Hall --- Pushed to master for 17.05, thanks Marcel, Jonathan! -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 --- Comment #17 from Marcel de Rooy--- (In reply to Jonathan Druart from comment #16) > I was expecting someone else to QA it. Well, let Kyle have a last look now.. -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 --- Comment #16 from Jonathan Druart--- I was expecting someone else to QA it. -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Marcel de Rooychanged: What|Removed |Added Status|Signed Off |Passed QA --- Comment #15 from Marcel de Rooy --- (In reply to Jonathan Druart from comment #14) > Signed-off-by: Jonathan Druart Thanks. Changing status. -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Mason Jameschanged: What|Removed |Added CC||m...@kohaaloha.com -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Jonathan Druartchanged: What|Removed |Added Assignee|k...@bywatersolutions.com |m.de.r...@rijksmuseum.nl -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Jonathan Druartchanged: What|Removed |Added Attachment #63651|0 |1 is obsolete|| --- Comment #14 from Jonathan Druart --- Created attachment 63663 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63663=edit Bug 18647: Resolve internal server error on category_type See bug 18552. When we resolved the housebound_role bug, the hash got filled correctly again. And this revealed that the (second) call to Koha::Patrons->find was not appropriate. It can be removed, as Jonathan explained on the report. Note: Commit 95429af6855e8ed37945b681dc466bf360deffb added this call, but it was hidden until the template variable hash got fixed. Test plan: Restart Plack and go to patron details again. Signed-off-by: Marcel de Rooy Signed-off-by: Jonathan Druart -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 --- Comment #13 from Marcel de Rooy--- >From IRC: Joubu marcelr: 290 $template->param(%$data); [14:51] Joubu patron-title.inc need patron's info, there are either picked from borrower.$field, or $field [14:51] Joubu in moremember we have $field, because of l.290 [14:51] Joubu we just need to reset the state to the one before the patron's club ft [14:51] Joubu and so delete the line -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Marcel de Rooychanged: What|Removed |Added Attachment #63648|0 |1 is obsolete|| --- Comment #12 from Marcel de Rooy --- Created attachment 63651 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63651=edit Bug 18647: Resolve internal server error on category_type See bug 18552. When we resolved the housebound_role bug, the hash got filled correctly again. And this revealed that the (second) call to Koha::Patrons->find was not appropriate. It can be removed, as Jonathan explained on the report. Note: Commit 95429af6855e8ed37945b681dc466bf360deffb added this call, but it was hidden until the template variable hash got fixed. Test plan: Restart Plack and go to patron details again. Signed-off-by: Marcel de Rooy -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 --- Comment #11 from Marcel de Rooy--- (In reply to Marcel de Rooy from comment #10) > (In reply to Jonathan Druart from comment #5) > > I'd say the patch should be: > > > > diff --git a/members/moremember.pl b/members/moremember.pl > No, you cannot remove the borrower template var yet. Yes, we can remove it. Probably the same Plack problem.. -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 --- Comment #10 from Marcel de Rooy--- (In reply to Jonathan Druart from comment #5) > I'd say the patch should be: > > diff --git a/members/moremember.pl b/members/moremember.pl > index 0e18d2234f..0649dcfd76 100755 > --- a/members/moremember.pl > +++ b/members/moremember.pl > @@ -343,7 +343,6 @@ my $translated_language = > C4::Languages::language_get_description( $subtag, $sub > > $template->param( > patron => $patron, > -borrower=> $patron, # TODO Still needed by includes, > translated_language => $translated_language, > detailview => 1, > borrowernumber => $borrowernumber, > > > The borrower variable should not be a Koha::Patron, because the include file > expects an hashref. > > Marcel, do you know why you add the TODO comment? I do not find where it is > used. No, you cannot remove the borrower template var yet. The TODO was kind of trivial. Passing the same information twice needs further attention :) Probably by forgetting another Plack restart, I did not catch the internal server error.. -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Marcel de Rooychanged: What|Removed |Added Attachment #63596|0 |1 is obsolete|| --- Comment #9 from Marcel de Rooy --- Created attachment 63648 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63648=edit Bug 18647: Resolve internal server error on category_type See bug 18552. When we resolved the housebound_role bug, the hash got filled correctly again. And this revealed that the (second) call to Koha::Patrons->find was not appropriate. We cannot yet remove the borrower template variable (reason for the TODO). So we should replace it by a call to GetMember. Note: Commit 95429af6855e8ed37945b681dc466bf360deffb added this call, but it was hidden until the template variable hash got fixed. Test plan: Restart Plack and go to patron details again. Signed-off-by: Marcel de Rooy -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Marcel de Rooychanged: What|Removed |Added Status|Failed QA |Signed Off -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Marcel de Rooychanged: What|Removed |Added Depends on||18552 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18552 [Bug 18552] Borrower debarments do not show on member detail page -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 --- Comment #8 from Marcel de Rooy--- (In reply to Tomás Cohen Arazi from comment #7) > (In reply to Marc Véron from comment #6) > > (In reply to Jonathan Druart from comment #3) > We still need Marcel's comment on why he put the TODO, because we might as > well miss some other edge case Marcel noticed. Looking here in a bit -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Tomás Cohen Arazichanged: What|Removed |Added CC||tomasco...@gmail.com --- Comment #7 from Tomás Cohen Arazi --- (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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 --- Comment #6 from Marc Véron--- (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. -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Jonathan Druartchanged: What|Removed |Added CC||k...@bywatersolutions.com, ||m.de.r...@rijksmuseum.nl --- Comment #5 from Jonathan Druart --- I'd say the patch should be: diff --git a/members/moremember.pl b/members/moremember.pl index 0e18d2234f..0649dcfd76 100755 --- a/members/moremember.pl +++ b/members/moremember.pl @@ -343,7 +343,6 @@ my $translated_language = C4::Languages::language_get_description( $subtag, $sub $template->param( patron => $patron, -borrower=> $patron, # TODO Still needed by includes, translated_language => $translated_language, detailview => 1, borrowernumber => $borrowernumber, The borrower variable should not be a Koha::Patron, because the include file expects an hashref. Marcel, do you know why you add the TODO comment? I do not find where it is used. -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Jonathan Druartchanged: What|Removed |Added Severity|critical|blocker -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Jonathan Druartchanged: What|Removed |Added Depends on||12461 Assignee|ve...@veron.ch |k...@bywatersolutions.com --- Comment #4 from Jonathan Druart --- This is caused by bug 12461. The borrower variable should NOT be a Koha::Patron object. Everything is cleared on bug 17829, but we will need to wait for the next release to see it in. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12461 [Bug 12461] Add patron clubs feature -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Jonathan Druartchanged: What|Removed |Added Status|Signed Off |Failed QA CC||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #3 from Jonathan Druart --- 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/) -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Josef Moravecchanged: What|Removed |Added Attachment #63590|0 |1 is obsolete|| --- Comment #2 from Josef Moravec --- Created attachment 63596 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63596=edit [SIGNED-OFF] Bug 18647: Internal server error on moremember.pl To reproduce: Go to the Details page of a member. Result: Internal server error Plack error log: Template process failed: undef error - The method category_code is not covered by tests! at (...)/koha/C4/Templates.pm line 121. To test: Apply patch Try to reproduce - patron's detail page should now display without error Signed-off-by: Josef Moravec -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Josef Moravecchanged: What|Removed |Added Status|Needs Signoff |Signed Off -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Marc Véronchanged: What|Removed |Added Status|NEW |Needs Signoff Patch complexity|--- |Trivial patch -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 --- Comment #1 from Marc Véron--- Created attachment 63590 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63590=edit Bug 18647: Internal server error on moremember.pl To reproduce: Go to the Details page of a member. Result: Internal server error Plack error log: Template process failed: undef error - The method category_code is not covered by tests! at (...)/koha/C4/Templates.pm line 121. To test: Apply patch Try to reproduce - patron's detail page should now display without error -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Marc Véronchanged: What|Removed |Added Version|unspecified |master -- 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/
[Koha-bugs] [Bug 18647] Internal server error on moremember.pl
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18647 Marc Véronchanged: What|Removed |Added Assignee|gmcha...@gmail.com |ve...@veron.ch -- 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/