Acquisition minor improvements
Fixing a SQL bug hdl introduced yesterday when updating queries & that I missed when reviewing them. changing the look of the basket & parcel by adding & moving some link The goal is to have the same kind of lists & links everywhere. (Will continue on monday) Signed-off-by: Chris Cormack <crc@liblime.com> Signed-off-by: Joshua Ferraro <jmf@liblime.com>
This commit is contained in:
parent
40b8937717
commit
897de14016
3 changed files with 20 additions and 15 deletions
|
@ -823,7 +823,7 @@ sub GetParcel {
|
|||
LEFT JOIN aqbasket ON aqbasket.basketno=aqorders.basketno
|
||||
LEFT JOIN borrowers ON aqbasket.authorisedby=borrowers.borrowernumber
|
||||
WHERE
|
||||
AND aqbasket.booksellerid=?
|
||||
aqbasket.booksellerid=?
|
||||
AND aqorders.booksellerinvoicenumber LIKE \"$code\"
|
||||
AND aqorders.datereceived= \'$datereceived\'";
|
||||
|
||||
|
@ -831,13 +831,14 @@ sub GetParcel {
|
|||
my $userenv = C4::Context->userenv;
|
||||
if ( ($userenv) && ( $userenv->{flags} != 1 ) ) {
|
||||
$strsth .=
|
||||
" and (borrowers.branchcode = '"
|
||||
" AND (borrowers.branchcode = '"
|
||||
. $userenv->{branch}
|
||||
. "' or borrowers.branchcode ='')";
|
||||
. "' OR borrowers.branchcode ='')";
|
||||
}
|
||||
}
|
||||
$strsth .= " order by aqbasket.basketno";
|
||||
$strsth .= " ORDER BY aqbasket.basketno";
|
||||
### parcelinformation : $strsth
|
||||
warn "STH : $strsth";
|
||||
my $sth = $dbh->prepare($strsth);
|
||||
$sth->execute($supplierid);
|
||||
while ( my $data = $sth->fetchrow_hashref ) {
|
||||
|
|
|
@ -69,7 +69,7 @@
|
|||
<td><!-- TMPL_VAR NAME="publishercode" --></td>
|
||||
<!-- TMPL_IF name="closedate" -->
|
||||
<td><p>
|
||||
<!-- TMPL_VAR NAME="title" --><br />
|
||||
<a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber="<!-- TMPL_VAR name="biblionumber" -->"><!-- TMPL_VAR NAME="title" --></a><br />
|
||||
<!-- TMPL_VAR NAME="author" -->
|
||||
<!-- TMPL_IF name="isbn"-->- <!-- TMPL_VAR name="isbn" --><!-- /TMPL_IF -->
|
||||
<!-- TMPL_IF name="notes" --></p><p><!--TMPL_VAR name="notes" --><!-- /TMPL_IF -->
|
||||
|
@ -78,7 +78,7 @@
|
|||
<!-- TMPL_ELSE -->
|
||||
<td>
|
||||
<p>
|
||||
<!-- TMPL_VAR NAME="title" --></p>
|
||||
<a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=<!-- TMPL_VAR name="biblionumber" -->"><!-- TMPL_VAR NAME="title" --></a></p>
|
||||
<p><!-- TMPL_VAR NAME="author" -->
|
||||
<!-- TMPL_IF name="isbn"-->- <!-- TMPL_VAR name="isbn" --><!-- /TMPL_IF -->
|
||||
<!-- TMPL_IF name="notes" --></p><p><!--TMPL_VAR name="notes" --><!-- /TMPL_IF -->
|
||||
|
|
|
@ -41,14 +41,13 @@
|
|||
<th>QTY ordered</th>
|
||||
<th>Est Cost</th>
|
||||
<th>ACTUAL</th>
|
||||
<th>P&P</th>
|
||||
<th>QTY received</th>
|
||||
<th>TOTAL</th>
|
||||
</tr>
|
||||
<!-- TMPL_LOOP NAME="loop_received" -->
|
||||
<tr>
|
||||
<td><!-- TMPL_VAR NAME="basketno" --></td>
|
||||
<td><a href="orderreceive.pl?recieve=<!-- TMPL_VAR NAME="ordernumber" -->&biblionumber=<!-- TMPL_VAR NAME="biblionumber" -->&daterecieved=<!-- TMPL_VAR NAME="datereceived" -->&invoice=<!-- TMPL_VAR NAME="invoice" -->&gst=<!-- TMPL_VAR NAME="gst" -->&freight=<!-- TMPL_VAR NAME="freight" -->&supplierid=<!-- TMPL_VAR NAME="supplierid" -->"><!-- TMPL_VAR NAME="title" --></a>
|
||||
<td><a href="/cgi-bin/koha/acqui/basket.pl?basketno=<!-- TMPL_VAR NAME="basketno" -->"><!-- TMPL_VAR NAME="basketno" --></a></td>
|
||||
<td><a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=<!-- TMPL_VAR name="biblionumber" -->"><!-- TMPL_VAR NAME="title" --></a>
|
||||
<!-- TMPL_IF NAME="isbn" --><br />ISBN: <!-- TMPL_VAR NAME="isbn" --><!--/TMPL_IF-->
|
||||
<!-- TMPL_IF NAME="author" --><br />author :<!-- TMPL_VAR NAME="author" --><!--/TMPL_IF-->
|
||||
<!-- TMPL_IF NAME="publishercode" --><br />Publisher :<!-- TMPL_VAR NAME="publishercode" --><!--/TMPL_IF-->
|
||||
|
@ -56,8 +55,7 @@
|
|||
<td><!-- TMPL_VAR NAME="quantity" --></td>
|
||||
<td><!-- TMPL_VAR NAME="ecost" --></td>
|
||||
<td><!-- TMPL_VAR NAME="unitprice" --></td>
|
||||
<td> </td>
|
||||
<td><!-- TMPL_VAR NAME="quantityreceived" --></td>
|
||||
<td><a href="orderreceive.pl?recieve=<!-- TMPL_VAR NAME="ordernumber" -->&biblionumber=<!-- TMPL_VAR NAME="biblionumber" -->&daterecieved=<!-- TMPL_VAR NAME="datereceived" -->&invoice=<!-- TMPL_VAR NAME="invoice" -->&gst=<!-- TMPL_VAR NAME="gst" -->&freight=<!-- TMPL_VAR NAME="freight" -->&supplierid=<!-- TMPL_VAR NAME="supplierid" -->"><!-- TMPL_VAR NAME="quantityreceived" --></a></td>
|
||||
<td><!-- TMPL_VAR NAME="total" --></td>
|
||||
</tr>
|
||||
<!-- /TMPL_LOOP -->
|
||||
|
@ -100,21 +98,27 @@
|
|||
<th>Summary</th>
|
||||
<th>Order qty</th>
|
||||
<th>Order cost</th>
|
||||
<th>Recieved qty</th>
|
||||
<th>Already Recieved </th>
|
||||
<th>Recieved cost</th>
|
||||
</tr>
|
||||
<!-- TMPL_LOOP NAME="loop_orders" -->
|
||||
<tr>
|
||||
<td><!-- TMPL_VAR NAME="basketno" --></td>
|
||||
<td><a href="/cgi-bin/koha/acqui/basket.pl?basketno=<!-- TMPL_VAR NAME="basketno" -->"><!-- TMPL_VAR NAME="basketno" --></a></td>
|
||||
<td>
|
||||
<a href="orderreceive.pl?recieve=<!-- TMPL_VAR NAME="ordernumber" -->&biblionumber=<!-- TMPL_VAR NAME="biblionumber" -->&daterecieved=<!-- TMPL_VAR NAME="datereceived" -->&invoice=<!-- TMPL_VAR NAME="invoice" -->&gst=<!-- TMPL_VAR NAME="gst" -->&freight=<!-- TMPL_VAR NAME="freight" -->&supplierid=<!-- TMPL_VAR NAME="supplierid" -->"><!-- TMPL_VAR NAME="title" --></a>
|
||||
<a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=<!-- TMPL_VAR name="biblionumber" -->"><!-- TMPL_VAR NAME="title" --></a>
|
||||
<!-- TMPL_VAR NAME="title" --></a>
|
||||
<!-- TMPL_IF NAME="isbn" --><br />ISBN: <!-- TMPL_VAR NAME="isbn" --><!--/TMPL_IF-->
|
||||
<!-- TMPL_IF NAME="author" --><br />author :<!-- TMPL_VAR NAME="author" --><!--/TMPL_IF-->
|
||||
<!-- TMPL_IF NAME="publishercode" --><br />Publisher :<!-- TMPL_VAR NAME="publishercode" --><!--/TMPL_IF-->
|
||||
</td>
|
||||
<td><!-- TMPL_VAR NAME="quantity" --></td>
|
||||
<td><!-- TMPL_VAR NAME="ecost" --></td>
|
||||
<td><!-- TMPL_VAR NAME="quantityreceived" --></td>
|
||||
<td><!-- TMPL_IF name="quantityreceived" -->
|
||||
<!-- TMPL_VAR NAME="quantityreceived" --> <a href="orderreceive.pl?recieve=<!-- TMPL_VAR NAME="ordernumber" -->&biblionumber=<!-- TMPL_VAR NAME="biblionumber" -->&daterecieved=<!-- TMPL_VAR NAME="datereceived" -->&invoice=<!-- TMPL_VAR NAME="invoice" -->&gst=<!-- TMPL_VAR NAME="gst" -->&freight=<!-- TMPL_VAR NAME="freight" -->&supplierid=<!-- TMPL_VAR NAME="supplierid" -->">More</a>
|
||||
<!-- TMPL_ELSE -->
|
||||
<a href="orderreceive.pl?recieve=<!-- TMPL_VAR NAME="ordernumber" -->&biblionumber=<!-- TMPL_VAR NAME="biblionumber" -->&daterecieved=<!-- TMPL_VAR NAME="datereceived" -->&invoice=<!-- TMPL_VAR NAME="invoice" -->&gst=<!-- TMPL_VAR NAME="gst" -->&freight=<!-- TMPL_VAR NAME="freight" -->&supplierid=<!-- TMPL_VAR NAME="supplierid" -->">Recieve</a>
|
||||
<!-- /TMPL_IF -->
|
||||
</td>
|
||||
<td><!-- TMPL_IF EXPR="unitprice ne '0.00'" --><!-- TMPL_VAR NAME="unitprice" --><!-- /TMPL_IF --></td>
|
||||
</tr>
|
||||
<!-- /TMPL_LOOP -->
|
||||
|
|
Loading…
Reference in a new issue