Refactor RoomDirectory

Change-Id: Ie3bd606fc91b2673d2a3422f8fd465258d3211b0

src/components/RoomDirectory.js

@@ -13,6 +13,7 @@ import {
   TextField,
   TopToolbar,
   useCreate,
+  useDataProvider,
   useListContext,
   useNotify,
   useTranslate,
@@ -28,7 +29,7 @@ const RoomDirectoryPagination = () => (
   <Pagination rowsPerPageOptions={[100, 500, 1000, 2000]} />
 );
 
-export const RoomDirectoryDeleteButton = props => {
+export const RoomDirectoryUnpublishButton = props => {
   const translate = useTranslate();
 
   return (
@@ -49,7 +50,7 @@ export const RoomDirectoryDeleteButton = props => {
   );
 };
 
-export const RoomDirectoryBulkDeleteButton = props => (
+export const RoomDirectoryBulkUnpublishButton = props => (
   <BulkDeleteButton
     {...props}
     label="resources.room_directory.action.erase"
@@ -61,57 +62,59 @@ export const RoomDirectoryBulkDeleteButton = props => (
   />
 );
 
-export const RoomDirectoryBulkSaveButton = () => {
+export const RoomDirectoryBulkPublishButton = props => {
   const { selectedIds } = useListContext();
   const notify = useNotify();
   const refresh = useRefresh();
   const unselectAllRooms = useUnselectAll("rooms");
-  const { createMany, isloading } = useMutation();
+  const dataProvider = useDataProvider();
-
+  const { mutate, isLoading } = useMutation(
-  const handleSend = values => {
+    () =>
-    createMany(
+      dataProvider.createMany("room_directory", {
-      ["room_directory", "createMany", { ids: selectedIds, data: {} }],
+        ids: selectedIds,
-      {
+        data: {},
-        onSuccess: data => {
+      }),
-          notify("resources.room_directory.action.send_success");
+    {
-          unselectAllRooms();
+      onSuccess: () => {
-          refresh();
+        notify("resources.room_directory.action.send_success");
-        },
+        unselectAllRooms();
-        onError: error =>
+        refresh();
-          notify("resources.room_directory.action.send_failure", {
+      },
-            type: "error",
+      onError: () =>
-          }),
+        notify("resources.room_directory.action.send_failure", {
-      }
+          type: "error",
-    );
+        }),
-  };
+    }
+  );
 
   return (
     <Button
+      {...props}
       label="resources.room_directory.action.create"
-      onClick={handleSend}
+      onClick={mutate}
-      disabled={isloading}
+      disabled={isLoading}
     >
       <RoomDirectoryIcon />
     </Button>
   );
 };
 
-export const RoomDirectorySaveButton = () => {
+export const RoomDirectoryPublishButton = props => {
   const record = useRecordContext();
   const notify = useNotify();
   const refresh = useRefresh();
-  const [create, { isloading }] = useCreate();
+  const [create, { isLoading }] = useCreate();
 
   const handleSend = () => {
     create(
       "room_directory",
       { data: { id: record.id } },
       {
-        onSuccess: _data => {
+        onSuccess: () => {
           notify("resources.room_directory.action.send_success");
           refresh();
         },
-        onError: _error =>
+        onError: () =>
           notify("resources.room_directory.action.send_failure", {
             type: "error",
           }),
@@ -121,17 +124,16 @@ export const RoomDirectorySaveButton = () => {
 
   return (
     <Button
+      {...props}
       label="resources.room_directory.action.create"
       onClick={handleSend}
-      disabled={isloading}
+      disabled={isLoading}
     >
       <RoomDirectoryIcon />
     </Button>
   );
 };
 
-const RoomDirectoryBulkActionButtons = () => <RoomDirectoryBulkDeleteButton />;
-
 const RoomDirectoryListActions = () => (
   <TopToolbar>
     <SelectColumnsButton />
@@ -146,8 +148,8 @@ export const RoomDirectoryList = () => (
     actions={<RoomDirectoryListActions />}
   >
     <DatagridConfigurable
-      rowClick={(id, resource, record) => "/rooms/" + id + "/show"}
+      rowClick={(id, _resource, _record) => "/rooms/" + id + "/show"}
-      bulkActionButtons={<RoomDirectoryBulkActionButtons />}
+      bulkActionButtons={<RoomDirectoryBulkUnpublishButton />}
       omit={["room_id", "canonical_alias", "topic"]}
     >
       <AvatarField

src/components/rooms.js

@@ -36,10 +36,10 @@ import VisibilityIcon from "@mui/icons-material/Visibility";
 import EventIcon from "@mui/icons-material/Event";
 import RoomIcon from "@mui/icons-material/ViewList";
 import {
-  RoomDirectoryBulkDeleteButton,
+  RoomDirectoryBulkUnpublishButton,
-  RoomDirectoryBulkSaveButton,
+  RoomDirectoryBulkPublishButton,
-  RoomDirectoryDeleteButton,
+  RoomDirectoryUnpublishButton,
-  RoomDirectorySaveButton,
+  RoomDirectoryPublishButton,
 } from "./RoomDirectory";
 
 const date_format = {
@@ -79,14 +79,9 @@ const RoomShowActions = () => {
 
   return (
     <TopToolbar>
-      {roomDirectoryStatus === false && (
+      {roomDirectoryStatus === false && <RoomDirectoryPublishButton />}
-        <RoomDirectorySaveButton record={record} />
+      {roomDirectoryStatus === true && <RoomDirectoryUnpublishButton />}
-      )}
-      {roomDirectoryStatus === true && (
-        <RoomDirectoryDeleteButton record={record} />
-      )}
       <DeleteButton
-        record={record}
         mutationMode="pessimistic"
         confirmTitle="resources.rooms.action.erase.title"
         confirmContent="resources.rooms.action.erase.content"
@@ -280,8 +275,8 @@ export const RoomShow = props => {
 
 const RoomBulkActionButtons = () => (
   <>
-    <RoomDirectoryBulkSaveButton />
+    <RoomDirectoryBulkPublishButton />
-    <RoomDirectoryBulkDeleteButton />
+    <RoomDirectoryBulkUnpublishButton />
     <BulkDeleteButton
       confirmTitle="resources.rooms.action.erase.title"
       confirmContent="resources.rooms.action.erase.content"