@mediusinc/mng-commons 5.4.0-rc.1 → 5.4.0-rc.3

package/fesm2022/mediusinc-mng-commons-tableview.mjs

@@ -2340,11 +2340,10 @@ class FormEditorComponent {
             const item = this.item();
             if (item !== this.initialItemInput) {
                 // new item is different from the one the form was initialized with. Must trigger reset of form model.
-                this.resetFormModel(item);
+                // should be executed in untracked mode, because it updates multiple components underneath
+                untracked(() => this.resetFormModel(item));
             }
-        }, 
-        // resetModel uses signal like concept
-        { allowSignalWrites: true });
+        });
         effect(() => {
             this._formlyOptions.formState.disabled =
                 this.isFormDisabled() || this.autoSavePreviewDisabledForm() || this._formlyOptions.formState.descriptor?.disabled === true || this._formlyOptions.formState.details;
@@ -2471,6 +2470,7 @@ class FormEditorComponent {
         return formValue;
     }
     resetFormModel(item, opts) {
+        // have to use formly's reset model, because _form.reset() does not reset the values correctly
         this._formlyOptions.resetModel?.(this.getFormModelFromItem(item));
         if (opts?.markAsPristine !== false) {
             // mark form as pristine, should be done in next event hook to allow formly to initialize first
@@ -2588,13 +2588,13 @@ class FormEditorComponent {
         const formModel = objectDeepCopy(item, { mapGettersToProperties: true }) ?? {};
         const descriptor = this.descriptor();
         descriptor.fields.forEach(field => {
-            if (field.getter && item) {
+            if (field.getter && formModel) {
                 const splitPath = field.property.split('.');
                 let currentObject = formModel;
                 for (let i = 0; i < splitPath.length; i++) {
                     const currentSubPath = splitPath[i];
                     if (i === splitPath.length - 1) {
-                        currentObject[currentSubPath] = field.getter(item?.[field.property], item);
+                        currentObject[currentSubPath] = field.getter(formModel?.[field.property], formModel);
                     }
                     else {
                         if (typeof currentObject[currentSubPath] !== 'object') {