Bugzilla – Attachment 171246 Details for
Bug 37882
Add classes to item statuses in SCO
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 37882: Add class for statuses to SCO module for holds and renewals
Bug-37882-Add-class-for-statuses-to-SCO-module-for.patch (text/plain), 8.95 KB, created by
Nick Clemens (kidclamp)
on 2024-09-10 11:41:34 UTC
(
hide
)
Description:
Bug 37882: Add class for statuses to SCO module for holds and renewals
Filename:
MIME Type:
Creator:
Nick Clemens (kidclamp)
Created:
2024-09-10 11:41:34 UTC
Size:
8.95 KB
patch
obsolete
>From 3df415a96def025facbb19961f77e608337e013a Mon Sep 17 00:00:00 2001 >From: Nick Clemens <nick@bywatersolutions.com> >Date: Tue, 10 Sep 2024 11:38:29 +0000 >Subject: [PATCH] Bug 37882: Add class for statuses to SCO module for holds and > renewals > >This patch should have no visual impact, but will add several classes > >To test: >1 - Place some holds and checkout items to a patron > Ensure some items cannot renew, and set a hold with a library as > pickup location, then unset library as a location in Library admin page >2 - View their record in sco >3 - Apply patch >4 - 'Finish' and start a new SCO session as same user >5 - No visual difference >6 - Inspect issues/hold to see if new psans are present >--- > .../bootstrap/en/modules/opac-user.tt | 52 +++++++++---------- > 1 file changed, 26 insertions(+), 26 deletions(-) > >diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt >index 67a54190c9d..994923b8496 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt >@@ -68,41 +68,41 @@ > <li> > [% SWITCH fail %] > [% CASE 'damaged' %] >- <span>Item is damaged</span> >+ <span class="sco_damaged">Item is damaged</span> > [% CASE 'ageRestricted' %] >- <span>The record and its items are age restricted</span> >+ <span class="sco_agerestricted">The record and its items are age restricted</span> > [% CASE 'tooManyHoldsForThisRecord' %] >- <span>You have exceeded the number of concurrent holds for this record</span> >+ <span class="sco_toomanyholdsforthisrecord">You have exceeded the number of concurrent holds for this record</span> > [% CASE 'tooManyReservesToday' %] >- <span>You've reached your daily holds limit</span> >+ <span class="sco_toomanyreservestoday">You've reached your daily holds limit</span> > [% CASE 'tooManyReserves' %] >- <span>You have too many holds already</span> >+ <span class="sco_toomanyreserves">You have too many holds already</span> > [% CASE 'notReservable' %] >- <span>Not holdable</span> >+ <span class="sco_notreservable">Not holdable</span> > [% CASE 'cannotReserveFromOtherBranches' %] >- <span>You are from a different library</span> >+ <span class="sco_cannotreservefromotherbranches">You are from a different library</span> > [% CASE 'branchNotInHoldGroup' %] >- <span>Holds are not allowed from your library</span> >+ <span class="sco_branchnotinholdgroup">Holds are not allowed from your library</span> > [% CASE 'itemAlreadyOnHold' %] >- <span>You already have a hold for this item</span> >+ <span class="sco_itemalreadyonhold">You already have a hold for this item</span> > [% CASE 'cannotBeTransferred' %] >- <span>Cannot be transferred to pickup library</span> >+ <span class="sco_cannotbetransferred">Cannot be transferred to pickup library</span> > [% CASE 'pickupNotInHoldGroup' %] >- <span>The chosen pickup location is not allowed</span> >+ <span class="sco_pickupnotinholdgroup">The chosen pickup location is not allowed</span> > [% CASE 'noReservesAllowed' %] >- <span>No holds are allowed on this item</span> >+ <span class="sco_noreservesallowed">No holds are allowed on this item</span> > [% CASE 'libraryNotPickupLocation' %] >- <span>Library is not a valid pickup location</span> >+ <span class="sco_librarynotpickuplocation">Library is not a valid pickup location</span> > [% CASE 'no_valid_pickup_location' %] >- <span>No valid pickup location</span> >+ <span class="sco_no_valid_pickup_location">No valid pickup location</span> > [% CASE 'notforloan' %] >- <span>Not for loan</span> >+ <span class="sco_notforloan">Not for loan</span> > [% CASE 'items_available' %] >- <span>There are items available in the library, please visit to check them out</span> >+ <span class="sco_items_available">There are items available in the library, please visit to check them out</span> > [% CASE 'not_placed' %] >- <span>Error when placing hold, please report this to the library</span> >+ <span class="sco_not_placed">Error when placing hold, please report this to the library</span> > [% CASE %] >- <span>Error: [% fail | html %]</span> >+ <span class="sco_error_placing_hold">Error: [% fail | html %]</span> > [% END %] > </li> > [% END %] >@@ -136,21 +136,21 @@ > <ul> > [% FOREACH error IN RENEW_ERROR.split('\|') %] > [% IF error == 'card_expired' %] >- <li>Your account has expired. Please contact the library for more information.</li> >+ <li class="sco_renew_card_expired">Your account has expired. Please contact the library for more information.</li> > [% ELSIF error == 'too_many' %] >- <li>You have renewed this item the maximum number of times allowed.</li> >+ <li class="sco_renew_too_many">You have renewed this item the maximum number of times allowed.</li> > [% ELSIF error == 'too_unseen' %] >- <li>You have renewed this item the maximum number of consecutive times without it being seen by the library.</li> >+ <li class="sco_renew_too_unseen">You have renewed this item the maximum number of consecutive times without it being seen by the library.</li> > [% ELSIF error == 'too_soon' %] >- <li>It is too soon after the checkout date for this item to be renewed.</li> >+ <li class="sco_renew_too_soon">It is too soon after the checkout date for this item to be renewed.</li> > [% ELSIF error == 'on_reserve' %] >- <li>This item is on hold for another patron.</li> >+ <li class="sco_renew_on_reserve">This item is on hold for another patron.</li> > [% ELSIF error == 'item_denied_renewal' %] >- <li>Item renewal is not allowed.</li> >+ <li class="sco_renew_item_denied_renewal">Item renewal is not allowed.</li> > [% ELSIF error == 'item_issued_to_other_patron'%] >- <li>This item is already checked out to another patron.</li> >+ <li class="sco_renew_item_issued_to_other_patron">This item is already checked out to another patron.</li> > [% ELSIF error == 'auto_too_soon' %] >- <li>This item is scheduled for auto renewal.</li> >+ <li class="sco_renew_auto_too_soon">This item is scheduled for auto renewal.</li> > [% END %] > [% END %] > </ul> >-- >2.39.2
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 37882
:
171246
|
171262