# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
# A PARTICULAR PURPOSE. See the GNU General Public License for more details.
#
-# You should have received a copy of the GNU General Public License along with
-# Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
-# Suite 330, Boston, MA 02111-1307 USA
+# You should have received a copy of the GNU General Public License along
+# with Koha; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
use strict;
use warnings;
+use CGI;
+
use C4::Auth;
use C4::Output;
-use CGI;
use C4::Members;
use C4::Branch;
+use List::MoreUtils qw/any/;
use C4::Dates qw/format_date/;
-my $input=new CGI;
+my $input = CGI->new;
-my $borrowernumber=$input->param('borrowernumber');
#get borrower details
-my $data=GetMember($borrowernumber,'borrowernumber');
-my $order=$input->param('order') || '';
-my $order2=$order;
-if ($order2 eq ''){
- $order2="date_due desc";
+my $data = undef;
+my $borrowernumber = undef;
+my $cardnumber = undef;
+
+if ($input->param('cardnumber')) {
+ $cardnumber = $input->param('cardnumber');
+ $data = GetMember(cardnumber => $cardnumber);
+ $borrowernumber = $data->{'borrowernumber'}; # we must define this as it is used to retrieve other data about the patron
}
+if ($input->param('borrowernumber')) {
+ $borrowernumber = $input->param('borrowernumber');
+ $data = GetMember(borrowernumber => $borrowernumber);
+}
+
+my $order=$input->param('order') || 'date_due desc';
my $limit=$input->param('limit');
if ($limit){
if ($limit eq 'full'){
$limit=0;
}
-}
+}
else {
$limit=50;
}
-my ($count,$issues)=GetAllIssues($borrowernumber,$order2,$limit);
+my ( $issues ) = GetAllIssues($borrowernumber,$order,$limit);
my ($template, $loggedinuser, $cookie)
= get_template_and_user({template_name => "members/readingrec.tmpl",
my @loop_reading;
-for (my $i=0;$i<$count;$i++){
+foreach my $issue (@{$issues}){
my %line;
- $line{biblionumber}=$issues->[$i]->{'biblionumber'};
- $line{title}=$issues->[$i]->{'title'};
- $line{author}=$issues->[$i]->{'author'};
- $line{classification} = $issues->[$i]->{'classification'} || $issues->[$i]->{'itemcallnumber'};
- $line{date_due}=format_date($issues->[$i]->{'date_due'});
- $line{returndate}=format_date($issues->[$i]->{'returndate'});
- $line{renewals}=$issues->[$i]->{'renewals'};
- $line{barcode}=$issues->[$i]->{'barcode'};
- $line{volumeddesc}=$issues->[$i]->{'volumeddesc'};
+ $line{issuestimestamp} = format_date($issue->{'issuestimestamp'});
+ $line{biblionumber} = $issue->{'biblionumber'};
+ $line{title} = $issue->{'title'};
+ $line{author} = $issue->{'author'};
+ $line{classification} = $issue->{'classification'} || $issue->{'itemcallnumber'};
+ $line{date_due} = format_date($issue->{'date_due'});
+ $line{returndate} = format_date($issue->{'returndate'});
+ $line{issuedate} = format_date($issue->{'issuedate'});
+ $line{issuingbranch} = GetBranchName($issue->{'branchcode'});
+ $line{renewals} = $issue->{'renewals'};
+ $line{barcode} = $issue->{'barcode'};
+ $line{volumeddesc} = $issue->{'volumeddesc'};
push(@loop_reading,\%line);
}
}
$template->param( adultborrower => 1 ) if ( $data->{'category_type'} eq 'A' );
-if (! $limit){
- $limit = 'full';
+if (! $limit){
+ $limit = 'full';
}
my ($picture, $dberror) = GetPatronImage($data->{'cardnumber'});
branchcode => $data->{'branchcode'},
is_child => ($data->{'category_type'} eq 'C'),
branchname => GetBranchName($data->{'branchcode'}),
- showfulllink => ($count > 50),
+ showfulllink => (scalar @loop_reading > 50),
loop_reading => \@loop_reading);
output_html_with_http_headers $input, $cookie, $template->output;