Merge branch '2090-sms.validations' of verdnatura/salix into dev

modules/client/front/sms/index.html

@@ -8,7 +8,9 @@
                 <vn-textfield
                     vn-one
                     label="Destination"
-                    ng-model="$ctrl.sms.destination">
+                    ng-model="$ctrl.sms.destination"                    
+                    required="true"
+                    rule>
                 </vn-textfield>
             </vn-horizontal>
             <vn-horizontal >
@@ -18,6 +20,7 @@
                     ng-model="$ctrl.sms.message"
                     rows="5"
                     maxlength="160"
+                    required="true"
                     rule>
                 </vn-textarea>
             </vn-horizontal>

modules/client/front/sms/index.js

@@ -28,13 +28,24 @@ class Controller extends Component {
 
     onResponse(response) {
         if (response === 'accept') {
+            try {
+                if (!this.sms.destination)
+                    throw new Error(`The destination can't be empty`);
+                if (!this.sms.message)
+                    throw new Error(`The message can't be empty`);
+
                 this.$http.post(`Clients/${this.$params.id}/sendSms`, this.sms).then(res => {
                     this.vnApp.showMessage(this.$translate.instant('SMS sent!'));
 
                     if (res.data) this.emit('send', {response: res.data});
                 });
+            } catch (e) {
+                this.vnApp.showError(this.$translate.instant(e.message));
+                return false;
             }
         }
+        return true;
+    }
 }
 
 Controller.$inject = ['$element', '$scope', '$http', '$translate', 'vnApp'];

modules/client/front/sms/index.spec.js

@@ -30,6 +30,26 @@ describe('Client', () => {
 
                 expect(controller.vnApp.showMessage).toHaveBeenCalledWith('SMS sent!');
             });
+
+            it('should call onResponse without the destination and show an error snackbar', () => {
+                controller.sms = {destinationFk: 101, message: 'My SMS'};
+
+                spyOn(controller.vnApp, 'showError');
+
+                controller.onResponse('accept');
+
+                expect(controller.vnApp.showError).toHaveBeenCalledWith(`The destination can't be empty`);
+            });
+
+            it('should call onResponse without the message and show an error snackbar', () => {
+                controller.sms = {destinationFk: 101, destination: 222222222};
+
+                spyOn(controller.vnApp, 'showError');
+
+                controller.onResponse('accept');
+
+                expect(controller.vnApp.showError).toHaveBeenCalledWith(`The message can't be empty`);
+            });
         });
 
         describe('charactersRemaining()', () => {

modules/client/front/sms/locale/es.yml

@@ -3,3 +3,5 @@ Destination: Destinatario
 Message: Mensaje
 SMS sent!: ¡SMS enviado!
 Characters remaining: Carácteres restantes
+The destination can't be empty: El destinatario no puede estar vacio
+The message can't be empty: El mensaje no puede estar vacio

modules/ticket/front/sms/index.html

@@ -8,7 +8,9 @@
                 <vn-textfield
                     vn-one
                     label="Destination"
-                    ng-model="$ctrl.sms.destination">
+                    ng-model="$ctrl.sms.destination"
+                    required="true"
+                    rule>
                 </vn-textfield>
             </vn-horizontal>
             <vn-horizontal >
@@ -18,6 +20,7 @@
                     ng-model="$ctrl.sms.message"
                     rows="5"
                     maxlength="160"
+                    required="true"
                     rule>
                 </vn-textarea>
             </vn-horizontal>

modules/ticket/front/sms/index.js

@@ -28,13 +28,24 @@ class Controller extends Component {
 
     onResponse(response) {
         if (response === 'accept') {
+            try {
+                if (!this.sms.destination)
+                    throw new Error(`The destination can't be empty`);
+                if (!this.sms.message)
+                    throw new Error(`The message can't be empty`);
+
                 this.$http.post(`Tickets/${this.$params.id}/sendSms`, this.sms).then(res => {
                     this.vnApp.showMessage(this.$translate.instant('SMS sent!'));
 
                     if (res.data) this.emit('send', {response: res.data});
                 });
+            } catch (e) {
+                this.vnApp.showError(this.$translate.instant(e.message));
+                return false;
             }
         }
+        return true;
+    }
 }
 
 Controller.$inject = ['$element', '$scope', '$http', '$translate', 'vnApp'];

modules/ticket/front/sms/index.spec.js

@@ -29,6 +29,26 @@ describe('Ticket', () => {
 
                 expect(controller.vnApp.showMessage).toHaveBeenCalledWith('SMS sent!');
             });
+
+            it('should call onResponse without the destination and show an error snackbar', () => {
+                controller.sms = {destinationFk: 101, message: 'My SMS'};
+
+                spyOn(controller.vnApp, 'showError');
+
+                controller.onResponse('accept');
+
+                expect(controller.vnApp.showError).toHaveBeenCalledWith(`The destination can't be empty`);
+            });
+
+            it('should call onResponse without the message and show an error snackbar', () => {
+                controller.sms = {destinationFk: 101, destination: 222222222};
+
+                spyOn(controller.vnApp, 'showError');
+
+                controller.onResponse('accept');
+
+                expect(controller.vnApp.showError).toHaveBeenCalledWith(`The message can't be empty`);
+            });
         });
 
         describe('charactersRemaining()', () => {

modules/ticket/front/sms/locale/es.yml

@@ -3,3 +3,5 @@ Destination: Destinatario
 Message: Mensaje
 SMS sent!: ¡SMS enviado!
 Characters remaining: Carácteres restantes
+The destination can't be empty: El destinatario no puede estar vacio
+The message can't be empty: El mensaje no puede estar vacio