itizawa 5 лет назад
Родитель
Сommit
8d5bfeb5f4
2 измененных файлов с 3 добавлено и 2 удалено
  1. 1 1
      src/client/js/components/PageHistory.jsx
  2. 2 1
      src/server/routes/revision.js

+ 1 - 1
src/client/js/components/PageHistory.jsx

@@ -114,7 +114,7 @@ class PageHistory extends React.Component {
     const shareLinkId = this.props.shareLinkId || null;
     const shareLinkId = this.props.shareLinkId || null;
 
 
     if (revision.body) {
     if (revision.body) {
-      return;
+      // return;
     }
     }
 
 
     this.props.crowi.apiGet('/revisions.get',
     this.props.crowi.apiGet('/revisions.get',

+ 2 - 1
src/server/routes/revision.js

@@ -102,6 +102,7 @@ module.exports = function(crowi, app) {
   actions.api.get = async function(req, res) {
   actions.api.get = async function(req, res) {
     const pageId = req.query.page_id;
     const pageId = req.query.page_id;
     const revisionId = req.query.revision_id;
     const revisionId = req.query.revision_id;
+    const { isSharedPage } = req;
 
 
     if (!pageId || !revisionId) {
     if (!pageId || !revisionId) {
       return res.json(ApiResponse.error('Parameter page_id and revision_id are required.'));
       return res.json(ApiResponse.error('Parameter page_id and revision_id are required.'));
@@ -109,7 +110,7 @@ module.exports = function(crowi, app) {
 
 
     // check whether accessible
     // check whether accessible
     const isAccessible = await Page.isAccessiblePageByViewer(pageId, req.user);
     const isAccessible = await Page.isAccessiblePageByViewer(pageId, req.user);
-    if (!isAccessible) {
+    if (!isSharedPage && !isAccessible) {
       return res.json(ApiResponse.error('Current user is not accessible to this page.'));
       return res.json(ApiResponse.error('Current user is not accessible to this page.'));
     }
     }