fix: Redux to update files properly after removing from UI

Author: tammy-a11y

packages/angular-sdk-components/src/lib/_components/widget/attachment/AttachmentUtils.ts

@@ -126,8 +126,7 @@ export const updateReduxState = (
   valueRef: string,
   options: PageInstructionOptions
 ) => {
-  const { allowMultiple, isOldAttachment, insertRedux, deleteRedux } = options;
-  let deleteIndex = -1;
+  const { allowMultiple, isOldAttachment, insertRedux, deleteRedux, deleteIndex } = options;
 
   if (allowMultiple || isOldAttachment) {
     transformedAttachments.forEach(attachment => {
@@ -147,11 +146,6 @@ export const updateReduxState = (
         };
         PCore.getStore()?.dispatch(actionPayLoad);
       } else if (deleteRedux) {
-        const uniqueKey = getMappedValue('pzInsKey');
-        deleteIndex = existingAttachments.findIndex(
-          existingAttachment =>
-            existingAttachment[uniqueKey as keyof ReduxAttachments] === transformedAttachments[0][uniqueKey as keyof ReduxAttachments]
-        );
         const actionPayLoad = {
           type: 'LIST_ACTION',
           payload: {

packages/angular-sdk-components/src/lib/_components/widget/attachment/attachment.component.ts

@@ -339,6 +339,12 @@ export class AttachmentComponent implements OnInit, OnDestroy {
 
   populateErrorAndUpdateRedux(file) {
     const fieldName = (this.pConn$.getStateProps() as any).value;
+    insertAttachments([file], this.pConn$, this.multiAttachmentsInInlineEdit, {
+      allowMultiple: this.allowMultiple$,
+      isOldAttachment: this.isOldAttachment,
+      isMultiAttachmentInInlineEditTable: this.isMultiAttachmentInInlineEditTable,
+      attachmentCount: this.attachmentCount
+    } as any);
     // set errors to property to block submit even on errors in file upload
     PCore.getMessageManager().addMessages({
       messages: [
@@ -351,12 +357,6 @@ export class AttachmentComponent implements OnInit, OnDestroy {
       pageReference: this.pConn$.getPageReference(),
       context: this.contextName
     });
-    insertAttachments([file], this.pConn$, this.multiAttachmentsInInlineEdit, {
-      allowMultiple: this.allowMultiple$,
-      isOldAttachment: this.isOldAttachment,
-      isMultiAttachmentInInlineEditTable: this.isMultiAttachmentInInlineEditTable,
-      attachmentCount: this.attachmentCount
-    } as any);
   }
 
   errorHandler(isFetchCanceled, file) {