Merge remote-tracking branch 'dtp/master'

- switch from `user-agent` to `user-agents`
- add favicons for feeds
- new layout for Venue embed

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);
+      await resourceService.recordView(req, 'Page', res.locals.page._id, res);
       res.locals.pageSlug = res.locals.page.slug;
       res.locals.pageTitle = `${res.locals.page.title} on ${this.dtp.config.site.name}`;
       res.render('page/view');

app/controllers/post.js

@@ -339,7 +339,7 @@ class PostController extends SiteController {
         }
       }
 
-      await resourceService.recordView(req, 'Post', res.locals.post._id);
+      await resourceService.recordView(req, 'Post', res.locals.post._id, res);
 
       res.locals.countPerPage = 20;
       res.locals.pagination = this.getPaginationParameters(req, res.locals.countPerPage);

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) {
+  async recordView (req, resourceType, resourceId, res) {
     const Model = mongoose.model(resourceType);
     const modelUpdate = { $inc: { } };
     
@@ -44,6 +44,16 @@ 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()}`;
     }