clean code

src/client/js/app.js

@@ -331,7 +331,6 @@ if (savePageControlsElem) {
   componentInstances.savePageControls = savePageControls;
 }
 
-// RecentCreated dev GC-939 start
 const recentCreatedControlsElem = document.getElementById('user-created-list');
 if (recentCreatedControlsElem) {
   let limit = crowi.getConfig().recentCreatedLimit;
@@ -344,7 +343,6 @@ if (recentCreatedControlsElem) {
     </RecentCreated>, document.getElementById('user-created-list')
   );
 }
-// RecentCreated dev GC-939 end
 
 /*
  * HackMD Editor

src/server/models/page.js

@@ -445,8 +445,6 @@ module.exports = function(crowi) {
     });
   };
 
-  // https://weseek.myjetbrains.com/youtrack/issue/GC-1224
-  // remove populate
   pageSchema.statics.findOneById = async function(id) {
     return this.findOne({_id: id});
   };
@@ -455,10 +453,6 @@ module.exports = function(crowi) {
    * @param {string} id ObjectId
    * @param {User} user User instance
    */
-  // https://weseek.myjetbrains.com/youtrack/issue/GC-1224
-  //
-  // TODO check whether NotFound or Forbidden in router
-  // org: findPageByIdAndGrantedUser
   pageSchema.statics.findOneByIdAndViewer = async function(id, user) {
     validateCrowi();
 

src/server/routes/page.js

@@ -158,7 +158,6 @@ module.exports = function(crowi, app) {
     renderVars.pages = pagePathUtils.encodePagesPath(pageList);
   }
 
-  // TODO rename to replacePlaceholdersOfTemplate
   function replacePlaceholdersOfTemplate(template, req) {
     const definitions = {
       pagepath: getPathFromRequest(req),
@@ -688,7 +687,7 @@ module.exports = function(crowi, app) {
         page = await Page.findPageByPathAndViewer(pagePath, req.user);
       }
       // https://weseek.myjetbrains.com/youtrack/issue/GC-1224
-      // TODO populate
+      // TODO populate for backward compatibility
     }
     catch (err) {
       return res.json(ApiResponse.error(err));