From 8ad1bec6b6e358516990025719f2a77c2a7bfa9c Mon Sep 17 00:00:00 2001
From: Paul POULAIN <paul@koha-fr.org>
Date: Tue, 30 Oct 2007 13:02:40 -0500
Subject: [PATCH] #1491: Subscription add does not check for all required
 fields

Signed-off-by: Chris Cormack <crc@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
---
 .../prog/en/modules/serials/subscription-add.tmpl    | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tmpl
index fda6e9c404..b925a2fd35 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tmpl
@@ -758,7 +758,11 @@ function Find_ISSN(f)
 
 
 function Check(f) {
-    if (f.startdate.value.length != 0 && f.sublength.value > 0) {
+    if (f.aqbooksellerid.value.length==0) {
+        alert(_('You must choose a bookseller'));
+    } else if (f.biblionumber.value.length==0) {
+        alert(_('You must choose or create a biblio'));
+    } else if(f.startdate.value.length != 0 && f.sublength.value > 0) {
         if (f.irreg_check.value == 1) {
             document.f.submit();
         } else {
@@ -769,7 +773,7 @@ function Check(f) {
             }
         }
     } else {
-        alert(_('Fields marked with * are mandatory'));
+        alert(_('You must choose a start date and a subscription length'));
     }
     return false;
 }
@@ -817,11 +821,11 @@ window.onload = irregular_order();
             <span class="label">Librarian: </span>            <!-- TMPL_VAR name="loggedinusername" -->
         </li>
         <li>
-            <label for="aqbooksellerid">Supplier:</label>
+            <label for="aqbooksellerid">Supplier: (*)</label>
             <input type="text" name="aqbooksellerid" id="aqbooksellerid" value="<!-- TMPL_VAR name="aqbooksellerid" -->" size="8" /> (<input type="text" name="aqbooksellername" value="<!-- TMPL_VAR name="aqbooksellername" -->" disabled="disabled" readonly="readonly" />) <div class="inputnote"><a href="#" onclick="FindAcqui(f)">Search for a Supplier</a></div>
         </li>
         <li>
-            <label for="biblionumber">Biblio</label>
+            <label for="biblionumber">Biblio: (*)</label>
             
                 <input type="text" name="biblionumber" id="biblionumber" value="<!-- TMPL_VAR name="biblionumber" -->" size="8" /> 
                 (<input type="text" name="title" value="<!-- TMPL_VAR name="bibliotitle" -->" disabled="disabled" readonly="readonly" />)
-- 
2.39.2