From 7bb8704e2d71e3db4a5b2df6d69910a4baaf5811 Mon Sep 17 00:00:00 2001
From: Mathieu Saby <mathsabypro@gmail.com>
Date: Thu, 17 Apr 2014 20:28:15 +0200
Subject: [PATCH 3/3] Followup : update the receipt page (parcel.tt)
Content-Type: text/plain; charset="utf-8"

In main patch, I forgot to made some changes in the parcel.tt page
This patch fixes that bu giving the librarian the ability to cancel an order in receipt page if the record is deleted.
It also displays the message "Deleted bibliographic record, can't find title" in Summary

Note that this behavior will be improved by bug 10758

To test : 
- create a basket and an order
- delete the record used by that order
- go on receipt page
- in the line matching this order, you should see instead of the title "Deleted bibliographic record, can't find title"
- click on "Delete order" : you should be able to delete the order
- go to the basket, to check the order was properly cancelled
- in receipt page, try to delete an other order (always linked to a record), to be sure there is no regression


---
 .../intranet-tmpl/prog/en/modules/acqui/parcel.tt  |   10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt
index b537148..3b0fc46 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt
@@ -265,7 +265,11 @@
                 </td>
                 <td class="orderfilterclass"><a href="neworderempty.pl?ordernumber=[% loop_order.ordernumber %]&amp;booksellerid=[% loop_order.booksellerid %]">[% loop_order.ordernumber %]</a></td>
                 <td class="summaryfilterclass">
-                  <a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% loop_order.biblionumber %]">[% loop_order.title |html %]</a>
+                    [% IF (books_loo.biblionumber) %]
+                        <a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% loop_order.biblionumber %]">[% loop_order.title |html %]</a>
+                    [% ELSE %]
+                        <em>Deleted bibliographic record, can't find title</em><br />
+                    [% END %]
                 [% IF ( loop_order.author ) %] by [% loop_order.author %][% END %]
                 [% IF ( loop_order.isbn ) %] &ndash; [% loop_order.isbn %][% END %]
                 [% IF ( loop_order.publishercode ) %]<br />Publisher :[% loop_order.publishercode %][% END %]
@@ -292,6 +296,9 @@
                     <a href="#" onclick="transfer_order_popup([% loop_order.ordernumber %]); return false;">Transfer</a>
 				</td>
 				<td>
+                    [% UNLESS (books_loo.biblionumber) %]
+                        <a href="javascript:confirm_delete_item([% loop_order.ordernumber %], [% loop_order.basketno %])" class="button">Delete order</a><br>
+                    [% ELSE %]
                         [% IF ( loop_order.left_holds_on_order ) %]
                         <span class="button" title="Can't delete order, ([% loop_order.holds_on_order %]) holds are linked with this order cancel holds first">Can't delete order</span><br>
                         [% ELSE %]
@@ -314,6 +321,7 @@
                         [% IF ( loop_order.left_holds ) %]
                         <b title="Can't delete catalog record or order, cancel holds first">[% loop_order.holds %] hold(s) left</b>
                         [% END %]
+                    [% END %]
                     </td>
                 </tr>
         [% END %]
-- 
1.7.9.5