@sumaris-net/ngx-components 2.6.30 → 2.6.32

package/fesm2020/sumaris-net.ngx-components.mjs

@@ -4930,7 +4930,8 @@ class AppFormArray extends UntypedFormArray {
         const disabled = this.disabled; // Save enabled here, because update can occur from resizeArray()
         // Reduce
         while (this.length > length) {
-            this.removeAt(this.length - 1, { emitEvent: options?.emitEvent });
+            // WARN: We use super.removeAt() and NOT this.removeAt(), to avoid infinite loop, when calling resize(0) AND options.allowEmptyArray=false
+            super.removeAt(this.length - 1, { emitEvent: options?.emitEvent });
         }
         // Or increase size
         while (this.length < length) {
@@ -4971,10 +4972,13 @@ class AppFormArray extends UntypedFormArray {
         // Do not remove if last criterion
         if (this.options.allowEmptyArray === false && this.length === 1) {
             this.clearAt(index, options);
+            return false;
         }
-        else {
+        else if (index < this.length) {
             super.removeAt(index, options);
+            return true;
         }
+        return false;
     }
     clearAt(index, options) {
         const control = this.at(index);
@@ -11327,7 +11331,7 @@ class AppForm {
         this.settings = injector.get(LocalSettingsService);
         this.errorTranslator = injector.get(FormErrorTranslator);
         this.autocompleteHelper = new MatAutocompleteConfigHolder(this.settings && {
-            getUserAttributes: (a, b) => this.settings.getFieldDisplayAttributes(a, b)
+            getUserAttributes: (a, b) => this.settings.getFieldDisplayAttributes(a, b),
         });
         this.autocompleteFields = this.autocompleteHelper.fields;
     }
@@ -11431,7 +11435,7 @@ class AppForm {
         else
             this.disable();
         this.errorTranslatorOptions = this.errorTranslatorOptions || {
-            controlPathTranslator: this
+            controlPathTranslator: this,
         };
     }
     ngOnDestroy() {
@@ -11475,7 +11479,7 @@ class AppForm {
     setForm(form) {
         if (this._form !== form) {
             this._form = form;
-            this._subscription.add(this._form.statusChanges.subscribe(_ => this.markForCheck()));
+            this._subscription.add(this._form.statusChanges.subscribe((_) => this.markForCheck()));
         }
     }
     setValue(data, opts) {
@@ -11544,7 +11548,7 @@ class AppForm {
         this.setLoading(false, opts);
     }
     markAsReady(opts) {
-        if (this.readySubject.value !== true) {
+        if (!this.readySubject.closed && this.readySubject.value !== true) {
             this.readySubject.next(true);
             // If subclasses implements OnReady
             if (typeof this['ngOnReady'] === 'function') {
@@ -11560,7 +11564,7 @@ class AppForm {
     ready(opts) {
         if (this.readySubject.value)
             return Promise.resolve();
-        return waitForTrue(this.readySubject, opts);
+        return waitForTrue(this.readySubject, { stop: this.destroySubject, ...opts });
     }
     /**
      * Wait form is not busy (=loaded)
@@ -11570,7 +11574,7 @@ class AppForm {
     waitIdle(opts) {
         if (!this.loadingSubject.value)
             return Promise.resolve();
-        return waitForFalse(this.loadingSubject, opts);
+        return waitForFalse(this.loadingSubject, { stop: this.destroySubject, ...opts });
     }
     updateValueAndValidity(opts) {
         AppFormUtils.updateValueAndValidity(this.form, opts);
@@ -11587,7 +11591,7 @@ class AppForm {
             return undefined;
         return this.errorTranslator.translateFormErrors(this.form, {
             controlPathTranslator: this,
-            ...opts
+            ...opts,
         });
     }
     getI18nFieldName(path) {
@@ -28011,7 +28015,7 @@ class AppTable {
         return !this._enabled;
     }
     markAsDirty(opts) {
-        if (this.dirtySubject.value !== true) {
+        if (!this.dirtySubject.closed && this.dirtySubject.value !== true) {
             this.dirtySubject.next(true);
             if (!opts || opts.emitEvent !== false) {
                 this.markForCheck();
@@ -28019,7 +28023,7 @@ class AppTable {
         }
     }
     markAsPristine(opts) {
-        if (this.dirtySubject.value !== false) {
+        if (!this.dirtySubject.closed && this.dirtySubject.value !== false) {
             this.dirtySubject.next(false);
             if (!opts || opts.emitEvent !== false) {
                 this.markForCheck();
@@ -28060,7 +28064,7 @@ class AppTable {
         }
     }
     markAllAsTouched(opts) {
-        if (this.touchedSubject.value !== true) {
+        if (!this.touchedSubject.closed && this.touchedSubject.value !== true) {
             this.touchedSubject.next(true);
             if (!opts || opts.emitEvent !== false)
                 this.markForCheck();
@@ -28070,7 +28074,7 @@ class AppTable {
         }
     }
     markAsSaving(opts) {
-        if (this.savingSubject.value !== true) {
+        if (!this.savingSubject.closed && this.savingSubject.value !== true) {
             this.focusColumn = undefined; // unselect focus column
             this.savingSubject.next(true);
             if (!opts || opts.emitEvent !== false)
@@ -28078,7 +28082,7 @@ class AppTable {
         }
     }
     markAsSaved(opts) {
-        if (this.savingSubject.value !== false) {
+        if (!this.savingSubject.closed && this.savingSubject.value !== false) {
             this.savingSubject.next(false);
             if (!opts || opts.emitEvent !== false)
                 this.markForCheck();
@@ -28091,7 +28095,7 @@ class AppTable {
         this.setLoading(false, opts);
     }
     markAsReady(opts) {
-        if (this.readySubject.value !== true) {
+        if (!this.readySubject.closed && this.readySubject.value !== true) {
             this.readySubject.next(true);
             // If subclasses implements OnReady
             if (typeof this['ngOnReady'] === 'function') {
@@ -28723,10 +28727,12 @@ class AppTable {
         this.dataSource.move(id, direction);
     }
     ready(opts) {
-        return waitForTrue(this.readySubject, opts);
+        return waitForTrue(this.readySubject, { stop: this.destroySubject, ...opts });
     }
     waitIdle(opts) {
-        return waitForFalse(this.loadingSubject, opts);
+        if (!this.loadingSubject.value)
+            return Promise.resolve();
+        return waitForFalse(this.loadingSubject, { stop: this.destroySubject, ...opts });
     }
     async openSelectColumnsModal(_) {
         // Copy current columns (deep copy)
@@ -29320,7 +29326,7 @@ class AppTable {
         }
     }
     setLoading(value, opts) {
-        if (this.loadingSubject.value !== value) {
+        if (!this.loadingSubject.closed && this.loadingSubject.value !== value) {
             this.loadingSubject.next(value);
             if (!opts || opts.emitEvent !== false) {
                 this.markForCheck();
@@ -30652,7 +30658,7 @@ class AppEditor {
         }
     }
     markAsLoaded(opts) {
-        if (this.loadingSubject.value) {
+        if (!this.loadingSubject.closed && this.loadingSubject.value) {
             this.loadingSubject.next(false);
             if (!opts || opts.onlySelf !== true) {
                 // Emit to children forms and editors
@@ -30667,18 +30673,18 @@ class AppEditor {
     }
     markAsReady(opts) {
         this._children?.forEach(c => c.markAsReady(opts));
-        if (!this.readySubject.value)
+        if (!this.readySubject.closed && !this.readySubject.value)
             this.readySubject.next(true);
     }
     ready(opts) {
         if (this.readySubject.value)
             return Promise.resolve();
-        return waitForTrue(this.readySubject, opts);
+        return waitForTrue(this.readySubject, { stop: this.destroySubject, ...opts });
     }
     waitIdle(opts) {
         if (!this.loadingSubject.value)
             return Promise.resolve();
-        return waitForFalse(this.loadingSubject, opts);
+        return waitForFalse(this.loadingSubject, { stop: this.destroySubject, ...opts });
     }
     markAsSaving(opts) {
         if (this.savingSubject.value !== true) {
@@ -30688,7 +30694,7 @@ class AppEditor {
         }
     }
     markAsSaved(opts) {
-        if (this.savingSubject.value !== false) {
+        if (!this.savingSubject.closed && this.savingSubject.value !== false) {
             this.savingSubject.next(false);
             if (!opts || opts.emitEvent !== false)
                 this.markForCheck();