From d61837144c003a30643fba19bb2fa8be42d59aed Mon Sep 17 00:00:00 2001
From: palita1991 <marianopalavecino7@gmail.com>
Date: Wed, 19 May 2021 15:57:37 -0300
Subject: [PATCH] fix(TOP): deshabilita boton hasta no cargar datos requeridos
 en formulario

---
 .../top/reglas/reglas.component.html          |  5 +++--
 .../components/top/reglas/reglas.component.ts | 21 ++++++++++++-------
 2 files changed, 17 insertions(+), 9 deletions(-)

diff --git a/src/app/components/top/reglas/reglas.component.html b/src/app/components/top/reglas/reglas.component.html
index c6858d38c5..47458cb553 100644
--- a/src/app/components/top/reglas/reglas.component.html
+++ b/src/app/components/top/reglas/reglas.component.html
@@ -133,9 +133,10 @@
                 </plex-button>
             </div>
             <div class="col text-right">
-                <plex-button type="success" label="Guardar" (click)="onSave($event)" title="Guardar">
+                <plex-button type="success" label="Guardar" (click)="onSave($event)" title="Guardar"
+                             [disabled]="form.invalid || !reglaCorrecta">
                 </plex-button>
             </div>
         </div>
     </footer>
-</form>
+</form>
\ No newline at end of file
diff --git a/src/app/components/top/reglas/reglas.component.ts b/src/app/components/top/reglas/reglas.component.ts
index f9dd8a9ba9..4b181ad5de 100644
--- a/src/app/components/top/reglas/reglas.component.ts
+++ b/src/app/components/top/reglas/reglas.component.ts
@@ -23,6 +23,7 @@ export class ReglasComponent {
     prestacionActiva = -1;
     prestacion = {};
     organizacion;
+    reglaCorrecta = false;
     constructor(
         private auth: Auth,
         private plex: Plex,
@@ -93,6 +94,7 @@ export class ReglasComponent {
     deleteOrganizacion(indice) {
         this.reglas.splice(indice, 1);
         this.reglaActiva = -1;
+        this.reglaCorrecta = false;
     }
 
     addPrestacion() {
@@ -110,6 +112,7 @@ export class ReglasComponent {
                 this.prestaciones.push({ prestacion: this.prestacionOrigen, auditable: this.auditable });
                 this.regla.origen.prestaciones = this.prestaciones;
                 this.prestacionOrigen = null;
+                this.verificarRegla();
             }
         } else {
             this.plex.info('info', 'Debe seleccionar la prestación origen');
@@ -118,12 +121,13 @@ export class ReglasComponent {
 
     deletePrestacion(indice) {
         this.regla.origen.prestaciones.splice(indice, 1);
+        this.reglaCorrecta = false;
     }
 
-
     activarRegla(indice) {
         this.reglaActiva = indice;
         this.regla = this.reglas[indice];
+        this.verificarRegla();
     }
 
     activarPrestacion(indice) {
@@ -135,18 +139,22 @@ export class ReglasComponent {
         this.reglas = [];
         this.reglaActiva = -1;
         this.regla = {};
+        this.reglaCorrecta = false;
     }
 
     cancelar() {
         this.cancel.emit();
     }
 
+    verificarRegla() {
+        if (this.reglas?.length) {
+            this.reglaCorrecta = this.reglas.every(regla => regla.destino && regla.destino.organizacion && regla.destino.prestacion && regla.origen.organizacion && regla.origen.prestaciones?.length);
+        }
+    }
+
     onSave($event) {
-        let condiciones = false;
-        if (this.reglas && this.reglas.length > 0) {
-            condiciones = this.reglas.every(regla => regla.destino && regla.destino.organizacion && regla.destino.prestacion
-                && regla.destino && regla.origen.organizacion && regla.origen.prestaciones && regla.origen.prestaciones.length > 0);
-            if (condiciones) {
+        if (this.reglas?.length) {
+            if (this.reglaCorrecta) {
                 let data = {
                     reglas: this.reglas
                 };
@@ -158,7 +166,6 @@ export class ReglasComponent {
                 });
             } else {
                 this.plex.info('info', 'Debe completar los datos de las reglas');
-
             }
         } else {
             this.servicioReglas.delete({ prestacionDestino: this.prestacionDestino.conceptId, organizacionDestino: this.organizacionDestino.id }).subscribe((resu) => {