Attachment 20372 Details for Bug 10736 – Bug 10736: show duplicate invoice warning on second invoice

[patch] Bug 10736: show duplicate invoice warning on second invoice

Bug-10736-show-duplicate-invoice-warning-on-second.patch (text/plain), 1.25 KB, created by Jared Camins-Esakov on 2013-08-15 15:30:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jared Camins-Esakov

Created: 2013-08-15 15:30:38 UTC

Size: 1.25 KB

patch

obsolete

>From 270276cb20bed921446d29bd74afd618bedfc48b Mon Sep 17 00:00:00 2001
>From: Jared Camins-Esakov <jcamins@cpbibliography.com>
>Date: Thu, 15 Aug 2013 11:28:30 -0400
>Subject: [PATCH] Bug 10736: show duplicate invoice warning on second invoice
>
>If you have AcqWarnOnDuplicateInvoice set to warn it only warns if you have 2
>or more invoices with the same number. It should warn if you're trying to
>create a duplicate.
>
>To test:
>1) Turn on AcqWarnOnDuplicateInvoice.
>2) Try to create an invoice that duplicates an invoice number you are already
>   using exactly once.
>3) Note that you get a warning after applying this patch.
>---
> acqui/parcels.pl |    2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/acqui/parcels.pl b/acqui/parcels.pl
>index 37a0023..fbfa9fa 100755
>--- a/acqui/parcels.pl
>+++ b/acqui/parcels.pl
>@@ -112,7 +112,7 @@ if ( $op and $op eq 'new' ) {
>             supplierid    => $booksellerid,
>             invoicenumber => $invoicenumber,
>         );
>-        if ( $#invoices > 0 ) {
>+        if ( scalar @invoices > 0 ) {
>             $template->{'VARS'}->{'duplicate_invoices'} = \@invoices;
>             $template->{'VARS'}->{'invoicenumber'}      = $invoicenumber;
>             $template->{'VARS'}->{'shipmentdate'}       = $shipmentdate;
>-- 
>1.7.9.5

Actions: View | Diff | Splinter Review

Attachments on bug 10736: 20372 | 20373 | 20391