[Patches] [PATCH] Bug 929 : Last follow up, implementing the last of Katrins
koha-patchbot at kohaaloha.com
koha-patchbot at kohaaloha.com
Thu Dec 29 09:15:02 NZDT 2011
From: Chris Cormack <chrisc at catalyst.net.nz>
Date: Thu, 29 Dec 2011 09:08:40 +1300
Subject: [PATCH] Bug 929 : Last follow up, implementing the last of Katrins
suggestions
Date formatting
Links
Nomenclature changes
---
acqui/ordered.pl | 37 +++++++++-------
acqui/spent.pl | 46 +++++++++-----------
.../intranet-tmpl/prog/en/modules/acqui/ordered.tt | 15 ++++--
3 files changed, 50 insertions(+), 48 deletions(-)
diff --git a/acqui/ordered.pl b/acqui/ordered.pl
index 5796f07..97a4da5 100755
--- a/acqui/ordered.pl
+++ b/acqui/ordered.pl
@@ -1,7 +1,7 @@
#!/usr/bin/perl
# Copyright 2008 - 2009 BibLibre SARL
-# Copyright 2010 Catalyst IT Limited
+# Copyright 2010,2011 Catalyst IT Limited
# This file is part of Koha.
#
# Koha is free software; you can redistribute it and/or modify it under the
@@ -27,19 +27,19 @@ this script is to show orders ordered but not yet received
=cut
-
use C4::Context;
use strict;
use warnings;
use CGI;
use C4::Auth;
use C4::Output;
+use C4::Dates;
-my $dbh = C4::Context->dbh;
-my $input = new CGI;
+my $dbh = C4::Context->dbh;
+my $input = new CGI;
my $fund_id = $input->param('fund');
-my $start = $input->param('start');
-my $end = $input->param('end');
+my $start = $input->param('start');
+my $end = $input->param('end');
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
{
@@ -56,7 +56,7 @@ my $query = <<EOQ;
SELECT
aqorders.basketno, aqorders.ordernumber,
quantity-quantityreceived AS tleft,
- ecost, budgetdate,
+ ecost, budgetdate, entrydate,
aqbasket.booksellerid,
itype,
title
@@ -73,13 +73,14 @@ WHERE
(datecancellationprinted IS NULL OR
datecancellationprinted='0000-00-00') AND
(quantity > quantityreceived OR quantityreceived IS NULL)
+ GROUP BY aqorders.ordernumber
EOQ
my $sth = $dbh->prepare($query);
-$sth->execute( $fund_id);
-if ($sth->err) {
- die "Error occurred fetching records: ".$sth->errstr;
+$sth->execute($fund_id);
+if ( $sth->err ) {
+ die "Error occurred fetching records: " . $sth->errstr;
}
my @ordered;
@@ -91,18 +92,20 @@ while ( my $data = $sth->fetchrow_hashref ) {
}
if ( $left && $left > 0 ) {
my $subtotal = $left * $data->{'ecost'};
- $data->{subtotal} = sprintf ("%.2f", $subtotal);
+ $data->{subtotal} = sprintf( "%.2f", $subtotal );
$data->{'left'} = $left;
push @ordered, $data;
$total += $subtotal;
}
+ my $entrydate = C4::Dates->new( $data->{'entrydate'}, 'iso' );
+ $data->{'entrydate'} = $entrydate->output("syspref");
}
-$total = sprintf ("%.2f", $total);
-$template->param(
- ordered => \@ordered,
- total => $total
-);
+$total = sprintf( "%.2f", $total );
+
+$template->{VARS}->{'fund'} = $fund_id;
+$template->{VARS}->{'ordered'} = \@ordered;
+$template->{VARS}->{'total'} = $total;
+
$sth->finish;
-$dbh->disconnect;
output_html_with_http_headers $input, $cookie, $template->output;
diff --git a/acqui/spent.pl b/acqui/spent.pl
index 8108401..4058b59 100755
--- a/acqui/spent.pl
+++ b/acqui/spent.pl
@@ -3,7 +3,7 @@
# script to show a breakdown of committed and spent budgets
# Copyright 2002-2009 Katipo Communications Limited
-# Copyright 2010 Catalyst IT Limited
+# Copyright 2010,2011 Catalyst IT Limited
# This file is part of Koha.
#
# Koha is free software; you can redistribute it and/or modify it under the
@@ -14,7 +14,7 @@
# Koha is distributed in the hope that it will be useful, but WITHOUT ANY
# 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.,
# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
@@ -29,11 +29,12 @@ this script is designed to show the spent amount in budges
=cut
-
use C4::Context;
use C4::Auth;
use C4::Output;
+use C4::Dates;
use strict;
+use warnings;
use CGI;
my $dbh = C4::Context->dbh;
@@ -82,9 +83,9 @@ WHERE
GROUP BY aqorders.ordernumber
EOQ
my $sth = $dbh->prepare($query);
-$sth->execute( $bookfund);
-if ($sth->err) {
- die "An error occurred fetching records: ".$sth->errstr;
+$sth->execute($bookfund);
+if ( $sth->err ) {
+ die "An error occurred fetching records: " . $sth->errstr;
}
my $total = 0;
my $toggle;
@@ -92,31 +93,24 @@ my @spent;
while ( my $data = $sth->fetchrow_hashref ) {
my $recv = $data->{'quantityreceived'};
if ( $recv > 0 ) {
- my $subtotal = $recv * ($data->{'unitprice'} + $data->{'freight'});
- $data->{'subtotal'} = sprintf ("%.2f", $subtotal);
- $data->{'freight'} = sprintf ("%.2f", $data->{'freight'});
- $data->{'unitprice'} = sprintf ("%.2f", $data->{'unitprice'} );
- $total += $subtotal;
-
- if ($toggle) {
- $toggle = 0;
- }
- else {
- $toggle = 1;
- }
- $data->{'toggle'} = $toggle;
+ my $subtotal = $recv * ( $data->{'unitprice'} + $data->{'freight'} );
+ $data->{'subtotal'} = sprintf( "%.2f", $subtotal );
+ $data->{'freight'} = sprintf( "%.2f", $data->{'freight'} );
+ $data->{'unitprice'} = sprintf( "%.2f", $data->{'unitprice'} );
+ $total += $subtotal;
+ my $entrydate = C4::Dates->new( $data->{'entrydate'}, 'iso' );
+ $data->{'entrydate'} = $entrydate->output("syspref");
+ my $datereceived = C4::Dates->new( $data->{'datereceived'}, 'iso' );
+ $data->{'datereceived'} = $datereceived->output("syspref");
push @spent, $data;
}
}
-$total = sprintf ("%.2f", $total);
+$total = sprintf( "%.2f", $total );
-$template->param(
- spent => \@spent,
- total => $total
-);
-$template->{VARS}->{'fund'} = $bookfund;
+$template->{VARS}->{'fund'} = $bookfund;
+$template->{VARS}->{'spent'} = \@spent;
+$template->{VARS}->{'total'} = $total;
$sth->finish;
-$dbh->disconnect;
output_html_with_http_headers $input, $cookie, $template->output;
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/ordered.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/ordered.tt
index 7c72cb1..29e78ba 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/ordered.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/ordered.tt
@@ -6,7 +6,7 @@
[% INCLUDE 'header.inc' %]
[% INCLUDE 'acquisitions-search.inc' %]
-<div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> › <a href="/cgi-bin/koha/acqui/acqui-home.pl">Acquisitions</a> › Ordered </div>
+<div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> › <a href="/cgi-bin/koha/acqui/acqui-home.pl">Acquisitions</a> › Ordered - [% fund %]</div>
<div id="doc3" class="yui-t2">
@@ -14,18 +14,19 @@
<div id="yui-main">
<div class="yui-b">
-<h1>Budgets & Bookfunds</h1>
-<h2>Ordered</h2>
+<h1>Bookfunds</h1>
+<h2>Ordered - [% fund %]</h2>
<table cellspacing="0" cellpadding="0" border="0" id="spent" class="collapse">
<thead>
<tr>
<th> Title </th>
<th> Order </th>
+ <th> Vendor </th>
<th> Itemtype </th>
<th> Left on Order </th>
<th> Estimated cost per unit </th>
- <th> Budget Date </th>
+ <th> Date Ordered </th>
<th> Subtotal </th>
</tr>
</thead>
@@ -43,6 +44,9 @@
<a href=/cgi-bin/koha/acqui/neworderempty.pl?ordernumber=[% order.ordernumber %]&booksellerid=[% order.booksellerid %]&basketno=[% order.basketno %]">[% order.ordernumber %]</a>
</td>
<td class="cell">
+ <a href="/cgi-bin/koha/acqui/supplier.pl?supplierid=[% order.booksellerid %]">[% order.booksellerid %]</a>
+ </td>
+ <td class="cell">
[% order.itype %]
</td>
<td class="cell">
@@ -52,7 +56,7 @@
[% order.ecost %]
</td>
<td class="cell">
- [% order.budgetdate %]
+ [% order.entrydate %]
</td>
<td class="cell" align="right">
[% order.subtotal %]
@@ -68,6 +72,7 @@
<td> </td>
<td> </td>
<td> </td>
+ <td> </td>
<td align="right">
[% total %]
</td>
--
1.7.5.4
More information about the Patches
mailing list