[Koha-bugs] [Bug 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 Jared Camins-Esakov changed: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #16 from Jared Camins-Esakov --- This patch has been pushed to master and 3.12.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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 Katrin Fischer changed: What|Removed |Added Attachment #18048|0 |1 is obsolete|| --- Comment #15 from Katrin Fischer --- Created attachment 18123 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=18123&action=edit [PASSED QA] Bug 9575 - Serious Holds Problem with IndependantBranches If IndependantBranches is enabled, all holds for an item display the currently logged in library no matter what, and modifying the holds will set all pickup locations for all the holds to that logged in library! Test plan: 1) Disable IndependantBranches 2) Place 3 holds on a record with 3 different pickup locations 3) Enable IndependantBranches 4) Refresh the holds page for that record, all the holds show now show a pickup location of the logged in branch 5) Modify the order of the holds 6) Disable IndependantBranches 7) Refresh the holds, verify the holds all now have a pickup location of the logged in branch 8) Apply the patch 9) Repeat steps 1-3 10) Refresh the holds page for that record, all the holds should now show the correct pickup branch, but it cannot be changed as IndependantBranches is enabled. Signed-off-by: Kyle M Hall Signed-off-by: Owen Leonard Signed-off-by: Katrin Fischer In my tests the above test plan only worked out if my user had no superlibrarian permission. But then behaviour for independentbranches off/on was all correct. All tests and QA script 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 Katrin Fischer changed: What|Removed |Added Status|Failed QA |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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 --- Comment #14 from Katrin Fischer --- Ok, I think I have resolved it - the problem occurs not if your user is a superlibrarian. It does occur, when you don't have superlibrarian permissions. -- 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 --- Comment #13 from Katrin Fischer --- Hi Kyle and Owen, I skipped 5) - modifying the holds, because the pull downs were wrong and it seemed obvious that modifying would ruing the pickup location. But now I have problems reproducing the original problem with the wrong pull downs, I will try to investigate more later. -- 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 --- Comment #12 from Owen Leonard --- (In reply to comment #11) > But after applying the patch and > refreshing the page several times, this is still the case. After you applied the patch did you start with a new set of three holds with different pickup locations? -- 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Failed QA CC||katrin.fisc...@bsz-bw.de --- Comment #11 from Katrin Fischer --- Kyle, I am sorry, but this doesn't work for me. I created 3 different holds with 3 different pickup libraries away from my logged in library. I turned on independentbranches and like you described, suddenly all holds showed my current branch as pickup library. But after applying the patch and refreshing the page several times, this is still the case. -- 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 Owen Leonard changed: What|Removed |Added Attachment #16861|0 |1 is obsolete|| --- Comment #10 from Owen Leonard --- Created attachment 18048 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=18048&action=edit [SIGNED-OFF] Bug 9575 - Serious Holds Problem with IndependantBranches If IndependantBranches is enabled, all holds for an item display the currently logged in library no matter what, and modifying the holds will set all pickup locations for all the holds to that logged in library! Test plan: 1) Disable IndependantBranches 2) Place 3 holds on a record with 3 different pickup locations 3) Enable IndependantBranches 4) Refresh the holds page for that record, all the holds show now show a pickup location of the logged in branch 5) Modify the order of the holds 6) Disable IndependantBranches 7) Refresh the holds, verify the holds all now have a pickup location of the logged in branch 8) Apply the patch 9) Repeat steps 1-3 10) Refresh the holds page for that record, all the holds should now show the correct pickup branch, but it cannot be changed as IndependantBranches is enabled. Signed-off-by: Kyle M Hall Signed-off-by: Owen Leonard -- 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 Owen Leonard changed: 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 Kyle M Hall changed: What|Removed |Added Attachment #16860|0 |1 is obsolete|| --- Comment #9 from Kyle M Hall --- Created attachment 16861 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=16861&action=edit Bug 9575 - Serious Holds Problem with IndependantBranches Test plan: 1) Disable IndependantBranches 2) Place 3 holds on a record with 3 different pickup locations 3) Enable IndependantBranches 4) Refresh the holds page for that record, all the holds show now show a pickup location of the logged in branch 5) Modify the order of the holds 6) Disable IndependantBranches 7) Refresh the holds, verify the holds all now have a pickup location of the logged in branch 8) Apply the patch 9) Repeat steps 1-3 10) Refresh the holds page for that record, all the holds should now show the correct pickup branch, but it cannot be changed as IndependantBranches is enabled. Signed-off-by: Kyle M Hall -- 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 Kyle M Hall changed: What|Removed |Added Attachment #16058|0 |1 is obsolete|| Attachment #16073|0 |1 is obsolete|| --- Comment #8 from Kyle M Hall --- Created attachment 16860 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=16860&action=edit Bug 9575 - Serious Holds Problem with IndependantBranches Test plan: 1) Disable IndependantBranches 2) Place 3 holds on a record with 3 different pickup locations 3) Enable IndependantBranches 4) Refresh the holds page for that record, all the holds show now show a pickup location of the logged in branch 5) Modify the order of the holds 6) Disable IndependantBranches 7) Refresh the holds, verify the holds all now have a pickup location of the logged in branch 8) Apply the patch 9) Repeat steps 1-3 10) Refresh the holds page for that record, all the holds should now show the correct pickup branch, but it cannot be changed as IndependantBranches is enabled. Signed-off-by: Kyle M Hall -- 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 Kyle M Hall changed: What|Removed |Added Status|Failed QA |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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 Liz Rea changed: What|Removed |Added Status|Needs Signoff |Failed QA --- Comment #7 from Liz Rea --- Hi Kyle, I with the alternative patch I seem to get empty pick up location drop downs without independant branches on. I didn't go any further than that. Liz -- 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 Jonathan Druart changed: What|Removed |Added Status|Signed Off |Needs Signoff --- Comment #6 from Jonathan Druart --- Thanks Kyle. I switch back to Needs Signed Off, because this alternative patch entirely differs from your first one. -- 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 --- Comment #5 from Kyle M Hall --- (In reply to comment #3) > QA comment: > > Why do you modify 2 routines when you can just modify the pl file? Only because I hadn't thought of that! Here's an alternative with that implementation. -- 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 --- Comment #4 from Kyle M Hall --- Created attachment 16073 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=16073&action=edit Bug 9575 - Serious Holds Problem with IndependantBranches [Alternative] Test plan: 1) Disable IndependantBranches 2) Place 3 holds on a record with 3 different pickup locations 3) Enable IndependantBranches 4) Refresh the holds page for that record, all the holds show now show a pickup location of the logged in branch 5) Modify the order of the holds 6) Disable IndependantBranches 7) Refresh the holds, verify the holds all now have a pickup location of the logged in branch 8) Apply the patch 9) Repeat steps 1-3 10) Refresh the holds page for that record, all the holds should now show the correct pickup branch, but it cannot be changed as IndependantBranches is enabled. -- 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575
Jonathan Druart changed:
What|Removed |Added
CC||jonathan.dru...@biblibre.co
||m
--- Comment #3 from Jonathan Druart ---
QA comment:
Why do you modify 2 routines when you can just modify the pl file? To me,
Getbranches($foo, $branch) is not consistent.
I think you can do something like
if ( C4::Context->preference('IndependantBranches') ) {
$reserve{'branchloop'} = GetBranchDetail($res->{'branchcode'});
} else {
$reserve{'branchloop'} = GetBranchesLoop($res->{'branchcode'});
}
Feel free to discuss if you disagree :)
--
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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 Liz Rea changed: What|Removed |Added CC||l...@catalyst.net.nz -- 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 Liz Rea changed: What|Removed |Added Attachment #15172|0 |1 is obsolete|| --- Comment #2 from Liz Rea --- Created attachment 16058 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=16058&action=edit Bug 9575 - Serious Holds Problem with IndependantBranches Test plan: 1) Disable IndependantBranches 2) Place 3 holds on a record with 3 different pickup locations 3) Enable IndependantBranches 4) Refresh the holds page for that record, all the holds show now show a pickup location of the logged in branch 5) Modify the order of the holds 6) Disable IndependantBranches 7) Refresh the holds, verify the holds all now have a pickup location of the logged in branch 8) Apply the patch 9) Repeat steps 1-3 10) Refresh the holds page for that record, all the holds should now show the correct pickup branch, but it cannot be changed as IndependantBranches is enabled. Signed-off-by: Liz Rea works as advertised, passes tests -- 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 Liz Rea changed: 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 mathieu saby changed: What|Removed |Added CC||mathieu.saby@univ-rennes2.f ||r -- 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 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 the assignee for the bug. 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 Kyle M Hall changed: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. 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 9575] Serious Holds Problem with IndependantBranches
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9575 --- Comment #1 from Kyle M Hall --- Created attachment 15172 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15172&action=edit Bug 9575 - Serious Holds Problem with IndependantBranches Test plan: 1) Disable IndependantBranches 2) Place 3 holds on a record with 3 different pickup locations 3) Enable IndependantBranches 4) Refresh the holds page for that record, all the holds show now show a pickup location of the logged in branch 5) Modify the order of the holds 6) Disable IndependantBranches 7) Refresh the holds, verify the holds all now have a pickup location of the logged in branch 8) Apply the patch 9) Repeat steps 1-3 10) Refresh the holds page for that record, all the holds should now show the correct pickup branch, but it cannot be changed as IndependantBranches is enabled. -- You are receiving this mail because: You are the assignee for the bug. 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/
