@@ -, +, @@ form should use standard formatting --- .../prog/en/modules/serials/subscription-add.tt | 42 ++++++++-------------- 1 file changed, 15 insertions(+), 27 deletions(-) --- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt @@ -77,22 +77,10 @@ function Check_page2(){ return false; } [% END %] - if($("#frequency").val().length == 0){ - alert(_("You must choose a frequency")); - return false; - } - if($("input[name='startdate']").val().length == 0){ - alert(_("You must choose a start date")); - return false; - } if($("#sublength").val().length == 0 && $("input[name='enddate']").val().length == 0){ alert(_("You must choose a subscription length or an end date.")); return false; } - if($("#numberpattern").val().length == 0){ - alert(_("You must choose a numbering pattern")); - return false; - } if(advancedpatternlocked == 0){ alert(_("You have modified the advanced prediction pattern. Please save your work or cancel modifications.")); return false; @@ -109,7 +97,7 @@ function frequencyload(){ $.getJSON("subscription-frequency.pl",{"frequency_id":document.f.frequency.value,ajax:'true'}, function(freqdata){ globalfreqdata=freqdata; - if ( globalfreqdata.unit.length == 0 ) { + if ( globalfreqdata.unit && globalfreqdata.unit.length == 0 ) { var option = $("#subtype option[value='issues']"); $(option).attr('selected', 'selected'); $("#subtype option[value!='issues']").attr('disabled', 'disabled') @@ -473,7 +461,7 @@ $(document).ready(function() {