state update after tag updat

packages/app/src/client/services/PageContainer.js

@@ -347,6 +347,7 @@ export default class PageContainer extends Container {
   updateStateAfterTagAdded(page) {
     const newState = {
       updatedAt: page.updatedAt,
+      revisionAuthor: page.revision.author,
     };
     this.setState(newState);
   }

packages/app/src/components/Page/TagLabels.jsx

@@ -55,12 +55,12 @@ class TagLabels extends React.Component {
     if (editorMode === 'edit') {
       return editorContainer.setState({ tags: newTags });
     }
-
+    let page;
     try {
-      const { tags } = await appContainer.apiPost('/tags.update', {
+      const { tags, savedPage } = await appContainer.apiPost('/tags.update', {
         pageId, tags: newTags, revisionId, options,
       });
-
+      page = savedPage;
       // update pageContainer.state
       pageContainer.setState({ tags });
       // update editorContainer.state
@@ -71,7 +71,7 @@ class TagLabels extends React.Component {
     catch (err) {
       toastError(err, 'fail to update tags');
     }
-    pageContainer.updateStateAfterTagAdded();
+    pageContainer.updateStateAfterTagAdded(page);
   }
 
 

packages/app/src/server/routes/tag.js

@@ -135,9 +135,6 @@ module.exports = function(crowi, app) {
    * @apiParam {array} tags
    */
   api.update = async function(req, res) {
-    // GW36 tag update feature problem (when adding/removing tags, page's updatedAt does not get updated)
-    // Idea : call Page.updatePage to update 'updatedAt' when tag is added/removed
-    // ref: api.update in pages.js router
     const Page = crowi.model('Page');
     const PageTagRelation = crowi.model('PageTagRelation');
     const Revision = crowi.model('Revision');
@@ -153,7 +150,7 @@ module.exports = function(crowi, app) {
       // TODO GC-1921 consider permission
       const page = await Page.findById(pageId);
       const previousRevision = await Revision.findById(revisionId);
-      await Page.updatePage(page, previousRevision.body, previousRevision.body, req.user, options);
+      result.savedPage = await Page.updatePage(page, previousRevision.body, previousRevision.body, req.user, options);
       await PageTagRelation.updatePageTags(pageId, tags);
       result.tags = await PageTagRelation.listTagNamesByPage(pageId);