Merged PR 1865: feat(oms-data-access, oms-return-review, oms-return-summary): fix return receipt mapping and ensure process completion

feat(oms-data-access, oms-return-review, oms-return-summary): fix return receipt mapping and ensure process completion

Corrects the mapping of return receipts in the return process flow to ensure that the correct receipt IDs are used when printing and finalizing returns. Updates the `finishProcess` method to associate return receipts with the correct entities and ensures the store is updated after process completion. This resolves issues where printed receipts or review steps could reference incorrect or missing data.

Ref: #5120

libs/oms/data-access/src/lib/stores/return-process.store.ts

@@ -182,8 +182,8 @@ export const ReturnProcessStore = signalStore(
       store.storeState();
     },
     finishProcess: (returnReceipts: { [id: number]: Receipt }) => {
-      const entities = store.entities().map((entity) => {
-        const receipt = returnReceipts[entity.receiptId];
+      const entities = store.entities().map((entity, i) => {
+        const receipt = returnReceipts[i];
         if (receipt) {
           return { ...entity, returnReceipt: receipt };
         }

libs/oms/feature/return-review/src/lib/return-review.component.ts

@@ -22,8 +22,8 @@ export class ReturnReviewComponent {
     if (processId) {
       const receiptIds = this.#returnProcessStore
         .entities()
-        .filter((p) => p.processId === processId && p.receiptId)
-        .map((p) => p.receiptId);
+        .filter((p) => p.processId === processId && p.returnReceipt?.id)
+        .map((p) => p.returnReceipt!.id);
 
       await this.#printReceiptsService.printReturnReceipts({
         returnReceiptIds: receiptIds,

libs/oms/feature/return-summary/src/lib/return-summary.component.ts

@@ -113,6 +113,7 @@ export class ReturnSummaryComponent {
       }));
       this.returnItemsAndPrintReciptStatus.set('success');
 
+      this.#returnProcessStore.finishProcess(returnReceipts);
       await this.#router.navigate(['../', 'review'], {
         relativeTo: this.#activatedRoute,
       });