@ngx-formly/core 6.3.9 → 6.3.11

package/fesm2020/ngx-formly-core.mjs

@@ -133,7 +133,7 @@ function clone(value) {
         isObservable(value) ||
         value instanceof TemplateRef ||
         /* instanceof SafeHtmlImpl */ value.changingThisBreaksApplicationSecurity ||
-        ['RegExp', 'FileList', 'File', 'Blob'].indexOf(value.constructor.name) !== -1) {
+        ['RegExp', 'FileList', 'File', 'Blob'].indexOf(value.constructor?.name) !== -1) {
         return value;
     }
     if (value instanceof Set) {
@@ -569,7 +569,7 @@ class FormlyFormBuilder {
                 if (field.parent && isHiddenField(field)) {
                     // when hide is used in expression set defaul value will not be set until detect hide changes
                     // which causes default value not set on new item is added
-                    options._hiddenFieldsForCheck?.push(field);
+                    options._hiddenFieldsForCheck?.push({ field, default: false });
                 }
                 options.checkExpressions?.(field, true);
                 options._detectChanges?.(field);
@@ -1114,16 +1114,15 @@ class FormlyForm {
                 }
             });
         }
-        let fieldChangesDetection = [];
+        let fieldChangesDetection = [
+            observeDeep(this.field.options, ['formState'], () => this.field.options.detectChanges(this.field)),
+        ];
         const valueChanges = this.field.options.fieldChanges
             .pipe(filter(({ field, type }) => hasKey(field) && type === 'valueChanges'), switchMap(() => (isNoopNgZone(this.ngZone) ? of(null) : this.ngZone.onStable.asObservable().pipe(take(1)))))
             .subscribe(() => this.ngZone.runGuarded(() => {
             // runGuarded is used to keep in sync the expression changes
             // https://github.com/ngx-formly/ngx-formly/issues/2095
             this.checkExpressionChange();
-            if (this.field.options) {
-                fieldChangesDetection.push(observeDeep(this.field.options, ['formState'], () => this.field.options.detectChanges(this.field)));
-            }
             this.modelChange.emit((this._modelChangeValue = clone(this.model)));
         }));
         return () => {
@@ -1555,7 +1554,7 @@ class FieldArrayType extends FieldType {
         }
         f.fieldGroup?.forEach((c) => this.markFieldForCheck(c));
         if (f.hide === false) {
-            this.options._hiddenFieldsForCheck.push(f);
+            this.options._hiddenFieldsForCheck.push({ field: f });
         }
     }
 }
@@ -1641,7 +1640,7 @@ class FieldExpressionExtension {
             defineHiddenProp(field, '_hide', !!currentValue);
             if (!firstChange || (firstChange && currentValue === true)) {
                 field.props.hidden = currentValue;
-                field.options._hiddenFieldsForCheck.push(field);
+                field.options._hiddenFieldsForCheck.push({ field });
             }
         });
         if (field.hideExpression) {
@@ -1687,8 +1686,8 @@ class FieldExpressionExtension {
                 const fieldChanged = this.checkExpressions(f, ignoreCache);
                 const options = field.options;
                 options._hiddenFieldsForCheck
-                    .sort((f) => (f.hide ? -1 : 1))
-                    .forEach((f) => this.changeHideState(f, f.hide, !ignoreCache));
+                    .sort((f) => (f.field.hide ? -1 : 1))
+                    .forEach((f) => this.changeHideState(f.field, f.field.hide ?? f.default, !ignoreCache));
                 options._hiddenFieldsForCheck = [];
                 if (fieldChanged) {
                     this.checkExpressions(field);