Attachment 50836 Details for Bug 16351 – [PASSED QA] Bug 16351: Making invoice number a required field

[patch] [PASSED QA] Bug 16351: Making invoice number a required field

PASSED-QA-Bug-16351-Making-invoice-number-a-requir.patch (text/plain), 2.00 KB, created by Katrin Fischer on 2016-04-27 16:56:22 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2016-04-27 16:56:22 UTC

Size: 2.00 KB

patch

obsolete

>From 38cb5a9697b3032ada01d7cc51e4621237d0272d Mon Sep 17 00:00:00 2001
>From: Aleisha <aleishaamohia@hotmail.com>
>Date: Tue, 26 Apr 2016 01:27:19 +0000
>Subject: [PATCH] [PASSED QA] Bug 16351: Making invoice number a required field
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>EDIT: Added 'validated' class to form and removed 'Required' span (became unnecessary)
>
>To test:
>1) Go to Acqui -> Do a vendor search
>2) Receive a shipment
>3) Attempt to click 'Next' without specifying invoice number. Notice error.
>4) Go back to vendor search then apply patch. Click Receive shipment
>5) Notice you cannot click 'Next' until you put something in invoice number.
>
>Sponsored-by: Catalyst IT
>
>Signed-off-by: Marc VÃ©ron <veron@veron.ch>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcels.tt | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
>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 613098d..8c38139 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcels.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcels.tt
>@@ -146,14 +146,14 @@
> [% END %]
> 
>     <div id="parcels_new_parcel">
>-        <form method="get" action="parcels.pl">
>+        <form method="get" action="parcels.pl" class="validated">
>     <fieldset class="rows">
>     <legend>Receive a new shipment</legend>
>        <ol> <li>
>-            <label for="invoice">Vendor invoice </label>
>+            <label for="invoice" class="required">Vendor invoice:</label>
>             <input type="hidden" name="booksellerid" value="[% booksellerid %]" />
>             <input type="hidden" name="op" value="new" />
>-			<input type="text" size="20" id="invoice" name="invoice" class="focus" />
>+            <input type="text" size="20" id="invoice" name="invoice" class="focus required" required="required" />
>         </li>
> 		[% IF ( gst ) %]
>         <li>
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 16351: 50692 | 50772 | 50816 | 50836