[Bug 30697] New: IsAvailableForItemLevelRequest returns wrong when items_any_available is defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30697 Bug ID: 30697 Summary: IsAvailableForItemLevelRequest returns wrong when items_any_available is defined Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: trivial Priority: P5 - low Component: Hold requests Assignee: koha-bugs@lists.koha-community.org Reporter: shi-yao.wang@inLibro.com QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com The IsAvailableForItemLevelRequest subroutine returns wrong value when $items_any_available is defined -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30697 --- Comment #1 from Shi Yao Wang <shi-yao.wang@inLibro.com> --- Created attachment 134661 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=134661&action=edit Bug 30697: Fix return value when $items_any_available is defined Return the value of $item_any_available when it is defined instead of the opposite value. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30697 Shi Yao Wang <shi-yao.wang@inLibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30697 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |arthur.suzuki@biblibre.com --- Comment #2 from Arthur Suzuki <arthur.suzuki@biblibre.com> --- missing automated tests, and a test plan demonstrating the error and the fix. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30697 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com Assignee|koha-bugs@lists.koha-commun |shi-yao.wang@inLibro.com |ity.org | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30697 Shi Yao Wang <shi-yao.wang@inLibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #134661|0 |1 is obsolete| | --- Comment #3 from Shi Yao Wang <shi-yao.wang@inLibro.com> --- Created attachment 134666 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=134666&action=edit Bug 30697: Fix return value when $items_any_available is defined Return the value of $item_any_available when it is defined instead of the opposite value. Here is a test plan to reproduce an issue created by this bug Test plan: 1- Go to administration > system preferences > AllowRenewalIfOtherItemsAvailable and select "Allow" 2- Go to administration > circulation and fine rules and create an All/All rule where "on shelf holds allowed" is set to "if all unavailable" 3- Have at least 2 library branches Go to administration > libraries > new library to create them 4- Have 2 users, one from each library Go to patrons > new patron to create them 5- Have 2 items of the same record, but one item from each library Go to catalog > create or find a record > new > new item to create them 6- Make the first user check out the first item 7- Make the second user place a hold on the same record (even though it is not allowd) 8- Go back to the first user's page and try to renew 9- Notice it is written "on hold" under "Renew" even though the first user should be able to renew his item 10- Apply the patch 11- Refresh and notice it now shows the usual checkbox to renew -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30697 --- Comment #4 from Arthur Suzuki <arthur.suzuki@biblibre.com> --- Thank you for the test case! Still missing automated tests though... Have a look at the file t/db_dependant/Reserves.t You may find some help in writing unit_tests in this bug too (actually, your patch may solve my problem!) : https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20985 Do you know how to run unit tests? If you use Koha-testing-docker (ktd) this is very easy : when in Koha source dir, within ktd (after kshell on the host), you can run this command : prove t/db_dependant/Reserves.t Best, -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30697 Shi Yao Wang <shi-yao.wang@inLibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #134666|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30697 Shi Yao Wang <shi-yao.wang@inLibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30697 Shi Yao Wang <shi-yao.wang@inLibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |caroline.cyr-la-rose@inlibr | |o.com, | |philippe.blouin@inlibro.com | |, shi-yao.wang@inLibro.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30697 Shi Yao Wang <shi-yao.wang@inLibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|IsAvailableForItemLevelRequ |Cannot renew when record is |est returns wrong when |on hold under specific |items_any_available is |conditions |defined | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30697 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution|--- |WONTFIX Assignee|shi-yao.wang@inLibro.com |koha-bugs@lists.koha-commun | |ity.org --- Comment #5 from Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> --- No longer working on this. Using AuthorityXSLTResultsDisplay solved the client's problem. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30697 --- Comment #6 from Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> --- (In reply to Caroline Cyr La Rose from comment #5)
No longer working on this. Using AuthorityXSLTResultsDisplay solved the client's problem.
I meant AllowRenewalIfOtherItemsAvailable... little copy-paste problem, lol -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org