0f122a9400
Test Plan:
1. Apply patch
2. create a patron list if there aren't any
3. search for a patron
4. observe the "Patron lists" tab showing the list that the patron is not in
5. try adding the patron to the list and removing them from the list to
be sure the feature has full operation
6. click "Check Out" on the side bar menu to navigate to the circulation
page for this patron
7. observe the "Patron lists" tab, and verify it operates as it did on the
patron details page
Bug 32730: (follow-up) Minor corrections
by Owen Leonard
This patch corrects an instance of an incorrect capital letter
("Patron Lists" -> "Patron lists") and makes minor tweaks to
indentation.
Signed-off-by: Stina Hallin <stina.hallin@ub.lu.se>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Edit: (tcohen) new files should be run through perltidy ALWAYS. Did it
and squashed it here.
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
240 lines
11 KiB
HTML
240 lines
11 KiB
HTML
<!-- patron-detail-tabs.inc -->
|
|
[% USE raw %]
|
|
[% PROCESS 'html_helpers.inc' %]
|
|
[% WRAPPER tabs id= "finesholdsissues" %]
|
|
[% WRAPPER tabs_nav %]
|
|
[% WRAPPER tab_item tabname= "checkouts" bt_active= 1 %]
|
|
<span>Checkouts ([% issuecount || 0 | html %])</span>
|
|
[% END %]
|
|
|
|
[% IF relatives_issues_count %]
|
|
[% WRAPPER tab_item tabname= "relatives-issues" %]
|
|
<span>Relatives' checkouts ([% relatives_issues_count | html %])</span>
|
|
[% END %]
|
|
[% END %]
|
|
|
|
[% IF ( fines ) %]
|
|
[% WRAPPER tab_item tabname= "finesandcharges" %]
|
|
<span>Charges ([% fines | $Price %])</span>
|
|
[% END %]
|
|
[% END %]
|
|
|
|
[% IF ( guarantees_fines ) %]
|
|
[% WRAPPER tab_item tabname= "guarantees_finesandcharges" %]
|
|
<span>Guarantees' charges ([% guarantees_fines | $Price %])</span>
|
|
[% END %]
|
|
[% END %]
|
|
|
|
[% IF ( CAN_user_circulate_circulate_remaining_permissions ) %]
|
|
[% WRAPPER tab_item tabname= "holds" %]
|
|
<span>Holds ([% holds_count || 0 | html %])</span>
|
|
[% END %]
|
|
[% END %]
|
|
|
|
[% IF Koha.Preference('UseRecalls') %]
|
|
[% WRAPPER tab_item tabname= "recalls" %]
|
|
<span>Recalls ([% recalls.count || 0 | html %])</span>
|
|
[% END %]
|
|
[% END %]
|
|
|
|
[% IF Koha.Preference('ArticleRequests') %]
|
|
[% SET article_requests = Context.Scalar( Context.Scalar( patron, 'article_requests' ), 'filter_by_current') %]
|
|
[% WRAPPER tab_item tabname= "article-requests" %]
|
|
<span>Article requests ([% article_requests.count || 0 | html %])</span>
|
|
[% END %]
|
|
[% END %]
|
|
|
|
[% IF Koha.Preference('ClaimReturnedLostValue') || Koha.Preference('BundleLostValue') %]
|
|
[% WRAPPER tab_item tabname= "return-claims" %]
|
|
<span>Claims</span>
|
|
[% IF ( patron.return_claims.count ) %]
|
|
([% IF patron.return_claims.resolved.count == 0 %]
|
|
<span title="Resolved claims" class="label label-default" id="return-claims-count-resolved">[% patron.return_claims.resolved.count | html %]</span>
|
|
[% ELSE %]
|
|
<span title="Resolved claims" class="label label-success" id="return-claims-count-resolved">[% patron.return_claims.resolved.count | html %]</span>
|
|
[% END %]
|
|
[% IF patron.return_claims.unresolved.count == 0 %]
|
|
<span title="Unresolved claims" class="label label-default" id="return-claims-count-unresolved">[% patron.return_claims.unresolved.count | html %]</span>
|
|
[% ELSE %]
|
|
<span title="Unresolved claims" class="label label-warning" id="return-claims-count-unresolved">[% patron.return_claims.unresolved.count | html %]</span>
|
|
[% END %])
|
|
[% ELSE %]
|
|
[% no_claims = 0 %]
|
|
(<span title="Resolved claims" class="label label-default" id="return-claims-count-resolved">[% no_claims | html %]</span>
|
|
<span title="Unresolved claims" class="label label-default" id="return-claims-count-unresolved">[% no_claims | html %]</span>)
|
|
[% END # /IF patron.return_claims.count %]
|
|
[% END # /WRAPPER tab_item return-claims %]
|
|
[% END %]
|
|
|
|
[% WRAPPER tab_item tabname= "reldebarments" %]
|
|
<span>Restrictions ([% patron.restrictions.count || 0 | html %])</span>
|
|
[% END %]
|
|
|
|
[% SET enrollments = patron.get_club_enrollments %]
|
|
[% SET enrollable = patron.get_enrollable_clubs(0) %] <!-- 0 => not OPAC -->
|
|
[% IF CAN_user_clubs && ( enrollable.count || enrollments.count ) %]
|
|
[% WRAPPER tab_item tabname= "clubs" %]
|
|
<span>Clubs ([% enrollments.count | html %]/[% enrollable.count | html %])</span>
|
|
[% END %]
|
|
[% END %]
|
|
|
|
[% IF CAN_user_tools_manage_patron_lists || patron_lists_count %]
|
|
[% WRAPPER tab_item tabname="pat_lists" %]
|
|
<span>Patron lists ([% patron_lists_count | html %])</span>
|
|
[% END %]
|
|
[% END %]
|
|
[% END # /WRAPPER tabs_nav %]
|
|
|
|
|
|
[% WRAPPER tab_panels %]
|
|
[% WRAPPER tab_panel tabname="checkouts" bt_active= 1 %]
|
|
[% INCLUDE "checkouts-table.inc" %]
|
|
[% END # /tab_panel# %]
|
|
|
|
[% WRAPPER tab_panel tabname="relatives-issues" %]
|
|
[% INCLUDE "relatives-issues-table.inc" %]
|
|
[% END # /tab_panel# %]
|
|
|
|
[% WRAPPER tab_panel tabname="finesandcharges" %]
|
|
[% IF ( fines ) %]
|
|
<p>Total due: [% fines | $Price %]</p>
|
|
[% ELSE %]
|
|
<p>No outstanding charges</p>
|
|
[% END %]
|
|
[% END # /tab_panel# %]
|
|
|
|
[% IF ( guarantees_fines ) %]
|
|
[% WRAPPER tab_panel tabname="guarantees_finesandcharges" %]
|
|
<table id="tguaranteesfines">
|
|
<thead>
|
|
<tr>
|
|
<th>Guarantee</th>
|
|
<th>Amount outstanding</th>
|
|
</tr>
|
|
</thead>
|
|
[% FOREACH guarantee IN guarantees %]
|
|
<tr>
|
|
<td>[% INCLUDE 'patron-title.inc' patron=guarantee hide_patron_infos_if_needed=1 %]</td>
|
|
[% IF logged_in_user.can_see_patron_infos( guarantee ) %]
|
|
<td>[% guarantee.account.balance | $Price %]</td>
|
|
[% ELSE %]
|
|
<td>-</td>
|
|
[% END %]
|
|
</tr>
|
|
[% END %]
|
|
<tfoot>
|
|
<td>Total due</td>
|
|
<td>[% guarantees_fines | $Price %]</td>
|
|
</tfoot>
|
|
</table>
|
|
[% END # /tab_panel# %]
|
|
[% END # /tab_panel# %]
|
|
|
|
[% IF CAN_user_clubs && ( enrollments.count || enrollable.count ) %]
|
|
[% WRAPPER tab_panel tabname="clubs" %]
|
|
Loading...
|
|
[% END # /tab_panel# %]
|
|
[% END %]
|
|
|
|
[% IF CAN_user_tools_manage_patron_lists || patron_lists_count %]
|
|
[% WRAPPER tab_panel tabname="pat_lists" %]
|
|
Loading...
|
|
[% END # /tab_panel# %]
|
|
[% END %]
|
|
|
|
[% WRAPPER tab_panel tabname="reldebarments" %]
|
|
[% INCLUDE 'patron-restrictions-tab.inc' %]
|
|
[% END # /tab_panel# %]
|
|
|
|
[% IF ( CAN_user_circulate_circulate_remaining_permissions ) %]
|
|
[% WRAPPER tab_panel tabname="holds" %]
|
|
[% IF ( holds_count ) %]
|
|
<form action="/cgi-bin/koha/reserve/modrequest.pl" method="post">
|
|
<input type="hidden" name="from" value="circ" />
|
|
<table id="holds-table" style="width: 100% !Important;">
|
|
<thead>
|
|
<tr>
|
|
<th>Hold date</th>
|
|
<th class="anti-the">Title</th>
|
|
<th>Call number</th>
|
|
<th>Item type</th>
|
|
<th>Barcode</th>
|
|
<th>Pickup at</th>
|
|
<th>Expiration</th>
|
|
<th>Priority</th>
|
|
<th>Notes</th>
|
|
<th>Delete?</th>
|
|
<th>Suspend?</th>
|
|
<th>Status</th>
|
|
</tr>
|
|
</thead>
|
|
</table>
|
|
|
|
<fieldset class="action">
|
|
<input type="submit" class="cancel" name="submit" value="Cancel marked holds" />
|
|
|
|
[% SET hold_cancellation = AuthorisedValues.GetAuthValueDropbox('HOLD_CANCELLATION') %]
|
|
[% IF hold_cancellation.count %]
|
|
<label for="cancellation-reason">Cancellation reason:</label>
|
|
<select name="cancellation-reason">
|
|
<option value="">No reason given</option>
|
|
[% FOREACH reason IN hold_cancellation %]
|
|
<option value="[% reason.authorised_value | html %]">[% reason.lib | html %]</option>
|
|
[% END %]
|
|
</select>
|
|
[% END %]
|
|
</fieldset>
|
|
</form>
|
|
|
|
[% IF Koha.Preference('SuspendHoldsIntranet') %]
|
|
<form action="/cgi-bin/koha/reserve/modrequest_suspendall.pl" method="post">
|
|
<fieldset class="action">
|
|
<input type="hidden" name="from" value="[% patronpage | html %]" />
|
|
<input type="hidden" name="borrowernumber" value="[% patron.borrowernumber | html %]" />
|
|
<input type="submit" class="btn btn-primary" value="Suspend all holds" />
|
|
|
|
[% IF Koha.Preference('AutoResumeSuspendedHolds') %]
|
|
<label for="suspend_until">until</label>
|
|
<input type="text" size="10" id="suspend_until" name="suspend_until" class="flatpickr" data-flatpickr-futuredate="true"/>
|
|
<span class="hint">Specify date on which to resume [% INCLUDE 'date-format.inc' %]: </span>
|
|
[% END %]
|
|
</fieldset>
|
|
</form>
|
|
|
|
<form action="/cgi-bin/koha/reserve/modrequest_suspendall.pl" method="post">
|
|
<fieldset class="action">
|
|
<input type="hidden" name="from" value="borrower" />
|
|
<input type="hidden" name="borrowernumber" value="[% patron.borrowernumber | html %]" />
|
|
<input type="hidden" name="suspend" value="0" />
|
|
<input type="submit" class="btn btn-primary" value="Resume all suspended holds" />
|
|
</fieldset>
|
|
</form>
|
|
[% END # IF SuspendHoldsIntranet %]
|
|
|
|
[% ELSE %]
|
|
<p>Patron has nothing on hold.</p>
|
|
[% END %]
|
|
[% END # /tab_panel#holds %]
|
|
[% END %]
|
|
|
|
[% IF Koha.Preference('UseRecalls') %]
|
|
[% WRAPPER tab_panel tabname="recalls" %]
|
|
[% INCLUDE 'recalls.inc' %]
|
|
[% END # /tab_panel# %]
|
|
[% END %]
|
|
|
|
[% IF Koha.Preference('ClaimReturnedLostValue') || Koha.Preference('BundleLostValue') %]
|
|
[% WRAPPER tab_panel tabname="return-claims" %]
|
|
[% INCLUDE 'patron-return-claims.inc' %]
|
|
[% END # /tab_panel# %]
|
|
[% END %]
|
|
|
|
[% IF Koha.Preference('ArticleRequests') %]
|
|
[% WRAPPER tab_panel tabname="article-requests" %]
|
|
[% INCLUDE 'patron-article-requests.inc' %]
|
|
[% END %]
|
|
[% END # /tab_panel# %]
|
|
|
|
[% END # /WRAPPER tab_panels %]
|
|
[% END # /WRAPPER tabs %]
|
|
<!-- /patron-detail-tabs.inc -->
|