[Koha-bugs] [Bug 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Mason James changed: What|Removed |Added CC||m...@kohaaloha.com --- Comment #20 from Mason James --- (In reply to Kyle M Hall from comment #18) > Pushed to master for Koha 16.11! Blocked by Enhancement (bz14757), skipping for 16.05.x -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Fridolin SOMERS changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m Status|Pushed to Master|RESOLVED Resolution|--- |FIXED --- Comment #21 from Fridolin SOMERS --- (In reply to Mason James from comment #20) > (In reply to Kyle M Hall from comment #18) > > Pushed to master for Koha 16.11! > > Blocked by Enhancement (bz14757), skipping for 16.05.x Resolved fixed then -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Kyle M Hall changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |k...@bywatersolutions.com |ity.org | -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Kyle M Hall changed: What|Removed |Added Depends on||14757 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14757 [Bug 14757] Allow the use of Template Toolkit syntax for slips and notices -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 --- Comment #1 from Kyle M Hall --- Created attachment 53497 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53497&action=edit Bug 16942 - Confirm hold results in ugly error Confirming a hold to set it to waiting will result in an DBIC error in master. Test Plan: 1) Attempt to check in an item on hold and confirm the hold 2) Note the error 3) Apply this patch 4) Repeat step 1 5) Note there is no 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Kyle M Hall changed: What|Removed |Added Status|NEW |Needs Signoff -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Kyle M Hall changed: What|Removed |Added CC||neng...@gmail.com --- Comment #2 from Kyle M Hall --- *** Bug 16904 has been marked as a duplicate of this bug. *** -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Mirko Tietgen changed: What|Removed |Added CC||mi...@abunchofthings.net 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Mirko Tietgen changed: What|Removed |Added Attachment #53497|0 |1 is obsolete|| --- Comment #3 from Mirko Tietgen --- Created attachment 53499 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53499&action=edit Bug 16942 - Confirm hold results in ugly error Confirming a hold to set it to waiting will result in an DBIC error in master. Test Plan: 1) Attempt to check in an item on hold and confirm the hold 2) Note the error 3) Apply this patch 4) Repeat step 1 5) Note there is no error! Signed-off-by: Mirko Tietgen https://bugs.koha-community.org/show_bug.cgi?id=14942 -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Jonathan Druart changed: What|Removed |Added Status|Signed Off |Failed QA CC||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #4 from Jonathan Druart --- Kyle, the following test should fail without this patch: is( $prepared_letter->{content}, $hold->id(), 'Hold object used correctly' ); Could you please adapt 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Marc Véron changed: What|Removed |Added CC||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/
[Koha-bugs] [Bug 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Jonathan Druart changed: What|Removed |Added Blocks||11360 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11360 [Bug 11360] Disable barcode field and submit button when a hold is found -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 --- Comment #5 from Kyle M Hall --- Created attachment 53726 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53726&action=edit Bug 16942 [QA Followup] - Add unit test -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Kyle M Hall changed: 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Passed QA Patch complexity|--- |Small 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Katrin Fischer changed: What|Removed |Added Attachment #53499|0 |1 is obsolete|| Attachment #53726|0 |1 is obsolete|| --- Comment #6 from Katrin Fischer --- Created attachment 53737 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53737&action=edit [PASSED QA] Bug 16942 - Confirm hold results in ugly error Confirming a hold to set it to waiting will result in an DBIC error in master. Test Plan: 1) Attempt to check in an item on hold and confirm the hold 2) Note the error 3) Apply this patch 4) Repeat step 1 5) Note there is no error! Signed-off-by: Mirko Tietgen https://bugs.koha-community.org/show_bug.cgi?id=14942 Signed-off-by: Katrin Fischer -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 --- Comment #7 from Katrin Fischer --- Created attachment 53738 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=53738&action=edit [PASSED QA] Bug 16942 [QA Followup] - Add unit test Signed-off-by: Katrin Fischer -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Failed QA --- Comment #8 from Jonathan Druart --- (In reply to Jonathan Druart from comment #4) > Kyle, the following test should fail without this patch: > is( $prepared_letter->{content}, $hold->id(), 'Hold object used correctly' > ); > > Could you please adapt it? The test still passes. -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 --- Comment #9 from Jonathan Druart --- *** Bug 17115 has been marked as a duplicate of this bug. *** -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Marc Véron changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=17101 -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Kyle M Hall changed: What|Removed |Added Blocks||14695 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14695 [Bug 14695] Add ability to place multiple item holds on a given record per patron -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Kyle M Hall changed: What|Removed |Added Status|Failed QA |Passed QA Blocks|14695 | --- Comment #10 from Kyle M Hall --- (In reply to Jonathan Druart from comment #8) > (In reply to Jonathan Druart from comment #4) > > Kyle, the following test should fail without this patch: > > is( $prepared_letter->{content}, $hold->id(), 'Hold object used correctly' > > ); > > > > Could you please adapt it? > > The test still passes. They unit tests fail for me without the initial patch applied. prove t/db_dependent/Letters/TemplateToolkit.t t/db_dependent/Letters/TemplateToolkit.t .. 1/14 DBD::mysql::st execute failed: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'BIBLIONUMBER '557' AND `ARRAY(0x8818f68)` BORROWERNUMBER '140' ) AND `ARRAY(0x88' at line 1 [for Statement "SELECT `me`.`reserve_id`, `me`.`borrowernumber`, `me`.`reservedate`, `me`.`biblionumber`, `me`.`branchcode`, `me`.`notificationdate`, `me`.`reminderdate`, `me`.`cancellationdate`, `me`.`reservenotes`, `me`.`priority`, `me`.`found`, `me`.`timestamp`, `me`.`itemnumber`, `me`.`waitingdate`, `me`.`expirationdate`, `me`.`lowestPriority`, `me`.`suspend`, `me`.`suspend_until`, `me`.`itemtype` FROM `reserves` `me` WHERE ( ( ( `ARRAY(0x8818f68)` BIBLIONUMBER ? AND `ARRAY(0x8818f68)` BORROWERNUMBER ? ) AND `ARRAY(0x8818f68)` RESERVE_ID ? ) )" with ParamValues: 0='557', 1='140', 2='114'] at /usr/share/perl5/DBIx/Class/Storage/DBI.pm line 1832. DBD::mysql::st execute failed: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'BIBLIONUMBER '557' AND `ARRAY(0x8818f68)` BORROWERNUMBER '140' ) AND `ARRAY(0x88' at line 1 [for Statement "SELECT `me`.`reserve_id`, `me`.`borrowernumber`, `me`.`reservedate`, `me`.`biblionumber`, `me`.`branchcode`, `me`.`notificationdate`, `me`.`reminderdate`, `me`.`cancellationdate`, `me`.`reservenotes`, `me`.`priority`, `me`.`found`, `me`.`timestamp`, `me`.`itemnumber`, `me`.`waitingdate`, `me`.`expirationdate`, `me`.`lowestPriority`, `me`.`suspend`, `me`.`suspend_until`, `me`.`itemtype` FROM `reserves` `me` WHERE ( ( ( `ARRAY(0x8818f68)` BIBLIONUMBER ? AND `ARRAY(0x8818f68)` BORROWERNUMBER ? ) AND `ARRAY(0x8818f68)` RESERVE_ID ? ) )" with ParamValues: 0='557', 1='140', 2='114'] at /usr/share/perl5/DBIx/Class/Storage/DBI.pm line 1832. # Looks like your test exited with 255 just after 8. t/db_dependent/Letters/TemplateToolkit.t .. Dubious, test returned 255 (wstat 65280, 0xff00) Failed 6/14 subtests Test Summary Report --- t/db_dependent/Letters/TemplateToolkit.t (Wstat: 65280 Tests: 8 Failed: 0) Non-zero exit status: 255 Parse errors: Bad plan. You planned 14 tests but ran 8. Files=1, Tests=8, 3 wallclock secs ( 0.02 usr 0.01 sys + 1.74 cusr 0.18 csys = 1.95 CPU) Result: FAIL Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14695 [Bug 14695] Add ability to place multiple item holds on a given record per patron -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 --- Comment #11 from Jonathan Druart --- (In reply to Kyle M Hall from comment #10) > (In reply to Jonathan Druart from comment #8) > > (In reply to Jonathan Druart from comment #4) > > > Kyle, the following test should fail without this patch: > > > is( $prepared_letter->{content}, $hold->id(), 'Hold object used > > > correctly' > > > ); > > > > > > Could you please adapt it? > > > > The test still passes. > > They unit tests fail for me without the initial patch applied. % git fetch origin % git reset --hard origin/master % git clean -f -d % prove t/db_dependent/Letters/TemplateToolkit.t t/db_dependent/Letters/TemplateToolkit.t .. ok All tests successful. Files=1, Tests=13, 4 wallclock secs ( 0.06 usr 0.00 sys + 2.78 cusr 0.10 csys = 2.94 CPU) Result: PASS What I meant is that the test 'Hold object used correctly' successes when we are expecting it to fail. -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Jesse Maseto changed: What|Removed |Added Attachment #53737|0 |1 is obsolete|| Attachment #53738|0 |1 is obsolete|| --- Comment #12 from Jesse Maseto --- Created attachment 54569 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=54569&action=edit Bug 16942 - Confirm hold results in ugly error Confirming a hold to set it to waiting will result in an DBIC error in master. Test Plan: 1) Attempt to check in an item on hold and confirm the hold 2) Note the error 3) Apply this patch 4) Repeat step 1 5) Note there is no error! Signed-off-by: Mirko Tietgen https://bugs.koha-community.org/show_bug.cgi?id=14942 Signed-off-by: Katrin Fischer Signed-off-by: Jesse Maseto -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 --- Comment #13 from Jesse Maseto --- Created attachment 54570 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=54570&action=edit Bug 16942 [QA Followup] - Add unit test Signed-off-by: Katrin Fischer Signed-off-by: Jesse Maseto -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 --- Comment #14 from Kyle M Hall --- > % git fetch origin > % git reset --hard origin/master > % git clean -f -d > % prove t/db_dependent/Letters/TemplateToolkit.t > t/db_dependent/Letters/TemplateToolkit.t .. ok > All tests successful. > Files=1, Tests=13, 4 wallclock secs ( 0.06 usr 0.00 sys + 2.78 cusr 0.10 > csys = 2.94 CPU) > Result: PASS > > What I meant is that the test 'Hold object used correctly' successes when we > are expecting it to fail. Are you applying the "Bug 16942 [QA Followup] - Add unit test" patch to master before running the tests? If you apply that patch, the test should fail. Then, if you apply "Bug 16942 - Confirm hold results in ugly error" the test should pass. -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 --- Comment #15 from Jonathan Druart --- (In reply to Kyle M Hall from comment #14) > > % git fetch origin > > % git reset --hard origin/master > > % git clean -f -d > > % prove t/db_dependent/Letters/TemplateToolkit.t > > t/db_dependent/Letters/TemplateToolkit.t .. ok > > All tests successful. > > Files=1, Tests=13, 4 wallclock secs ( 0.06 usr 0.00 sys + 2.78 cusr 0.10 > > csys = 2.94 CPU) > > Result: PASS > > > > What I meant is that the test 'Hold object used correctly' successes when we > > are expecting it to fail. > > Are you applying the "Bug 16942 [QA Followup] - Add unit test" patch to > master before running the tests? If you apply that patch, the test should > fail. Then, if you apply "Bug 16942 - Confirm hold results in ugly error" > the test should pass. I completely understood that. So maybe you should explain me why the following test pass: 180 $prepared_letter = GetPreparedLetter( 181 ( 182 module => 'test', 183 letter_code => 'TEST_HOLD', 184 tables => { 185 reserves => [ $patron->{borrowernumber}, $biblio->id() ] 186 }, 187 ) 188 ); 189 is( $prepared_letter->{content}, $hold->id(), 'Hold object used correctly' ) What does reserves => { $borrowernumber, $biblionumber } mean? How do you know that $borrowernumber is a borrowernumber? It looks like something magic and to me it should not work. -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Jacek Ablewicz changed: What|Removed |Added CC||a...@biblos.pk.edu.pl -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 --- Comment #16 from Kyle M Hall --- > So maybe you should explain me why the following test pass: > > 180 $prepared_letter = GetPreparedLetter( > 181 ( > 182 module => 'test', > 183 letter_code => 'TEST_HOLD', > 184 tables => { > 185 reserves => [ $patron->{borrowernumber}, $biblio->id() ] > 186 }, > 187 ) > 188 ); > 189 is( $prepared_letter->{content}, $hold->id(), 'Hold object used > correctly' ) > > What does reserves => { $borrowernumber, $biblionumber } mean? > How do you know that $borrowernumber is a borrowernumber? > It looks like something magic and to me it should not work. Ah, I see now! I'll look into that! -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 --- Comment #17 from Kyle M Hall --- I looked into this and it's working exactly as intended! Basically, the tables hashref is a list of key/value pairs. The value can either be a) a hashref or b) an arrayref. If it is a hashref, the keys are pulled out as would be expected, based on the key name. For the arrayref, they simply need to be ordered correctly. In the case of Holds, the lookup key is the fk combo borrowernumber/biblionumber, in that order. (In reply to Kyle M Hall from comment #16) > > So maybe you should explain me why the following test pass: > > > > 180 $prepared_letter = GetPreparedLetter( > > 181 ( > > 182 module => 'test', > > 183 letter_code => 'TEST_HOLD', > > 184 tables => { > > 185 reserves => [ $patron->{borrowernumber}, $biblio->id() ] > > 186 }, > > 187 ) > > 188 ); > > 189 is( $prepared_letter->{content}, $hold->id(), 'Hold object used > > correctly' ) > > > > What does reserves => { $borrowernumber, $biblionumber } mean? > > How do you know that $borrowernumber is a borrowernumber? > > It looks like something magic and to me it should not work. > > Ah, I see now! I'll look into that! -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Kyle M Hall changed: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #18 from Kyle M Hall --- Pushed to master for Koha 16.11! -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Jonathan Druart changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=17246 --- Comment #19 from Jonathan Druart --- (In reply to Kyle M Hall from comment #17) > I looked into this and it's working exactly as intended! Basically, the > tables hashref is a list of key/value pairs. The value can either be a) a > hashref or b) an arrayref. If it is a hashref, the keys are pulled out as > would be expected, based on the key name. For the arrayref, they simply need > to be ordered correctly. In the case of Holds, the lookup key is the fk > combo borrowernumber/biblionumber, in that order. I'd suggest to remove this behavior and only supports hashref, would you agree? See bug 17246. -- 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 16942] Confirm hold results in ugly error
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16942 Nicole C. Engard changed: What|Removed |Added CC|neng...@gmail.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/