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

Merge pull request #5416 from weseek/fix/89241-open-rename-modal

fix: 89241 open rename modal
Yuki Takei 4 лет назад
Родитель
Сommit
7f97cf9146

+ 1 - 2
packages/app/src/components/PageRenameModal.jsx

@@ -65,8 +65,7 @@ const PageRenameModal = (props) => {
   const updateSubordinatedList = useCallback(async() => {
     try {
       const res = await apiv3Get('/pages/subordinated-list', { path });
-      const { subordinatedPaths } = res.data;
-      setSubordinatedPages(subordinatedPaths);
+      setSubordinatedPages(res.data.subordinatedPages);
     }
     catch (err) {
       setErrs(err);

+ 5 - 1
packages/app/src/components/Sidebar/PageTree/Item.tsx

@@ -236,7 +236,11 @@ const Item: FC<ItemProps> = (props: ItemProps) => {
 
     const { _id: pageId, revision: revisionId, path } = page;
 
-    if (pageId == null || revisionId == null || path == null) {
+    if (!page.isEmpty && revisionId == null) {
+      throw Error('Existing page should have revisionId');
+    }
+
+    if (pageId == null || path == null) {
       throw Error('Any of _id and revisionId and path must not be null.');
     }