From 94036d849054ba7167fd7c06a86106e12606e3aa Mon Sep 17 00:00:00 2001
From: Ryan Higgins <rch@liblime.com>
Date: Tue, 1 Jan 2008 15:36:34 -0600
Subject: [PATCH] Fix for bug 1251 (blank supplier name)

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
---
 acqui/updatesupplier.pl                           | 15 +++++++++------
 .../prog/en/modules/acqui/supplier.tmpl           | 15 +++++++++++----
 2 files changed, 20 insertions(+), 10 deletions(-)

diff --git a/acqui/updatesupplier.pl b/acqui/updatesupplier.pl
index 2a10238045..419cc9e0a6 100755
--- a/acqui/updatesupplier.pl
+++ b/acqui/updatesupplier.pl
@@ -85,11 +85,14 @@ $data{'gstreg'}=$input->param('gst');
 $data{'listincgst'}=$input->param('list_gst');
 $data{'invoiceincgst'}=$input->param('invoice_gst');
 $data{'discount'}=$input->param('discount');
-if ($data{'id'}){
-    ModBookseller(\%data);
-} else {
-    $data{id}=AddBookseller(\%data);
-}
-
+if($data{'name'}) {
+	if ($data{'id'}){
+	    ModBookseller(\%data);
+	} else {
+	    $data{id}=AddBookseller(\%data);
+	}
 #redirect to booksellers.pl
 print $input->redirect("booksellers.pl?supplier=".$data{id});
+} else {
+print $input->redirect("supplier.pl?op=enter"); # fail silently.
+}
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tmpl
index 44aa598bd2..6a1064e7e4 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/supplier.tmpl
@@ -8,6 +8,13 @@ function confirm_deletion() {
         window.location="/cgi-bin/koha/acqui/supplier.pl?supplierid=<!--TMPL_VAR Name="id"-->&op=delete";
     }
 }
+function check(f) {
+if (f.company.value == "") {
+    alert("You must specify a name for this supplier.");
+    return false;
+}
+    f.submit();
+}
 //]]>
 </script>
 </head>
@@ -30,13 +37,13 @@ function confirm_deletion() {
     <!-- /TMPL_IF -->
 <!-- TMPL_UNLESS NAME="enter" --><!-- TMPL_INCLUDE NAME="acquisitions-toolbar.inc" --><!-- /TMPL_UNLESS -->
 <!--TMPL_IF Name="enter"-->
-    <form action="updatesupplier.pl" method="post">
+    <form action="updatesupplier.pl" name="updatesupplier" method="post">
 	<div class="yui-g">
 	<div class="yui-u first">
         <input type="hidden" name="id" value="<!-- TMPL_VAR NAME="id" -->" />
         <fieldset class="rows">
             <legend>Company details</legend>
-            <ol><li><label for="company">Company Name</label>
+            <ol><li><label for="company" class="required">Company Name * </label>
                 <input type="text" size="40" id="company" name="company" value="<!-- TMPL_VAR NAME="name" -->" /></li>
             <li><label for="company_postal">Postal Address</label>
                     <textarea id="company_postal" name="company_postal" cols="40" rows="3"><!-- TMPL_VAR NAME="postal" --></textarea></li>
@@ -133,7 +140,7 @@ function confirm_deletion() {
 				<input type="hidden"  name="invoice_gst" value="0" />
 			<!-- /TMPL_UNLESS -->
         </fieldset>
-        <fieldset class="action"><input type="submit" value="Save" /> <a class="cancel" href="/cgi-bin/koha/acqui/supplier.pl?supplierid=<!-- TMPL_VAR NAME="id" -->">Cancel</a></fieldset>
+        <fieldset class="action"><input type="button" value="Save" onclick="check(this.form);" /> <a class="cancel" href="/cgi-bin/koha/acqui/supplier.pl?supplierid=<!-- TMPL_VAR NAME="id" -->">Cancel</a></fieldset>
 		</div>
     </form>
 <!--TMPL_ELSE-->
@@ -200,4 +207,4 @@ function confirm_deletion() {
 
 </div>
 </div>
-<!-- TMPL_INCLUDE NAME="intranet-bottom.inc" -->
\ No newline at end of file
+<!-- TMPL_INCLUDE NAME="intranet-bottom.inc" -->
-- 
2.39.5