LP1904036 SMS number / carrier required if SMS enabled

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Jane Sandberg <js7389@princeton.edu>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>

diff --git a/Open-ILS/src/eg2/src/app/staff/circ/patron/edit.component.html b/Open-ILS/src/eg2/src/app/staff/circ/patron/edit.component.html
index c46961e..48dc650 100644 (file)
--- a/Open-ILS/src/eg2/src/app/staff/circ/patron/edit.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/circ/patron/edit.component.html
@@ -147,6 +147,7 @@
       id="cust-{{args.settingName}}-input"
       [ngModel]="userSettings[args.settingName]"
       (ngModelChange)="userSettingChange(args.settingName, $event)"
+      [required]="settingFieldRequired(args.settingName)"
       [disabled]="args.disabled"
     />
   </div>
@@ -570,6 +571,7 @@
           name="cust-opac.default_sms_carrier-input"
           domId="cust-opac.default_sms_carrier-input"
           [selectedId]="userSettings['opac.default_sms_carrier']"
+          [required]="settingFieldRequired('opac.default_sms_carrier')"
           (onChange)="userSettingChange(
             'opac.default_sms_carrier', $event ? $event.id : null)">
         </eg-combobox>

diff --git a/Open-ILS/src/eg2/src/app/staff/circ/patron/edit.component.ts b/Open-ILS/src/eg2/src/app/staff/circ/patron/edit.component.ts
index c95260d..15d29b3 100644 (file)
--- a/Open-ILS/src/eg2/src/app/staff/circ/patron/edit.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/circ/patron/edit.component.ts
@@ -1145,6 +1145,16 @@ export class EditComponent implements OnInit, AfterViewInit {
         return this.fieldVisibility[field] === 3;
     }
 
+    settingFieldRequired(name: string): boolean {
+
+        switch (name) {
+            case 'opac.default_sms_notify':
+            case 'opac.default_sms_carrier':
+                return this.holdNotifyTypes.sms;
+        }
+
+        return false;
+    }
 
     fieldPattern(idlClass: string, field: string): RegExp {
         if (!this.fieldPatterns[idlClass][field]) {