Bug 20392 - Multiple biblio holding does not work when patron is not set
Summary: Multiple biblio holding does not work when patron is not set
Status: RESOLVED DUPLICATE of bug 18474
Alias: None
Product: Koha
Classification: Unclassified
Component: Hold requests (show other bugs)
Version: master
Hardware: All All
: P5 - low normal (vote)
Assignee: Josef Moravec
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-03-13 20:02 UTC by Josef Moravec
Modified: 2018-04-02 19:25 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 20392: Fix handling multi hold when not patron set (4.65 KB, patch)
2018-03-13 20:09 UTC, Josef Moravec
Details | Diff | Splinter Review
Bug 20392: Fix handling multi hold when not patron set (4.70 KB, patch)
2018-03-14 13:04 UTC, Biblibre Sandboxes
Details | Diff | Splinter Review
Bug 20392: Fix handling multi hold when not patron set (4.75 KB, patch)
2018-03-16 19:12 UTC, Maksim Sen
Details | Diff | Splinter Review
Bug 20392: Fix handling multi hold when not patron set (4.83 KB, patch)
2018-03-30 11:15 UTC, Kyle M Hall
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Josef Moravec 2018-03-13 20:02:57 UTC
To recreate:

1) Set the system preference DisplayMultiPlaceHold to enable
2) Search in staff client catalogue
3) On results page check more then one result and click on Place hold button at the top of the page
4) Search for patron, but don't be to specific, you need to have more then one result
5) Click on one of the found patron
6) Now you are going to confirm a hold, but only for one record!
Comment 1 Josef Moravec 2018-03-13 20:09:27 UTC
Created attachment 72814 [details] [review]
Bug 20392: Fix handling multi hold when not patron set

Test plan:
0) Do not apply the patch
1) Set the system preference DisplayMultiPlaceHold to enable
2) Search in staff client catalogue
3) On results page check more then one result and click on Place hold
button at the top of the page
4) Search for patron, but don't be to specific, you need to have more
then one result
5) Click on one of the found patron
6) Now you are going to confirm a hold, but only for one record!
7) Apply the patch
8) Repeat 2-5 and confirm that now it is working as expected ( place
    hold for multiple records)
Comment 2 Biblibre Sandboxes 2018-03-14 13:04:11 UTC
Patch tested with a sandbox, by delaye <stephane.delaye@biblibre.com>
Comment 3 Biblibre Sandboxes 2018-03-14 13:04:32 UTC
Created attachment 72856 [details] [review]
Bug 20392: Fix handling multi hold when not patron set

Test plan:
0) Do not apply the patch
1) Set the system preference DisplayMultiPlaceHold to enable
2) Search in staff client catalogue
3) On results page check more then one result and click on Place hold
button at the top of the page
4) Search for patron, but don't be to specific, you need to have more
then one result
5) Click on one of the found patron
6) Now you are going to confirm a hold, but only for one record!
7) Apply the patch
8) Repeat 2-5 and confirm that now it is working as expected ( place
    hold for multiple records)

Signed-off-by: delaye <stephane.delaye@biblibre.com>
Comment 4 Maksim Sen 2018-03-16 19:12:01 UTC
Created attachment 73060 [details] [review]
Bug 20392: Fix handling multi hold when not patron set

Test plan:
0) Do not apply the patch
1) Set the system preference DisplayMultiPlaceHold to enable
2) Search in staff client catalogue
3) On results page check more then one result and click on Place hold
button at the top of the page
4) Search for patron, but don't be to specific, you need to have more
then one result
5) Click on one of the found patron
6) Now you are going to confirm a hold, but only for one record!
7) Apply the patch
8) Repeat 2-5 and confirm that now it is working as expected ( place
    hold for multiple records)

Signed-off-by: delaye <stephane.delaye@biblibre.com>
Signed-off-by: Maksim Sen <maksim.sen@inlibro.com>
Comment 5 Kyle M Hall 2018-03-30 11:15:45 UTC
Created attachment 73466 [details] [review]
Bug 20392: Fix handling multi hold when not patron set

Test plan:
0) Do not apply the patch
1) Set the system preference DisplayMultiPlaceHold to enable
2) Search in staff client catalogue
3) On results page check more then one result and click on Place hold
button at the top of the page
4) Search for patron, but don't be to specific, you need to have more
then one result
5) Click on one of the found patron
6) Now you are going to confirm a hold, but only for one record!
7) Apply the patch
8) Repeat 2-5 and confirm that now it is working as expected ( place
    hold for multiple records)

Signed-off-by: delaye <stephane.delaye@biblibre.com>
Signed-off-by: Maksim Sen <maksim.sen@inlibro.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 6 Kyle M Hall 2018-03-30 11:17:09 UTC
Bug 18474 and Bug 20392 appear to solve the same problem. Leaving to RM to decide which to use!
Comment 7 Jonathan Druart 2018-04-02 19:25:48 UTC

*** This bug has been marked as a duplicate of bug 18474 ***