Bugzilla – Attachment 33373 Details for
Bug 13220
Shipment date not saved when creating an invoice which has a duplicated number
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 13220 - Shipment date not saved when creating an invoice which has a duplicated number
Bug-13220---Shipment-date-not-saved-when-creating-.patch (text/plain), 3.15 KB, created by
Jacek Ablewicz
on 2014-11-07 12:16:19 UTC
(
hide
)
Description:
Bug 13220 - Shipment date not saved when creating an invoice which has a duplicated number
Filename:
MIME Type:
Creator:
Jacek Ablewicz
Created:
2014-11-07 12:16:19 UTC
Size:
3.15 KB
patch
obsolete
>From ec18d83c0a0aaa8b982a1ffb33b19b428c1de07f Mon Sep 17 00:00:00 2001 >From: Jacek Ablewicz <abl@biblos.pk.edu.pl> >Date: Fri, 7 Nov 2014 13:04:22 +0100 >Subject: [PATCH] Bug 13220 - Shipment date not saved when creating an invoice > which has a duplicated number > >When creating an invoice which has a duplicated number, if the user chooses to 'Create new invoice anyway', previously entered shipment date (todays date by default) is not being saved, because the date value is passed to the script in the wrong format (acqui/parcels.pl expects shipmentdate parameter to be in the system-configured date format, but what it's getting in such cases is ISO-formatted date instead). As a consequence (when receiving orders from invoice whith empty shipment date) 'datereceived' field in order records are also not being populated. Here and there, Koha is using datereceived field to establish if the order was received or not received, so such not-quite-complete orders: > >- can be cancelled from the basket (even when they are de facto already received), >- it's not possible to cancel receipt of those orders from the invoice (because Koha is considering them as not yet received). > >To reproduce: > >1) Make sure you have some system date format configured in your test environment which is different from ISO format (e.g., DD/MM/YYYY) and the AcqWarnOnDuplicateInvoice syspref is enabled >2) Create some invoice with e.g. '11111' number, >3) Create another invoice with the same number (using 'Create new invoice anyway' button) >4) Try to create yet another invoice with the same number; observe that already existing invoice created in step 3) does have empty shipment date. >5) Optional: create some orders and receive them from the invoice with empty shipment date; observe that such orders are not being treated as received in all places (e.g. it's not possible to cancel receipts of such orders, and the message displayed is not in any way helpfull to determine why not). > >To test: > >1) Apply patch >2) Retest >3) Ensure that the issue is no longer reproductible, and that there are no apparent regressions of any kind. >--- > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcels.tt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcels.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcels.tt >index 5e73a24..147a26f 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcels.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcels.tt >@@ -70,7 +70,7 @@ > <input type="hidden" name="booksellerid" value="[% booksellerid %]" /> > <input type="hidden" name="op" value="confirm" /> > <input type="hidden" name="invoice" value="[% invoicenumber %]" /> >- <input type="hidden" name="shipmentdate" value="[% shipmentdate %]" /> >+ <input type="hidden" name="shipmentdate" value="[% shipmentdate | $KohaDates %]" /> > <input type="hidden" name="shipmentcost" value="[% shipmentcost %]" /> > <input type="hidden" name="shipmentcost_budgetid" value="[% shipmentcost_budgetid %]" /> > <input type="submit" class="button" value="Create new invoice anyway" /> >-- >1.7.10.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 13220
:
33373
|
33461
|
33467