Bug 9487

Summary: Allow items fields to be used in the HOLDPLACED notice
Product: Koha Reporter: Kyle M Hall <kyle>
Component: NoticesAssignee: Kyle M Hall <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:
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 2013-01-24 19:16:46 UTC
A number of libraries have asked why the HOLDPLACED notice allows you to choose item level fields but they are never filled in. There is no reason to disallow the use of items fields in the HOLDPLACED notice. 

For bib-level holds they will not be populated, but as long as the libraries know this, it does not seem like it would be an issue. It would probably be best to add a note about this in the manual.
Comment 1 Kyle M Hall 2013-01-24 19:19:05 UTC Comment hidden (obsolete)
Comment 2 Kyle M Hall 2013-01-24 19:21:10 UTC Comment hidden (obsolete)
Comment 3 Koha Team University Lyon 3 2013-03-18 14:30:35 UTC
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
Comment 4 Srdjan Jankovic 2013-06-11 04:44:56 UTC Comment hidden (obsolete)
Comment 5 Srdjan Jankovic 2013-06-11 04:47:12 UTC
Looks fine, however there could be some unwanted effects (blanks) in case of placing a hold on the next available.
Comment 6 Katrin Fischer 2013-07-11 06:02:28 UTC
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.
Comment 7 Galen Charlton 2013-08-14 22:08:13 UTC
Pushed to master.  Thanks, Kyle!