@reltio/components 1.4.2207 → 1.4.2208

package/cjs/features/workflow/hooks/useApplyDcrChanges.js

@@ -0,0 +1,145 @@
+"use strict";
+var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
+    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
+    return new (P || (P = Promise))(function (resolve, reject) {
+        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
+        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
+        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
+        step((generator = generator.apply(thisArg, _arguments || [])).next());
+    });
+};
+var __generator = (this && this.__generator) || function (thisArg, body) {
+    var _ = { label: 0, sent: function() { if (t[0] & 1) throw t[1]; return t[1]; }, trys: [], ops: [] }, f, y, t, g = Object.create((typeof Iterator === "function" ? Iterator : Object).prototype);
+    return g.next = verb(0), g["throw"] = verb(1), g["return"] = verb(2), typeof Symbol === "function" && (g[Symbol.iterator] = function() { return this; }), g;
+    function verb(n) { return function (v) { return step([n, v]); }; }
+    function step(op) {
+        if (f) throw new TypeError("Generator is already executing.");
+        while (g && (g = 0, op[0] && (_ = 0)), _) try {
+            if (f = 1, y && (t = op[0] & 2 ? y["return"] : op[0] ? y["throw"] || ((t = y["return"]) && t.call(y), 0) : y.next) && !(t = t.call(y, op[1])).done) return t;
+            if (y = 0, t) op = [op[0] & 2, t.value];
+            switch (op[0]) {
+                case 0: case 1: t = op; break;
+                case 4: _.label++; return { value: op[1], done: false };
+                case 5: _.label++; y = op[1]; op = [0]; continue;
+                case 7: op = _.ops.pop(); _.trys.pop(); continue;
+                default:
+                    if (!(t = _.trys, t = t.length > 0 && t[t.length - 1]) && (op[0] === 6 || op[0] === 2)) { _ = 0; continue; }
+                    if (op[0] === 3 && (!t || (op[1] > t[0] && op[1] < t[3]))) { _.label = op[1]; break; }
+                    if (op[0] === 6 && _.label < t[1]) { _.label = t[1]; t = op; break; }
+                    if (t && _.label < t[2]) { _.label = t[2]; _.ops.push(op); break; }
+                    if (t[2]) _.ops.pop();
+                    _.trys.pop(); continue;
+            }
+            op = body.call(thisArg, _);
+        } catch (e) { op = [6, e]; y = 0; } finally { f = t = 0; }
+        if (op[0] & 5) throw op[1]; return { value: op[0] ? op[1] : void 0, done: true };
+    }
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.useApplyDcrChanges = void 0;
+var react_1 = require("react");
+var react_context_selector_1 = require("@fluentui/react-context-selector");
+var mdm_sdk_1 = require("@reltio/mdm-sdk");
+var ramda_1 = require("ramda");
+var MdmModuleContext_1 = require("../../../contexts/MdmModuleContext");
+var ChangeRequestEditorContext_1 = require("../contexts/ChangeRequestEditorContext");
+var dcr_1 = require("../helpers/dcr");
+var withChangeRequestId = function (dcrUri) { return function (url) {
+    return url + (url.includes('?') ? '&' : '?') + "changeRequestId=".concat(encodeURIComponent(dcrUri.split('/')[1]));
+}; };
+var useApplyDcrChanges = function (dcr) {
+    var getLinesByChangeId = (0, react_context_selector_1.useContextSelector)(ChangeRequestEditorContext_1.ChangeRequestEditorContext, function (context) { return context.getLinesByChangeId; });
+    var metadata = (0, MdmModuleContext_1.useMdmMetadata)();
+    var applyDcrChanges = (0, react_1.useCallback)(function () { return __awaiter(void 0, void 0, void 0, function () {
+        var _a, rejectedChangeIds, updatedChanges, deleteEntityAttributes, deleteRelationAttributes, deleteChangeResults, deleteEntityAttributesResults, deleteRelationAttributesResults, deleteChangeFailures, error_1, deleteEntityAttributesFailures, deleteRelationAttributesFailures;
+        return __generator(this, function (_b) {
+            switch (_b.label) {
+                case 0:
+                    if (!(dcr === null || dcr === void 0 ? void 0 : dcr.changes)) return [3 /*break*/, 11];
+                    _a = (0, dcr_1.buildDCRSavePlan)(dcr.changes, getLinesByChangeId, metadata), rejectedChangeIds = _a.rejectedChangeIds, updatedChanges = _a.updatedChanges, deleteEntityAttributes = _a.deleteEntityAttributes, deleteRelationAttributes = _a.deleteRelationAttributes;
+                    deleteChangeResults = [];
+                    deleteEntityAttributesResults = [];
+                    deleteRelationAttributesResults = [];
+                    if (!(rejectedChangeIds.length > 0)) return [3 /*break*/, 2];
+                    return [4 /*yield*/, (0, mdm_sdk_1.promiseAllSettled)(rejectedChangeIds.map(function (changeId) {
+                            return (0, mdm_sdk_1.deleteChangeRequestItem)({ dcrUri: dcr.uri, changeItemId: changeId });
+                        }))];
+                case 1:
+                    deleteChangeResults = _b.sent();
+                    _b.label = 2;
+                case 2:
+                    deleteChangeFailures = deleteChangeResults.filter(function (result) { return result.status === 'rejected'; });
+                    if (deleteChangeFailures.length) {
+                        return [2 /*return*/, {
+                                proceed: false,
+                                error: new Error("Failed to delete DCR change(s): ".concat(deleteChangeFailures.map(function (result) { return result.reason; }).join(', ')))
+                            }];
+                    }
+                    if (!!(0, ramda_1.isEmpty)(updatedChanges)) return [3 /*break*/, 6];
+                    _b.label = 3;
+                case 3:
+                    _b.trys.push([3, 5, , 6]);
+                    return [4 /*yield*/, (0, mdm_sdk_1.updateChangeRequest)({ dcrUri: dcr.uri, payload: updatedChanges })];
+                case 4:
+                    _b.sent();
+                    return [3 /*break*/, 6];
+                case 5:
+                    error_1 = _b.sent();
+                    return [2 /*return*/, {
+                            proceed: false,
+                            error: error_1
+                        }];
+                case 6:
+                    if (!!(0, ramda_1.isEmpty)(deleteEntityAttributes)) return [3 /*break*/, 8];
+                    return [4 /*yield*/, (0, mdm_sdk_1.promiseAllSettled)(Object.entries(deleteEntityAttributes).map(function (_a) {
+                            var entityUri = _a[0], diff = _a[1];
+                            return (0, mdm_sdk_1.entityCumulativeUpdate)({
+                                uri: entityUri,
+                                diff: diff,
+                                uriPreprocessor: withChangeRequestId(dcr.uri)
+                            });
+                        }))];
+                case 7:
+                    deleteEntityAttributesResults = _b.sent();
+                    _b.label = 8;
+                case 8:
+                    deleteEntityAttributesFailures = deleteEntityAttributesResults.filter(function (result) { return result.status === 'rejected'; });
+                    if (deleteEntityAttributesFailures.length) {
+                        return [2 /*return*/, {
+                                proceed: false,
+                                error: new Error("Failed to delete entity attributes: ".concat(deleteEntityAttributesFailures.map(function (result) { return result.reason; }).join(', ')))
+                            }];
+                    }
+                    if (!!(0, ramda_1.isEmpty)(deleteRelationAttributes)) return [3 /*break*/, 10];
+                    return [4 /*yield*/, (0, mdm_sdk_1.promiseAllSettled)(Object.entries(deleteRelationAttributes).map(function (_a) {
+                            var relationUri = _a[0], diff = _a[1];
+                            return (0, mdm_sdk_1.updateRelationWithDiff)({
+                                uri: relationUri,
+                                diff: diff,
+                                uriPreprocessor: withChangeRequestId(dcr.uri)
+                            });
+                        }))];
+                case 9:
+                    deleteRelationAttributesResults = _b.sent();
+                    _b.label = 10;
+                case 10:
+                    deleteRelationAttributesFailures = deleteRelationAttributesResults.filter(function (result) { return result.status === 'rejected'; });
+                    if (deleteRelationAttributesFailures.length) {
+                        return [2 /*return*/, {
+                                proceed: false,
+                                error: new Error("Failed to delete relation attributes: ".concat(deleteRelationAttributesFailures.map(function (result) { return result.reason; }).join(', ')))
+                            }];
+                    }
+                    return [2 /*return*/, {
+                            proceed: true
+                        }];
+                case 11: return [2 /*return*/, {
+                        proceed: false,
+                        error: new Error('No changes to apply')
+                    }];
+            }
+        });
+    }); }, [dcr, getLinesByChangeId, metadata]);
+    return applyDcrChanges;
+};
+exports.useApplyDcrChanges = useApplyDcrChanges;

package/cjs/features/workflow/hooks/useWorkflowActions.d.ts

@@ -1,7 +1,12 @@
 import { WorkflowTaskData } from '@reltio/mdm-sdk';
-import { TaskAction } from '../types';
+import { BeforeActionOutcome, TaskAction } from '../types';
 type Task = Partial<Pick<WorkflowTaskData, 'taskId' | 'possibleActions' | 'assignee' | 'isOpen'>>;
-export declare const useWorkflowActions: (task: Task, onActionSuccess?: (action: string) => void) => {
+type Props = {
+    task: Task;
+    onActionSuccess?: (action: string) => void;
+    beforeAction?: () => Promise<BeforeActionOutcome>;
+};
+export declare const useWorkflowActions: ({ task, onActionSuccess, beforeAction }: Props) => {
     actions: TaskAction[];
     actionRequestIsInProgress: boolean;
 };

package/cjs/features/workflow/hooks/useWorkflowActions.js

@@ -44,23 +44,25 @@ var common_1 = require("../helpers/common");
 var WorkflowTasksContext_1 = require("../../../contexts/WorkflowTasksContext");
 var MdmModuleContext_1 = require("../../../contexts/MdmModuleContext");
 var CommentDialogContext_1 = require("../../../contexts/CommentDialogContext");
-var useWorkflowActions = function (task, onActionSuccess) {
-    var _a = task !== null && task !== void 0 ? task : {}, possibleActions = _a.possibleActions, assignee = _a.assignee, taskId = _a.taskId, isOpen = _a.isOpen;
+var errors_2 = require("../../../helpers/errors");
+var useWorkflowActions = function (_a) {
+    var task = _a.task, onActionSuccess = _a.onActionSuccess, beforeAction = _a.beforeAction;
+    var _b = task !== null && task !== void 0 ? task : {}, possibleActions = _b.possibleActions, assignee = _b.assignee, taskId = _b.taskId, isOpen = _b.isOpen;
     var workflowTasks = (0, react_1.useContext)(WorkflowTasksContext_1.WorkflowTasksContext);
     var workflowPath = (0, MdmModuleContext_1.useMdmWorkflowPath)();
     var environment = (0, MdmModuleContext_1.useMdmWorkflowEnvironmentUrl)();
     var tenant = (0, MdmModuleContext_1.useMdmTenant)();
     var username = (0, MdmModuleContext_1.useMdmUsername)();
     var showCommentDialog = (0, CommentDialogContext_1.useCommentDialog)();
-    var _b = (0, react_1.useState)(false), actionRequestIsInProgress = _b[0], setActionRequestIsInProgress = _b[1];
+    var _c = (0, react_1.useState)(false), actionRequestIsInProgress = _c[0], setActionRequestIsInProgress = _c[1];
     var handleActionSuccess = onActionSuccess !== null && onActionSuccess !== void 0 ? onActionSuccess : workflowTasks === null || workflowTasks === void 0 ? void 0 : workflowTasks.onActionSuccess;
     var performTaskAction = (0, react_1.useCallback)(function (action, commentRequired) {
         return function (comment) { return __awaiter(void 0, void 0, void 0, function () {
-            var processInstanceComment;
+            var processInstanceComment, outcome, error_1;
             return __generator(this, function (_a) {
                 switch (_a.label) {
                     case 0:
-                        if (!(workflowPath && taskId)) return [3 /*break*/, 3];
+                        if (!(workflowPath && taskId)) return [3 /*break*/, 7];
                         setActionRequestIsInProgress(true);
                         processInstanceComment = typeof comment === 'string' && comment.length ? comment : null;
                         if (!(commentRequired && !processInstanceComment)) return [3 /*break*/, 2];
@@ -73,6 +75,27 @@ var useWorkflowActions = function (task, onActionSuccess) {
                         }
                         _a.label = 2;
                     case 2:
+                        if (!beforeAction) return [3 /*break*/, 6];
+                        _a.label = 3;
+                    case 3:
+                        _a.trys.push([3, 5, , 6]);
+                        return [4 /*yield*/, beforeAction()];
+                    case 4:
+                        outcome = _a.sent();
+                        if (!outcome.proceed) {
+                            if (outcome.error) {
+                                (0, errors_2.showDefaultErrorMessage)(outcome.error);
+                            }
+                            setActionRequestIsInProgress(false);
+                            return [2 /*return*/];
+                        }
+                        return [3 /*break*/, 6];
+                    case 5:
+                        error_1 = _a.sent();
+                        (0, errors_2.showDefaultErrorMessage)(error_1);
+                        setActionRequestIsInProgress(false);
+                        return [2 /*return*/];
+                    case 6:
                         (0, mdm_sdk_1.runTaskAction)({
                             taskId: taskId,
                             workflowPath: workflowPath,
@@ -88,12 +111,12 @@ var useWorkflowActions = function (task, onActionSuccess) {
                             .finally(function () {
                             setActionRequestIsInProgress(false);
                         });
-                        _a.label = 3;
-                    case 3: return [2 /*return*/];
+                        _a.label = 7;
+                    case 7: return [2 /*return*/];
                 }
             });
         }); };
-    }, [workflowPath, taskId, environment, tenant, handleActionSuccess, showCommentDialog]);
+    }, [workflowPath, taskId, environment, tenant, handleActionSuccess, showCommentDialog, beforeAction]);
     var actions = (0, react_1.useMemo)(function () {
         return username === assignee && (0, common_1.isOpenTask)(isOpen)
             ? (possibleActions || []).map(function (_a) {

package/cjs/features/workflow/types.d.ts

@@ -19,3 +19,7 @@ export type Diff = {
 export type Title = {
     label: string;
 };
+export type BeforeActionOutcome = {
+    proceed: boolean;
+    error?: unknown;
+};

package/features/workflow/ChangeRequestEditor/ChangeRequestEditor.js

@@ -5,7 +5,6 @@ import Typography from '@mui/material/Typography';
 import { addLineIdToValue } from './helpers/helpers';
 import SvgSuggestedChanges from '../../../icons/SuggestedChanges';
 import { EntityChangesGroup, SkeletonEntityChangesGroupLoader } from './components/EntityChangesGroup';
-import { ChangeRequestEditorProvider } from './context';
 import styles from './ChangeRequestEditor.module.css';
 export var ChangeRequestEditor = function (_a) {
     var dcr = _a.dcr, task = _a.task, groupedObjects = _a.groupedObjects;
@@ -23,5 +22,5 @@ export var ChangeRequestEditor = function (_a) {
             React.createElement("div", { className: styles.suggestedChangesIcon },
                 React.createElement(SvgSuggestedChanges, null)),
             React.createElement(Typography, { variant: "h6", className: styles.suggestedChangesTitle }, i18n.text('Suggested changes'))),
-        dcrLoaded ? (React.createElement(ChangeRequestEditorProvider, null, groupedObjects.map(function (object, index) { return (React.createElement(EntityChangesGroup, { key: "entity-".concat(index), entityInfo: object.entity, relationsInfo: object.relations, task: task, dcr: enrichedDCR })); }))) : (React.createElement(SkeletonEntityChangesGroupLoader, null))));
+        dcrLoaded ? (groupedObjects.map(function (object, index) { return (React.createElement(EntityChangesGroup, { key: "entity-".concat(index), entityInfo: object.entity, relationsInfo: object.relations, task: task, dcr: enrichedDCR })); })) : (React.createElement(SkeletonEntityChangesGroupLoader, null))));
 };

package/features/workflow/ChangeRequestEditor/components/ChangeItemEditor/ChangeItemEditor.js

@@ -21,7 +21,7 @@ export var ChangeItemEditor = function (_a) {
     var _b, _c, _d;
     var change = _a.change, _e = _a.editable, editable = _e === void 0 ? true : _e, classes = _a.classes;
     var metadata = useMdmMetadata();
-    var _f = useChangeRequestEditor({ change: change }), handleChange = _f.handleChange, handleReject = _f.handleReject, handleRevert = _f.handleRevert, lineData = _f.lineData, isLineRejected = _f.isLineRejected, isParentLineRejected = _f.isParentLineRejected, isUpdated = _f.isUpdated, hasChangeId = _f.hasChangeId;
+    var _f = useChangeRequestEditor({ change: change }), handleChange = _f.handleChange, handleReject = _f.handleReject, handleRevert = _f.handleRevert, lineData = _f.lineData, isLineRejected = _f.isLineRejected, isParentLineRejected = _f.isParentLineRejected, isUpdated = _f.isUpdated, hasChangeId = _f.hasChangeId, isSubAttributeOfDeletedAttribute = _f.isSubAttributeOfDeletedAttribute;
     var entity = useMdmEntity();
     var entityType = getEntityType(metadata, entity === null || entity === void 0 ? void 0 : entity.type);
     var isUpdateOperation = change.operation === DCROperationTypes.EDITED;
@@ -48,6 +48,7 @@ export var ChangeItemEditor = function (_a) {
         !isUpdated &&
         !isParentLineRejected &&
         hasChangeId &&
+        !isSubAttributeOfDeletedAttribute &&
         !(change.isReferenceSubAttribute && !editable);
     var showStatusChips = !isParentLineRejected;
     var isRejected = isLineRejected || isParentLineRejected;

package/features/workflow/ChangeRequestEditor/components/ChangeItemRow/ChangeItemRow.js

@@ -3,7 +3,7 @@ import React from 'react';
 import { useContextSelector } from '@fluentui/react-context-selector';
 import { DCROperationTypes } from '@reltio/mdm-sdk';
 import { isEditableChange } from '../../helpers/helpers';
-import { ChangeRequestEditorContext } from '../../context';
+import { ChangeRequestEditorContext } from '../../../contexts/ChangeRequestEditorContext';
 import { ChangeItemEditor, ChangeItemOldValue } from '../ChangeItemEditor';
 import { extractLineIds } from '../../../helpers/merge';
 import styles from './ChangeItemRow.module.css';

package/features/workflow/ChangeRequestEditor/components/DCRValueEditor/DCRValueEditor.d.ts

@@ -1,6 +1,6 @@
 import React from 'react';
 import { AttributeType, ChangeValue, PrimitiveValue, SimpleAttributeValue } from '@reltio/mdm-sdk';
-import { LineDataValue } from '../../context';
+import { LineDataValue } from '../../../contexts/ChangeRequestEditorContext';
 type Props = {
     attributeType: AttributeType;
     value: LineDataValue | ChangeValue;

package/features/workflow/ChangeRequestEditor/components/DCRValueEditorFactory/DCRValueEditorFactory.d.ts

@@ -1,6 +1,6 @@
 import React from 'react';
 import { AttributeType, ChangeValue, DataTypeDefinition, PrimitiveValue, SimpleAttributeValue } from '@reltio/mdm-sdk';
-import { LineDataValue } from '../../context';
+import { LineDataValue } from '../../../contexts/ChangeRequestEditorContext';
 type Props = {
     attributeType: AttributeType;
     value: LineDataValue | ChangeValue;

package/features/workflow/ChangeRequestEditor/hooks/useChangeRequestEditor.d.ts

@@ -7,10 +7,11 @@ export declare const useChangeRequestEditor: ({ change }: Props) => {
     handleChange: (newValue: SimpleAttributeValue | string[] | PrimitiveValue) => void;
     handleReject: () => void;
     handleRevert: () => void;
-    lineData: import("../context").LineData;
+    lineData: import("../../contexts/ChangeRequestEditorContext").LineData;
     isLineRejected: boolean;
     isParentLineRejected: boolean;
     isUpdated: boolean;
     hasChangeId: boolean;
+    isSubAttributeOfDeletedAttribute: any;
 };
 export {};

package/features/workflow/ChangeRequestEditor/hooks/useChangeRequestEditor.js

@@ -1,6 +1,7 @@
 import { useCallback } from 'react';
 import { useContextSelector } from '@fluentui/react-context-selector';
-import { ChangeRequestEditorContext } from '../context';
+import { DCROperationTypes } from '@reltio/mdm-sdk';
+import { ChangeRequestEditorContext } from '../../contexts/ChangeRequestEditorContext';
 import { extractLineIds } from '../../helpers/merge';
 var emptyLineIds = [];
 export var useChangeRequestEditor = function (_a) {
@@ -16,6 +17,10 @@ export var useChangeRequestEditor = function (_a) {
     var isUpdated = useContextSelector(ChangeRequestEditorContext, function (context) { return context.getIsLineUpdated(lineIds[0]); });
     var isLineRejected = lineData === null || lineData === void 0 ? void 0 : lineData.isRejected;
     var hasChangeId = lineIds.length > 0;
+    var isSubAttributeOfDeletedAttribute = change.operation === DCROperationTypes.DELETED &&
+        change.level > 0 &&
+        lineIds.length > 0 &&
+        lineIds[0].includes('/value/');
     var handleChange = useCallback(function (newValue) {
         updateChange(lineIds, typeof newValue === 'object' && 'value' in newValue ? newValue : { value: newValue });
     }, [lineIds, updateChange]);
@@ -33,6 +38,7 @@ export var useChangeRequestEditor = function (_a) {
         isLineRejected: isLineRejected,
         isParentLineRejected: isParentLineRejected,
         isUpdated: isUpdated,
-        hasChangeId: hasChangeId
+        hasChangeId: hasChangeId,
+        isSubAttributeOfDeletedAttribute: isSubAttributeOfDeletedAttribute
     };
 };

package/features/workflow/ReviewDCRDialog/ReviewDCRDialog.js

@@ -9,7 +9,6 @@ import i18n from 'ui-i18n';
 import { useWorkflowCheckPermission } from '../hooks/useWorkflowCheckPermission';
 import { useWorkflowActions } from '../hooks/useWorkflowActions';
 import { useMdmEntity } from '../../../contexts/MdmModuleContext';
-import { ChangeRequestEditorProvider } from '../ChangeRequestEditor/context';
 import { ProfileBand } from '../../../ProfileBand';
 import { DueDateField } from '../DueDateField';
 import { PrioritySelector } from '../PrioritySelector';
@@ -18,17 +17,22 @@ import { WorkflowComments } from '../WorkflowComments';
 import { WorkflowActionButtons } from '../WorkflowActionButtons';
 import { DCRSummaryInfo } from './components/DCRSummaryInfo/DCRSummaryInfo';
 import { ChangeRequestEditor } from '../ChangeRequestEditor';
+import { useApplyDcrChanges } from '../hooks/useApplyDcrChanges';
 import styles from './ReviewDCRDialog.module.css';
 export var ReviewDCRDialog = function (_a) {
     var open = _a.open, onClose = _a.onClose, dcr = _a.dcr, task = _a.task, groupedObjects = _a.groupedObjects;
     var entity = useMdmEntity();
     var checkWorkflowPermission = useWorkflowCheckPermission();
     var canViewComments = checkWorkflowPermission('VIEW_PROCESS_INSTANCE_COMMENTS');
+    var applyDcrChanges = useApplyDcrChanges(dcr);
     var _b = useWorkflowActions({
-        taskId: task.taskId,
-        possibleActions: task.possibleActions,
-        assignee: task.assignee,
-        isOpen: task.isOpen
+        task: {
+            taskId: task.taskId,
+            possibleActions: task.possibleActions,
+            assignee: task.assignee,
+            isOpen: task.isOpen
+        },
+        beforeAction: applyDcrChanges
     }), workflowActions = _b.actions, actionRequestIsInProgress = _b.actionRequestIsInProgress;
     return (React.createElement(Dialog, { open: open, onClose: onClose, fullScreen: true },
         React.createElement(DialogTitle, { className: styles.dialogTitle },
@@ -49,8 +53,7 @@ export var ReviewDCRDialog = function (_a) {
                         React.createElement("div", { className: styles.editableItem },
                             React.createElement(AssigneeSelector, { taskId: task.taskId, assignee: task.assignee, isTaskOpen: task.isOpen })))),
                 React.createElement("div", { className: styles.changesSection },
-                    React.createElement(ChangeRequestEditorProvider, null,
-                        React.createElement(ChangeRequestEditor, { dcr: dcr, task: task, groupedObjects: groupedObjects })))),
+                    React.createElement(ChangeRequestEditor, { dcr: dcr, task: task, groupedObjects: groupedObjects }))),
             canViewComments && (React.createElement(WorkflowComments, { workflowActions: workflowActions, actionRequestIsInProgress: actionRequestIsInProgress, preferredAction: task.preferredAction, taskId: task.taskId, processInstanceComments: task.processInstanceComments, isTaskOpen: task.isOpen, showActionButtons: false, alwaysExpanded: true, classes: {
                     container: styles.commentsRootContainer,
                     commentsContainer: styles.commentsContainer

package/features/workflow/TaskActionButtons/TaskActionButtons.js

@@ -6,10 +6,12 @@ export var TaskActionButtons = function (_a) {
     var task = _a.task;
     var taskId = task.taskId, possibleActions = task.possibleActions, assignee = task.assignee, isOpen = task.isOpen, preferredAction = task.preferredAction;
     var _b = useWorkflowActions({
-        taskId: taskId,
-        possibleActions: possibleActions,
-        assignee: assignee,
-        isOpen: isOpen
+        task: {
+            taskId: taskId,
+            possibleActions: possibleActions,
+            assignee: assignee,
+            isOpen: isOpen
+        }
     }), actions = _b.actions, disabled = _b.actionRequestIsInProgress;
     var _c = partition(propEq('text', preferredAction === null || preferredAction === void 0 ? void 0 : preferredAction.label), actions), preferredActions = _c[0], otherActions = _c[1];
     return (React.createElement(React.Fragment, null,

package/features/workflow/WorkflowTaskCard/components/DataChangeRequestTaskCard/DataChangeRequestTaskCard.js

@@ -1,7 +1,8 @@
 import React, { useCallback, useContext, useMemo, useState } from 'react';
 import { isNil } from 'ramda';
+import { isDCRUri } from '@reltio/mdm-sdk';
 import DcrTaskIcon from '../../../../../icons/DcrTaskIcon';
-import { useMdmReviewDCREnabled, useMdmUsername } from '../../../../../contexts/MdmModuleContext';
+import { useMdmEntity, useMdmReviewDCREnabled, useMdmUsername } from '../../../../../contexts/MdmModuleContext';
 import { GenericWorkflowTaskCard } from '../GenericWorkflowTaskCard';
 import { AttributesChanges } from '../../../AttributesChanges';
 import { LineDecorator } from '../../../LineDecorator';
@@ -9,11 +10,14 @@ import { useChangeRequest } from '../../../hooks/useChangeRequest';
 import { getDCRUri } from '../../../helpers/common';
 import { ReviewDCRDialog } from '../../../ReviewDCRDialog';
 import { ReviewDCRButton } from '../../../ReviewDCRButton';
+import { ChangeRequestEditorProvider } from '../../../contexts/ChangeRequestEditorContext';
 import { WorkflowTasksContext } from '../../../../../contexts/WorkflowTasksContext';
 export var DataChangeRequestTaskCard = function (_a) {
     var task = _a.task;
     var reviewDCREnabledByEntityType = (useContext(WorkflowTasksContext) || {}).reviewDCR;
     var reviewDCREnabled = useMdmReviewDCREnabled();
+    var entity = useMdmEntity();
+    var isDCREntity = isDCRUri(entity === null || entity === void 0 ? void 0 : entity.uri);
     var isReviewDCREnabled = isNil(reviewDCREnabledByEntityType) ? reviewDCREnabled : reviewDCREnabledByEntityType;
     var username = useMdmUsername();
     var _b = useState(false), isReviewDCRDialogOpen = _b[0], setIsReviewDCRDialogOpen = _b[1];
@@ -26,12 +30,13 @@ export var DataChangeRequestTaskCard = function (_a) {
     var handleCloseReviewDCRDialog = useCallback(function () {
         setIsReviewDCRDialogOpen(false);
     }, []);
-    var showReviewDCRButton = isReviewDCREnabled && assignee == username;
-    var showReviewDialog = isReviewDCRDialogOpen && (showReviewDCRButton || isReviewDCREnabled);
+    var showReviewDCRButton = !isDCREntity && isReviewDCREnabled && assignee == username;
+    var showReviewDialog = !isDCREntity && isReviewDCRDialogOpen && (showReviewDCRButton || isReviewDCREnabled);
     var customActionsSlot = useMemo(function () { return showReviewDCRButton && React.createElement(ReviewDCRButton, { onClick: handleReviewDCR }); }, [showReviewDCRButton, handleReviewDCR]);
     return (React.createElement(React.Fragment, null,
         React.createElement(GenericWorkflowTaskCard, { task: task, Icon: DcrTaskIcon, customActionsSlot: customActionsSlot }, dcr &&
             groupedObjects.map(function (object, index) { return (React.createElement(LineDecorator, { key: index, last: index === groupedObjects.length - 1, divider: true },
                 React.createElement(AttributesChanges, { entityInfo: object.entity, relationsInfo: object.relations, changes: dcr.changes, objectsInfo: dcr.objectsInfo, dcrUri: dcrUri }))); })),
-        showReviewDialog && (React.createElement(ReviewDCRDialog, { open: isReviewDCRDialogOpen, onClose: handleCloseReviewDCRDialog, dcr: dcr, groupedObjects: groupedObjects, task: task }))));
+        showReviewDialog && (React.createElement(ChangeRequestEditorProvider, null,
+            React.createElement(ReviewDCRDialog, { open: isReviewDCRDialogOpen, onClose: handleCloseReviewDCRDialog, dcr: dcr, groupedObjects: groupedObjects, task: task })))));
 };

package/features/workflow/WorkflowTaskCard/components/GenericWorkflowTaskCard/GenericWorkflowTaskCard.js

@@ -26,10 +26,12 @@ export var GenericWorkflowTaskCard = function (_a) {
     var checkWorkflowPermission = useWorkflowCheckPermission();
     var canViewComments = checkWorkflowPermission('VIEW_PROCESS_INSTANCE_COMMENTS');
     var _d = useWorkflowActions({
-        taskId: taskId,
-        possibleActions: possibleActions,
-        assignee: assignee,
-        isOpen: isTaskOpen
+        task: {
+            taskId: taskId,
+            possibleActions: possibleActions,
+            assignee: assignee,
+            isOpen: isTaskOpen
+        }
     }), workflowActions = _d.actions, actionRequestIsInProgress = _d.actionRequestIsInProgress;
     return (React.createElement("div", { className: styles.card },
         React.createElement("div", { className: classnames((_b = {}, _b[styles.withBackground] = isExpanded, _b)) },

package/features/workflow/{ChangeRequestEditor/context → contexts/ChangeRequestEditorContext}/index.d.ts

@@ -10,6 +10,7 @@ export type LineData = {
 type ChangeRequestEditorContextType = {
     updateChange: (lineIds: string[], data: LineDataValue) => void;
     getLineData: (lineId: string) => LineData | undefined;
+    getLinesByChangeId: (changeId: string) => Array<[string, LineData]>;
     rejectChange: (lineIds: string[]) => void;
     revertChangeUpdateOrReject: (lineIds: string[]) => void;
     getIsParentLineRejected: (lineId: string) => boolean;

package/features/workflow/{ChangeRequestEditor/context → contexts/ChangeRequestEditorContext}/index.js

@@ -1,9 +1,10 @@
-import React, { useCallback, useState } from 'react';
+import React, { useCallback, useEffect, useRef, useState } from 'react';
 import { isNil } from 'ramda';
 import { createContext } from '@fluentui/react-context-selector';
 export var ChangeRequestEditorContext = createContext({
     updateChange: undefined,
     getLineData: undefined,
+    getLinesByChangeId: undefined,
     rejectChange: undefined,
     revertChangeUpdateOrReject: undefined,
     getIsParentLineRejected: undefined,
@@ -13,6 +14,7 @@ export var ChangeRequestEditorContext = createContext({
 export var ChangeRequestEditorProvider = function (_a) {
     var children = _a.children;
     var _b = useState(new Map()), lineDataMap = _b[0], setLineDataMap = _b[1];
+    var lineDataMapRef = useRef(lineDataMap);
     var updateChange = useCallback(function (lineIds, data) {
         setLineDataMap(function (prevMap) {
             var newMap = new Map(prevMap);
@@ -62,9 +64,19 @@ export var ChangeRequestEditorProvider = function (_a) {
         var _a;
         return !isNil((_a = lineDataMap.get(lineId)) === null || _a === void 0 ? void 0 : _a.value);
     }, [lineDataMap]);
+    var getLinesByChangeId = useCallback(function (changeId) {
+        return Array.from(lineDataMapRef.current).filter(function (_a) {
+            var key = _a[0], _value = _a[1];
+            return key.split('/')[1] === changeId;
+        });
+    }, []);
+    useEffect(function () {
+        lineDataMapRef.current = lineDataMap;
+    }, [lineDataMap]);
     var contextValue = {
         updateChange: updateChange,
         getLineData: getLineData,
+        getLinesByChangeId: getLinesByChangeId,
         rejectChange: rejectChange,
         revertChangeUpdateOrReject: revertChangeUpdateOrReject,
         getIsParentLineRejected: getIsParentLineRejected,

package/features/workflow/helpers/dcr.d.ts

@@ -1,4 +1,5 @@
-import { ChangeRequest, DCRChanges, DCRObjectInfo, DCRObjectsInfo, GroupedObjectsInfo } from '@reltio/mdm-sdk';
+import { AttributeDelete, ChangeRequest, DCRChange, DCRChanges, DCRObjectInfo, DCRObjectsInfo, GroupedObjectsInfo, Metadata } from '@reltio/mdm-sdk';
+import { LineData } from '../contexts/ChangeRequestEditorContext';
 export declare const getEntitiesObjects: any;
 export declare const getRelationsObjects: any;
 export declare const extractObjectsFromChangeRequest: (changes: DCRChanges, objectsInfo: DCRObjectsInfo) => DCRObjectInfo[];
@@ -7,3 +8,9 @@ export declare const createEntityRelationsObjectByRelation: any;
 export declare const concatEntityRelationsObjects: (acc: Record<string, GroupedObjectsInfo>, object: GroupedObjectsInfo) => Record<string, GroupedObjectsInfo>;
 export declare const getEntityUriForChangeRequest: (dcrUri: string, entityUri: string) => string;
 export declare const isCreateEntityOperation: (dcr: ChangeRequest) => boolean;
+export declare const buildDCRSavePlan: (changes: DCRChanges, getLinesByChangeId: (changeId: string) => Array<[string, LineData]>, metadata: Metadata) => {
+    rejectedChangeIds: string[];
+    updatedChanges: Record<string, DCRChange[]>;
+    deleteEntityAttributes: Record<string, AttributeDelete[]>;
+    deleteRelationAttributes: Record<string, AttributeDelete[]>;
+};