Jonathan Druart
cae4b98060
This unnecessary complicated subroutine returned an arrayref and an
hashref of the patron categories available for the logged in user, for a given
category_type, ordered by categorycode.
This can now be done with the search_limited method.
Test plan:
- Same prerequisite as before
For the following pages, you should not see patron categories limited to other
libraries. They should be ordered as before this patch, by categorycode.
- Add/edit a patron, change his/her patron category value.
- On the 3 following reports:
reports/bor_issues_top.pl
reports/borrowers_out.pl
reports/cat_issues_top.pl
The display for these 3 reports are different than the 2 from the first
patch (borrowers_stats.pl issues_avg_stats.pl): they are ordered by
categorycode and the ones limited to other libraries are not displayed
(should certainly be fixed).
Note that the big part of this patch has already been tested before
(update child related: CATCODE_MULTI).
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
96 lines
3.3 KiB
Text
96 lines
3.3 KiB
Text
[% INCLUDE 'doc-head-open.inc' %]
|
|
<title>Koha › Choose Adult category</title>
|
|
[% INCLUDE 'doc-head-close.inc' %]
|
|
<link rel="stylesheet" type="text/css" href="[% interface %]/[% theme %]/css/datatables.css" />
|
|
[% INCLUDE 'datatables.inc' %]
|
|
<script type="text/javascript">
|
|
$(document).ready(function() {
|
|
$("#catst").dataTable($.extend(true, {}, dataTablesDefaults, {
|
|
"sDom": 't',
|
|
"aaSorting": [[ 2, "asc" ]],
|
|
"aoColumnDefs": [
|
|
{ "aTargets": [ 0 ], "bSortable": false, "bSearchable": false }
|
|
],
|
|
"bPaginate": false
|
|
}));
|
|
});
|
|
</script>
|
|
<style type="text/css">
|
|
#custom-doc { width:29em;*width:28.3em;min-width:377px; margin:auto; text-align:left; }
|
|
</style>
|
|
</head>
|
|
<body id="pat_update-child" class="pat">
|
|
|
|
<div id="custom-doc" class="yui-t7">
|
|
<div id="bd">
|
|
|
|
[% IF ( CONFIRM ) %]
|
|
<script type="text/javascript">
|
|
function confirm_updatechild() {
|
|
var is_confirmed = window.confirm('Are you sure you want to update this child to an Adult category? This cannot be undone.');
|
|
|
|
if (is_confirmed) {
|
|
window.location='/cgi-bin/koha/members/update-child.pl?op=update&borrowernumber=[% borrowernumber %]&catcode=[% catcode %]&catcode_multi=[% CATCODE_MULTI %]';
|
|
}
|
|
}
|
|
|
|
confirm_updatechild([% borrowernumber %]);
|
|
</script>
|
|
[% END %]
|
|
|
|
[% IF ( SUCCESS ) %]
|
|
<script type="text/javascript">
|
|
self.opener.location.href='/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrowernumber %]';
|
|
window.close();
|
|
</script>
|
|
[% END %]
|
|
|
|
|
|
|
|
[% IF ( MULTI ) %]
|
|
|
|
<h3> Choose Adult category </h3>
|
|
|
|
[% IF patron_categories %]
|
|
<form method="post" action="update-child.pl">
|
|
<fieldset>
|
|
<table id="catst">
|
|
<thead>
|
|
<tr>
|
|
<th> </th>
|
|
<th>Code</th>
|
|
<th>Description</th>
|
|
</tr>
|
|
</thead>
|
|
<tbody>
|
|
[% FOREACH patron_category IN patron_categories %]
|
|
<tr>
|
|
<td>
|
|
<input type="radio" id="catcode[% patron_category.categorycode %]" name="catcode" value="[% patron_category.categorycode %]" />
|
|
</td>
|
|
<td>[% patron_category.categorycode %]</td>
|
|
<td><label for="catcode[% patron_category.categorycode %]"><strong>[% patron_category.description %]</strong></label></td>
|
|
</tr>
|
|
[% END %]
|
|
</tbody>
|
|
</table>
|
|
<input type="hidden" name="op" value="update" />
|
|
<input type="hidden" name="borrowernumber" value="[% borrowernumber %]" />
|
|
<input type="hidden" name="catcode" value="[% catcode %]" />
|
|
<input type="hidden" name="cattype" value="[% cattype %]" />
|
|
<input type="hidden" name="catcode_multi" value="[% CATCODE_MULTI %]" />
|
|
<fieldset class="action">
|
|
<input class="submit" type="submit" value="Submit" />
|
|
<a href="#" class="cancel close">Cancel</a>
|
|
</fieldset>
|
|
</fieldset>
|
|
</form>
|
|
[% END %]
|
|
[% END %]
|
|
|
|
|
|
|
|
|
|
|
|
</div>
|
|
[% INCLUDE 'popup-bottom.inc' %]
|