Просмотр исходного кода

Revert "update frontend"

This reverts commit 5b638ff729c7766719eeb6e91f3ebd1581cbd125.
ryoji-s 2 лет назад
Родитель
Сommit
e9ee854993

+ 0 - 1
apps/app/src/components/Bookmarks/BookmarkItem.tsx

@@ -167,7 +167,6 @@ export const BookmarkItem = (props: Props): JSX.Element => {
         <div className="grw-foldertree-control">
           <PageItemControl
             pageId={bookmarkedPage._id}
-            pagePath={bookmarkedPage.path}
             isEnableActions
             isReadOnlyUser={isReadOnlyUser}
             pageInfo={pageInfo}

+ 13 - 16
apps/app/src/components/Common/Dropdown/PageItemControl.tsx

@@ -2,8 +2,9 @@ import React, {
   useState, useCallback, useEffect,
 } from 'react';
 
-import { type IPageInfoAll, isIPageInfoForOperation } from '@growi/core';
-import { pagePathUtils } from '@growi/core/dist/utils';
+import {
+  type IPageInfoAll, isIPageInfoForOperation,
+} from '@growi/core';
 import { getCustomModifiers } from '@growi/ui/dist/utils';
 import { useTranslation } from 'next-i18next';
 import {
@@ -56,7 +57,6 @@ type CommonProps = {
 
 type DropdownMenuProps = CommonProps & {
   pageId: string,
-  pagePath: string,
   isLoading?: boolean,
   operationProcessData?: IPageOperationProcessData,
 }
@@ -65,7 +65,7 @@ const PageItemControlDropdownMenu = React.memo((props: DropdownMenuProps): JSX.E
   const { t } = useTranslation('');
 
   const {
-    pageId, pagePath, isLoading, pageInfo, isEnableActions, isReadOnlyUser, forceHideMenuItems, operationProcessData,
+    pageId, isLoading, pageInfo, isEnableActions, isReadOnlyUser, forceHideMenuItems, operationProcessData,
     onClickBookmarkMenuItem, onClickRenameMenuItem, onClickDuplicateMenuItem, onClickDeleteMenuItem,
     onClickRevertMenuItem, onClickPathRecoveryMenuItem,
     additionalMenuItemOnTopRenderer: AdditionalMenuItemsOnTop,
@@ -178,10 +178,10 @@ const PageItemControlDropdownMenu = React.memo((props: DropdownMenuProps): JSX.E
         ) }
 
         {/* Move/Rename */}
-        { !forceHideMenuItems?.includes(MenuItemType.RENAME) && isEnableActions && !isReadOnlyUser && pagePathUtils.isMovablePage(pagePath) && (
+        { !forceHideMenuItems?.includes(MenuItemType.RENAME) && isEnableActions && !isReadOnlyUser && (
           <DropdownItem
             onClick={renameItemClickedHandler}
-            disabled={!pagePathUtils.isMovablePage(pagePath)}
+            disabled={!pageInfo.isDeletable}
             data-testid="open-page-move-rename-modal-btn"
             className="grw-page-control-dropdown-item"
           >
@@ -233,7 +233,7 @@ const PageItemControlDropdownMenu = React.memo((props: DropdownMenuProps): JSX.E
 
         {/* divider */}
         {/* Delete */}
-        { !forceHideMenuItems?.includes(MenuItemType.DELETE) && isEnableActions && !isReadOnlyUser && pageInfo.isDeletable && (
+        { !forceHideMenuItems?.includes(MenuItemType.DELETE) && isEnableActions && !isReadOnlyUser && (
           <>
             { showDeviderBeforeDelete && <DropdownItem divider /> }
             <DropdownItem
@@ -271,7 +271,6 @@ PageItemControlDropdownMenu.displayName = 'PageItemControl';
 
 type PageItemControlSubstanceProps = CommonProps & {
   pageId: string,
-  pagePath: string,
   fetchOnInit?: boolean,
   children?: React.ReactNode,
   operationProcessData?: IPageOperationProcessData,
@@ -370,34 +369,32 @@ export const PageItemControlSubstance = (props: PageItemControlSubstanceProps):
 
 export type PageItemControlProps = CommonProps & {
   pageId?: string,
-  pagePath?: string,
   children?: React.ReactNode,
   operationProcessData?: IPageOperationProcessData,
 }
 
 export const PageItemControl = (props: PageItemControlProps): JSX.Element => {
-  const { pageId, pagePath } = props;
+  const { pageId } = props;
 
-  if (pageId == null || pagePath == null) {
+  if (pageId == null) {
     return <></>;
   }
 
-  return <PageItemControlSubstance pageId={pageId} pagePath={pagePath} {...props} />;
+  return <PageItemControlSubstance pageId={pageId} {...props} />;
 };
 
 
 type AsyncPageItemControlProps = Omit<CommonProps, 'pageInfo'> & {
   pageId?: string,
-  pagePath?: string,
   children?: React.ReactNode,
 }
 
 export const AsyncPageItemControl = (props: AsyncPageItemControlProps): JSX.Element => {
-  const { pageId, pagePath } = props;
+  const { pageId } = props;
 
-  if (pageId == null || pagePath == null) {
+  if (pageId == null) {
     return <></>;
   }
 
-  return <PageItemControlSubstance pageId={pageId} pagePath={pagePath} fetchOnInit {...props} />;
+  return <PageItemControlSubstance pageId={pageId} fetchOnInit {...props} />;
 };

+ 2 - 3
apps/app/src/components/Navbar/SubNavButtons.tsx

@@ -78,7 +78,7 @@ type SubNavButtonsSubstanceProps = CommonProps & {
   pageId: string,
   shareLinkId?: string | null,
   revisionId: string | null,
-  path?: string,
+  path?: string | null,
   pageInfo: IPageInfoForOperation,
   expandContentWidth?: boolean,
 }
@@ -247,7 +247,6 @@ const SubNavButtonsSubstance = (props: SubNavButtonsSubstanceProps): JSX.Element
         <PageItemControl
           alignRight
           pageId={pageId}
-          pagePath={path}
           pageInfo={pageInfo}
           isEnableActions={!isGuestUser}
           isReadOnlyUser={!!isReadOnlyUser}
@@ -267,7 +266,7 @@ export type SubNavButtonsProps = CommonProps & {
   pageId: string,
   shareLinkId?: string | null,
   revisionId?: string | null,
-  path?: string,
+  path?: string | null,
   expandContentWidth?: boolean,
 };
 

+ 0 - 1
apps/app/src/components/PageList/PageListItemL.tsx

@@ -262,7 +262,6 @@ const PageListItemLSubstance: ForwardRefRenderFunction<ISelectable, Props> = (pr
                     <PageItemControl
                       alignRight
                       pageId={pageData._id}
-                      pagePath={pageData.path}
                       pageInfo={isIPageInfoForListing(pageMeta) ? pageMeta : undefined}
                       isEnableActions={isEnableActions}
                       isReadOnlyUser={isReadOnlyUser}

+ 0 - 1
apps/app/src/components/Sidebar/PageTree/Item.tsx

@@ -489,7 +489,6 @@ const Item: FC<ItemProps> = (props: ItemProps) => {
           <div className="grw-pagetree-control d-flex">
             <PageItemControl
               pageId={page._id}
-              pagePath={page.path}
               isEnableActions={isEnableActions}
               isReadOnlyUser={isReadOnlyUser}
               onClickBookmarkMenuItem={bookmarkMenuItemClickHandler}