Bug 5787 - Move OPACItemHolds system preference to the Circulation Matrix.
Summary: Move OPACItemHolds system preference to the Circulation Matrix.
Status: RESOLVED DUPLICATE of bug 5786
Alias: None
Product: Koha
Classification: Unclassified
Component: Hold requests (show other bugs)
Version: 3.4
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Srdjan Jankovic
QA Contact: Bugs List
URL:
Keywords:
Depends on: 5786
Blocks: 5609
  Show dependency treegraph
 
Reported: 2011-02-21 20:01 UTC by Chris Cormack
Modified: 2012-08-06 14:27 UTC (History)
4 users (show)

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


Attachments
patch (35.44 KB, patch)
2011-07-22 05:58 UTC, Srdjan Jankovic
Details | Diff | Splinter Review
patch (35.45 KB, patch)
2011-07-22 07:32 UTC, Srdjan Jankovic
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Cormack 2011-02-21 20:01:32 UTC
Allowing it to be restricted by itemtype or patrontype, not a blanket preference
Comment 1 Srdjan Jankovic 2011-07-22 05:58:36 UTC Comment hidden (obsolete)
Comment 2 Srdjan Jankovic 2011-07-22 07:32:26 UTC Comment hidden (obsolete)
Comment 3 Paul Poulain 2012-02-17 14:39:05 UTC
The script devel/misc/testbugzillapatches.pl has detected that this bug does not apply anymore, so i'm marking this bug as "does not apply".

The script is supposed to handle conflicts related to updatedatabase, but no other cases of merge conflict. So the conflict may be trivial to fix.

If it happens that the patch attached does apply cleanly, please send me an email, there's something wrong in the script. It's possible, as it's a recent script.
If the patch applies does not apply as I think, please rebase and resubmit, thanks
Comment 4 Melia Meggs 2012-07-24 23:26:26 UTC
A new patch coming soon for this!!  I want to provide test plans to facilitate community testing.


Test plan for OPAC Item Holds only:


Rule 1:
Patron Category 1
Item type 1
Allow patrons to place item specific holds

Rule 2:
Patron Category 2
Item type 2
Don't Allow patrons to place item specific holds

In both the staff client and the OPAC try the following for each rule combo:

Place an item specific hold

--------------------

Please test with Bug 5786 as well.


Test Plan for both On Shelf Holds and OPAC Item Holds together


Set up the following rules:

Rule 1:
Patron Category 1
Item type 1
On Shelf Holds Not Allowed
Allow patrons to place item specific holds

Rule 2:
Patron Category 2
Item type 2
On Shelf Holds Allowed
Allow patrons to place item specific holds

Rule 3:
Patron Category 3
Item type 3
On Shelf Holds Allowed
Don't allow patrons to place item specific holds

Rule 4:
Patron Category 4
Item type 4
On Shelf Holds not Allowed
Don't allow patrons to place item specific holds


In both the staff client and the OPAC try the following for each rule combo:

Place hold on shelf item
Place hold on checked out item
Place an item specific hold
Comment 5 Srdjan Jankovic 2012-07-25 03:47:53 UTC
This bug will be resolved in unified patch on bug 5786
Comment 6 Srdjan Jankovic 2012-07-25 03:49:39 UTC
Comment on attachment 4712 [details] [review]
patch

Integrated with 5786 and 5788
Comment 7 Julian Maurice 2012-08-06 14:27:59 UTC
As it was merged with bug 5786, I think the appropriate status is RESOLVED - DUPLICATE

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