From 4bb0ff51e07cbb4d932da2cb9ff05bdc841560fe Mon Sep 17 00:00:00 2001 From: Andrew Date: Mon, 27 Feb 2023 11:57:45 -0600 Subject: [PATCH] revert page and post author check --- app/controllers/page.js | 2 +- app/controllers/post.js | 2 +- app/services/resource.js | 12 +----------- 3 files changed, 3 insertions(+), 13 deletions(-) diff --git a/app/controllers/page.js b/app/controllers/page.js index 9752ee0..6d71085 100644 --- a/app/controllers/page.js +++ b/app/controllers/page.js @@ -51,7 +51,7 @@ class PageController extends SiteController { async getView (req, res, next) { const { resource: resourceService } = this.dtp.services; try { - await resourceService.recordView(req, 'Page', res.locals.page._id, res); + await resourceService.recordView(req, 'Page', res.locals.page._id); res.locals.pageSlug = res.locals.page.slug; res.locals.pageTitle = `${res.locals.page.title} on ${this.dtp.config.site.name}`; res.render('page/view'); diff --git a/app/controllers/post.js b/app/controllers/post.js index 50a3583..f60edff 100644 --- a/app/controllers/post.js +++ b/app/controllers/post.js @@ -339,7 +339,7 @@ class PostController extends SiteController { } } - await resourceService.recordView(req, 'Post', res.locals.post._id, res); + await resourceService.recordView(req, 'Post', res.locals.post._id); res.locals.countPerPage = 20; res.locals.pagination = this.getPaginationParameters(req, res.locals.countPerPage); diff --git a/app/services/resource.js b/app/services/resource.js index b0f6f69..a4f6d84 100644 --- a/app/services/resource.js +++ b/app/services/resource.js @@ -34,7 +34,7 @@ class ResourceService extends SiteService { * @param {mongoose.Types.ObjectId} resourceId The _id of the object for which * a view is being tracked. */ - async recordView (req, resourceType, resourceId, res) { + async recordView (req, resourceType, resourceId) { const Model = mongoose.model(resourceType); const modelUpdate = { $inc: { } }; @@ -44,16 +44,6 @@ class ResourceService extends SiteService { let uniqueKey = req.ip.toString().trim().toLowerCase(); if (req.user) { - // if (resourceType === 'Post') { - // if (req.user._id.equals(res.locals.post.author._id)) { - // return; - // } - // } - // if (resourceType === 'Page') { - // if (req.user._id.equals(res.locals.page.author._id)) { - // return; - // } - // } uniqueKey += `:user:${req.user._id.toString()}`; }