Bug 2889: Removed unused toggle variable from pcard-member-search.pl. Added table highlighting to .tmpl.
Removed unused toggle variable form pcard-member-search.pl. Added highlighting to pcard-members-search.tmpl and fixed some XHMTL errors. Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
This commit is contained in:
parent
7f28053a65
commit
4ec6c7b10d
|
|
@ -85,9 +85,9 @@ function add_item3(borrowernumber){
|
|||
<input type="hidden" name="batch_id" value="<!-- TMPL_VAR NAME="batch_id" -->" />
|
||||
<input type="hidden" name="type" value="<!-- TMPL_VAR NAME="type" -->" />
|
||||
<label for="member">Name</label>
|
||||
<input name="member" />
|
||||
<input id="member" name="member" />
|
||||
<label for="category">Category code</label>
|
||||
<input name="category" />
|
||||
<input id="category" name="category" />
|
||||
<input type="submit" value="Search" /></p>
|
||||
</form>
|
||||
|
||||
|
|
@ -100,7 +100,7 @@ function add_item3(borrowernumber){
|
|||
|
||||
<table style="float: left; margin: .5em 0;">
|
||||
<tr>
|
||||
<th><label for="patron<!-- TMPL_VAR NAME="borrowernumber" -->" style="font-weight:bold;">Select</label></th>
|
||||
<th><label style="font-weight:bold;">Select</label></th>
|
||||
<th><a href="pcard-member-search.pl?member=<!--TMPL_VAR Name="member" -->&category=<!--TMPL_VAR Name="category_type" -->&batch_id=<!-- TMPL_VAR NAME="batch_id" -->&type=<!-- TMPL_VAR NAME="type" -->&orderby=cardnumber">Card</a></th>
|
||||
<th><a href="pcard-member-search.pl?member=<!--TMPL_VAR Name="member" -->&category=<!--TMPL_VAR Name="category_type" -->&batch_id=<!-- TMPL_VAR NAME="batch_id" -->&type=<!-- TMPL_VAR NAME="type" -->&orderby=surname">Name</a></th>
|
||||
<th><a href="pcard-member-search.pl?member=<!--TMPL_VAR Name="member" -->&category=<!--TMPL_VAR Name="category_type" -->&batch_id=<!-- TMPL_VAR NAME="batch_id" -->&type=<!-- TMPL_VAR NAME="type" -->&orderby=borrowers.categorycode">Category</a></th>
|
||||
|
|
@ -110,7 +110,11 @@ function add_item3(borrowernumber){
|
|||
<th></th>
|
||||
</tr>
|
||||
<!-- TMPL_LOOP NAME="resultsloop" -->
|
||||
<!-- TMPL_UNLESS NAME="__odd__" -->
|
||||
<tr class="highlight">
|
||||
<!-- TMPL_ELSE -->
|
||||
<tr>
|
||||
<!-- /TMPL_UNLESS -->
|
||||
<td> <input type="checkbox" name="borrowernumber" id="patron<!-- TMPL_VAR NAME="borrowernumber" -->" value="<!-- TMPL_VAR NAME="borrowernumber" -->" /> </td>
|
||||
<td><!-- TMPL_VAR NAME="cardnumber" --></td>
|
||||
<td style="white-space: nowrap;"><a href="/cgi-bin/koha/members/morepcard-member-search.pl?borrowernumber=<!-- TMPL_VAR NAME="borrowernumber" -->"><!-- TMPL_VAR NAME="surname" -->, <!-- TMPL_VAR NAME="firstname" --></a> <br /> <!-- TMPL_VAR NAME="address" --><!-- TMPL_IF NAME="address2" --><br /><!-- TMPL_VAR NAME="address2" --><!-- /TMPL_IF --><!-- TMPL_IF NAME="city" --><br /><!-- TMPL_VAR NAME=city --><!-- /TMPL_IF --></td>
|
||||
|
|
|
|||
|
|
@ -95,14 +95,12 @@ else
|
|||
|
||||
|
||||
my @resultsdata;
|
||||
my $toggle = 0;
|
||||
my $to=($count>($startfrom*$resultsperpage)?$startfrom*$resultsperpage:$count);
|
||||
for (my $i=($startfrom-1)*$resultsperpage; $i < $to; $i++){
|
||||
#find out stats
|
||||
my ($od,$issue,$fines)=GetMemberIssuesAndFines($results->[$i]{'borrowernumber'});
|
||||
|
||||
my %row = (
|
||||
toggle => $toggle,
|
||||
count => $i+1,
|
||||
borrowernumber => $results->[$i]{'borrowernumber'},
|
||||
cardnumber => $results->[$i]{'cardnumber'},
|
||||
|
|
@ -125,7 +123,6 @@ for (my $i=($startfrom-1)*$resultsperpage; $i < $to; $i++){
|
|||
sort2 => $results->[$i]{'sort2'},
|
||||
dateexpiry => C4::Dates->new($results->[$i]{'dateexpiry'},'iso')->output('syspref'),
|
||||
);
|
||||
if ( $toggle ) { $toggle = 0; } else {$toggle = 1; }
|
||||
push(@resultsdata, \%row);
|
||||
}
|
||||
my $base_url =
|
||||
|
|
|
|||
Loading…
Reference in New Issue