From 82c042f80f733de9fc7dff7ad43916736fceb190 Mon Sep 17 00:00:00 2001
From: Galen Charlton <gmc@equinoxinitiative.org>
Date: Fri, 9 Jun 2017 15:06:01 -0400
Subject: [PATCH] serials: discourage create of act with null loan_duration or
 fine_level

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
---
 Open-ILS/src/templates/staff/admin/serials/t_attr_edit.tt2 | 4 ++--
 Open-ILS/web/js/ui/default/staff/admin/serials/app.js      | 7 ++++++-
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/Open-ILS/src/templates/staff/admin/serials/t_attr_edit.tt2 b/Open-ILS/src/templates/staff/admin/serials/t_attr_edit.tt2
index d400ae421b..d1be2d35e4 100644
--- a/Open-ILS/src/templates/staff/admin/serials/t_attr_edit.tt2
+++ b/Open-ILS/src/templates/staff/admin/serials/t_attr_edit.tt2
@@ -35,8 +35,8 @@
         <div class="col-md-4">
             <div class="btn-group pull-right">
                 <button class="btn btn-default" ng-click="clearWorking()" type="button">[% l('Clear') %]</button>
-                <button class="btn btn-default" ng-disabled="working.name==''" ng-click="saveTemplate()" type="button">[% l('Save') %]</label>
-                <button class="btn btn-default" ng-click="close_modal()" type="button">[% l('Close') %]</label>
+                <button class="btn btn-primary" ng-disabled="working.name=='' || working.loan_duration == null || working.fine_level == null" ng-click="saveTemplate()" type="button">[% l('Save') %]</label>
+                <button class="btn btn-warning" ng-click="close_modal()" type="button">[% l('Close') %]</label>
             </div>
         </div>
     </div>
diff --git a/Open-ILS/web/js/ui/default/staff/admin/serials/app.js b/Open-ILS/web/js/ui/default/staff/admin/serials/app.js
index 1837a33636..742309631c 100644
--- a/Open-ILS/web/js/ui/default/staff/admin/serials/app.js
+++ b/Open-ILS/web/js/ui/default/staff/admin/serials/app.js
@@ -487,11 +487,16 @@ function($scope , $q , $window , $routeParams , $location , $timeout , egCore ,
                         }
                     });
                     $scope.working.circ_lib = undefined; // special
+                    $scope.working.loan_duration = 2;
+                    $scope.working.fine_level    = 2;
                     //console.log('unsetting dirty for clearWorking');
                     $scope.dirty = false;
                 }
 
-                $scope.working = {};
+                $scope.working = {
+                    loan_duration : 2,
+                    fine_level    : 2
+                };
                 $scope.location_orgs = [];
                 $scope.location_cache = {};
             
-- 
2.11.0