Summary: | Allow items fields to be used in the HOLDPLACED notice | ||
---|---|---|---|
Product: | Koha | Reporter: | Kyle M Hall (khall) <kyle> |
Component: | Notices | Assignee: | Kyle M Hall (khall) <kyle> |
Status: | CLOSED FIXED | QA Contact: | |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | gmcharlt, koha, srdjan |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 9487 - Allow items fields to be used in the HOLDPLACED notice
Bug 9487 - Allow items fields to be used in the HOLDPLACED notice [SIGNED-OFF] Bug 9487 - Allow items fields to be used in the HOLDPLACED notice [PASSED QA] Bug 9487 - Allow items fields to be used in the HOLDPLACED notice |
Description
Kyle M Hall (khall)
2013-01-24 19:16:46 UTC
Created attachment 14836 [details] [review] Bug 9487 - Allow items fields to be used in the HOLDPLACED notice Test Plan: 1) Enable the syspref emailLibrarianWhenHoldIsPlaced 2) Modify the HOLDPLACED notice, add some item level fields 3) Place an item level hold 4) Check the email you recieve ( or just look at it from the db ) You should see the item level fields are new populated Created attachment 14838 [details] [review] Bug 9487 - Allow items fields to be used in the HOLDPLACED notice Test Plan: 1) Enable the syspref emailLibrarianWhenHoldIsPlaced 2) Modify the HOLDPLACED notice, add some item level fields 3) Place an item level hold 4) Check the email you recieve ( or just look at it from the db ) You should see the item level fields are new populated I'm not sure to use the good field for this test. I try to add <<reserves.priority>> in the notice HOLDPLACED but it doesn't appear in the email. Which fields can be add to add to test the item level Created attachment 18833 [details] [review] [SIGNED-OFF] Bug 9487 - Allow items fields to be used in the HOLDPLACED notice Signed-off-by: Srdjan <srdjan@catalyst.net.nz> Looks fine, however there could be some unwanted effects (blanks) in case of placing a hold on the next available. Created attachment 19538 [details] [review] [PASSED QA] Bug 9487 - Allow items fields to be used in the HOLDPLACED notice Test Plan: 1) Enable the syspref emailLibrarianWhenHoldIsPlaced 2) Modify the HOLDPLACED notice, add some item level fields 3) Place an item level hold 4) Check the email you receive ( or just look at it from the db ) You should see the item level fields are new populated Signed-off-by: Srdjan <srdjan@catalyst.net.nz> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Passes all tests and QA script. Amended test plan: 5) Place a title level hold 6) Check the email you receive - item fields are not populated, but notice still looks ok. Pushed to master. Thanks, Kyle! |