Bug 22752

Created attachment 115294 [details] [review]
Bug 22752: Show if a placed hold is a record or item level in the patron account in the OPAC

Test Plan
1. Have a record with an item
2. You will need to set a hold through staff account (to import books
        Cataloging>New from Z39.50/SRU>
3. Keyword(enter any keyword e.g pokemon)
4. tick library of congress
5. choose a title and click actions then import
6. 003 should be OSt
7. tab 9 koha item type can be books or whatever and press save
8. select the hold and place hold
9. on Koha OPAC click on your account and go to the holds and
underneath the title it should say Only item and the barcode

Created attachment 115302 [details] [review]
Bug 22752: Show if a placed hold is a record or item level in the patron account in the OPAC

Test Plan
1. Have a record with an item
2. You will need to set a hold through staff account (to import books
        Cataloging>New from Z39.50/SRU>
3. Keyword(enter any keyword e.g pokemon)
4. tick library of congress
5. choose a title and click actions then import
6. 003 should be OSt
7. tab 9 koha item type can be books or whatever and press save
8. select the hold and place hold
9. on Koha OPAC click on your account and go to the holds and
underneath the title it should say Only item and the barcode

Signed-off-by: David Nind <david@davidnind.com>

Testing notes:
- Worked as expected, I tested placing next available item and specific item holds
- Works fine without needing to add a new record and item (it worked fine with existing example records and items)

Created attachment 115526 [details] [review]
Bug 22752: Show if a placed hold is a record or item level in the patron account in the OPAC

Test Plan
1. Have a record with an item
2. You will need to set a hold through staff account (to import books
        Cataloging>New from Z39.50/SRU>
3. Keyword(enter any keyword e.g pokemon)
4. tick library of congress
5. choose a title and click actions then import
6. 003 should be OSt
7. tab 9 koha item type can be books or whatever and press save
8. select the hold and place hold
9. on Koha OPAC click on your account and go to the holds and
underneath the title it should say Only item and the barcode

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Created attachment 115527 [details] [review]
Bug 22752: (QA follow-up) Add missing filters

For security, we filter all template variables.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Thanks Victoria, 

I've added a minor follow-up to coax this through QA (https://wiki.koha-community.org/wiki/Coding_Guidelines#HTML9:_Filter_all_template_variables), but otherwise this looks great.

Passing QA

It's placed to close to the author in my opinion: https://snipboard.io/utPVk4.jpg
Maybe we should add a new line?

Created attachment 115558 [details] [review]
Bug 22752: (follow-up) Add style; Only show message about item-level holds

I think it's generally assumed that a hold has, by default, been made at
the bibliographic level. I think it's sufficent to add a note for only
item-level holds.

To test, apply the patch and log into the OPAC as a user who has
multiple holds, some at the bibliographic level and some at the item
level.

- Go to Your summary --> Holds
- Confirm that any item-level hold has a note under the title: "Item on
  hold <barcode>".
- The note should be on a new line and styled differently than the other
  text.

My follow-up goes further than just adding some style. I can submit a less-opinionated patch if folks prefer.

https://snipboard.io/D6JH5R.jpg

Thanks Owen!

Pushed to master for 21.05, thanks to everybody involved!

Updated the bug title to reflect the actual changes made, and added suggested text for the release notes.

Should we backport to 20.11.x ?