fix: implement logic

dhis2 · Jan 16, 2025 · 585be0d · 585be0d
1 parent 0be3dcf
commit 585be0d
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 7 deletions.
diff --git a/...core_modules/capture-core/components/WidgetProfile/OverflowMenu/OverflowMenu.component.js b/...core_modules/capture-core/components/WidgetProfile/OverflowMenu/OverflowMenu.component.js
@@ -17,11 +17,16 @@ export const OverflowMenuComponent = ({
     displayChangelog,
     teiId,
     programAPI,
+    readOnlyMode,
 }: PlainProps) => {
     const [actionsIsOpen, setActionsIsOpen] = useState(false);
     const [deleteModalIsOpen, setDeleteModalIsOpen] = useState(false);
     const [changelogIsOpen, setChangelogIsOpen] = useState(false);
 
+    if (readOnlyMode && !displayChangelog) {
+        return null;
+    }
+
     return (
         <>
             <OverflowButton
@@ -45,13 +50,15 @@ export const OverflowMenuComponent = ({
                                 <MenuDivider dense />
                             </>
                         )}
-                        <DeleteMenuItem
-                            trackedEntityTypeName={trackedEntityTypeName}
-                            canWriteData={canWriteData}
-                            canCascadeDeleteTei={canCascadeDeleteTei}
-                            setActionsIsOpen={setActionsIsOpen}
-                            setDeleteModalIsOpen={setDeleteModalIsOpen}
-                        />
+                        {!readOnlyMode && (
+                            <DeleteMenuItem
+                                trackedEntityTypeName={trackedEntityTypeName}
+                                canWriteData={canWriteData}
+                                canCascadeDeleteTei={canCascadeDeleteTei}
+                                setActionsIsOpen={setActionsIsOpen}
+                                setDeleteModalIsOpen={setDeleteModalIsOpen}
+                            />
+                        )}
                     </FlyoutMenu>
                 }
             />

diff --git a/...core_modules/capture-core/components/WidgetProfile/OverflowMenu/OverflowMenu.container.js b/...core_modules/capture-core/components/WidgetProfile/OverflowMenu/OverflowMenu.container.js
@@ -13,6 +13,7 @@ export const OverflowMenu = ({
     displayChangelog,
     teiId,
     programAPI,
+    readOnlyMode,
 }: Props) => {
     const { hasAuthority } = useAuthorities({ authorities: ['F_TEI_CASCADE_DELETE'] });
 
@@ -27,6 +28,7 @@ export const OverflowMenu = ({
             displayChangelog={displayChangelog}
             teiId={teiId}
             programAPI={programAPI}
+            readOnlyMode={readOnlyMode}
         />
     );
 };
diff --git a/src/core_modules/capture-core/components/WidgetProfile/OverflowMenu/OverflowMenu.types.js b/src/core_modules/capture-core/components/WidgetProfile/OverflowMenu/OverflowMenu.types.js
@@ -9,6 +9,7 @@ export type Props = {|
     displayChangelog: boolean,
     teiId: string,
     programAPI: any,
+    readOnlyMode: boolean,
 |};
 
 export type PlainProps = {|
@@ -21,4 +22,5 @@ export type PlainProps = {|
     displayChangelog: boolean,
     teiId: string,
     programAPI: any,
+    readOnlyMode: boolean,
 |};
diff --git a/src/core_modules/capture-core/components/WidgetProfile/WidgetProfile.component.js b/src/core_modules/capture-core/components/WidgetProfile/WidgetProfile.component.js
@@ -171,6 +171,7 @@ const WidgetProfilePlain = ({
                                 trackedEntityData={clientAttributesWithSubvalues}
                                 teiId={teiId}
                                 programAPI={program}
+                                readOnlyMode={readOnlyMode || false}
                             />
                         </div>
                     </div>