Bug 26184

Summary: Wrap 'items available for pick-up' note when placing a hold in the OPAC in a div element
Product: Koha Reporter: Lucas Gass <lucas>
Component: OPACAssignee: Lucas Gass <lucas>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: aleisha, brandon.jimenez, david, jonathan.druart, oleonard
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:
20.11.00, 20.05.06
Attachments: Bug 26184: Add html element to note in OPAC reserve template
Bug : 26184 - element missing a proper HTML element
Bug 26184: Add html element to note in OPAC reserve template
Bug 26184: Add div element to 'items available' note in OPAC reserve template

Description Lucas Gass 2020-08-07 18:14:17 UTC
In the opac-reserve template there are these lines:

[% IF at_least_one_library_not_available_for_pickup %]
<br>Note: Library policy does not allow hold/pickup of an item available locally. Please come to the library to retrieve these items
[% END %]

This is wording that some libraries would like to change but since it is not wrapped in an HTML element is difficult.

We should wrap this in an HTML element
Comment 1 Lucas Gass 2020-08-07 18:20:16 UTC
Created attachment 107972 [details] [review]
Bug 26184: Add html element to note in OPAC reserve template
Comment 2 Brandon J 2020-08-18 15:44:11 UTC
Created attachment 108526 [details] [review]
Bug : 26184 - element missing a proper HTML element

Signed-off-by: Brandon J <brandon.jimenez@inLibro.com>
Comment 3 Brandon J 2020-08-18 15:46:52 UTC
to test :
  2 -before applying the patch :
  3  -- 1 Home (OPAC)
  4  -- 2 Click on a book
  5  -- 3 Click Place Hold
  6  NOTE: if you don't see the text, the library has to disable hold.
  7  To disable Hold:
  8  -- 1 HOME (Intranet)
  9  -- 2 Preferences
 10  -- 3 Search for OPACHoldsIfAvailableAtPickup and turn it to "Don't allow"
 11 
 12 - APPLY PATCH
 13 - Retest (in the inspector the note is enclosed now by a SPAN tag)
Comment 4 Lucas Gass 2020-08-18 21:12:05 UTC
Hi Brandon,

Thanks for looking at this and testing. Did you sign-off on this patch? Looks like you added a sign-off but didn't change the status to signed-off.

Looks like you also supplied a patch which includes some changes to updatedatabase.pl, I am going to mark that patch as obsolete.
Comment 5 David Nind 2020-09-18 14:42:42 UTC
Created attachment 110401 [details] [review]
Bug 26184: Add html element to note in OPAC reserve template

Signed-off-by: David Nind <david@davidnind.com>
Comment 6 David Nind 2020-09-18 14:44:01 UTC
I signed-off using Brandon J's test plan in comment 3 - feel free to change the sign off attribution to Brandon J!
Comment 7 Katrin Fischer 2020-09-20 17:24:25 UTC
Created attachment 110427 [details] [review]
Bug 26184: Add div element to 'items available' note in OPAC reserve template

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

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 8 Jonathan Druart 2020-09-28 14:08:56 UTC
Why this class name? Isn't it just a note?
Comment 9 Lucas Gass 2020-09-28 15:08:49 UTC
(In reply to Jonathan Druart from comment #8)
> Why this class name? Isn't it just a note?

Jonathan, would you rather the class name be something else? I don't mind changing it.
Comment 10 Jonathan Druart 2020-09-29 10:02:29 UTC
class="note"?
Comment 11 Katrin Fischer 2020-09-29 10:54:14 UTC
Hi Jonathan, as the stated intention here was to be able to change the text, a specific class (or id) makes more sense. With possibly changing translations hooking the jQuery on a test or word is not ideal.
Comment 12 Katrin Fischer 2020-09-29 10:59:40 UTC
test = text...
Comment 13 Jonathan Druart 2020-10-08 09:37:56 UTC
Pushed to master for 20.11, thanks to everybody involved!
Comment 14 Lucas Gass 2020-11-09 18:46:51 UTC
backported to 20.05.x for 20.05.06
Comment 15 Aleisha Amohia 2020-11-17 03:27:04 UTC
enhancement, not backported to 19.11.x