fixed undefined slugs in topic emails. closes #2372

modules/topic-serialization/lib/notifications/aggregated-comment-strategy.js

@@ -21,7 +21,7 @@ AggregatedCommentStrategy.prototype = {
       editedAt: reply.editedAt,
 
       // TODO: permalink?
-      uri: owningForum.uri + '/topic/' + owningTopicId + '/' + reply.slug
+      uri: owningForum.uri + '/topic/' + owningTopicId + '/' + owningTopic.slug
     };
   },
 

modules/topic-serialization/lib/notifications/aggregated-reply-strategy.js

@@ -26,7 +26,7 @@ AggregatedReplyStrategy.prototype = {
       lastModified: reply.lastModified,
 
       // TODO: permalink?
-      uri: owningForum.uri + '/topic/' + owningTopicId + '/' + reply.slug
+      uri: owningForum.uri + '/topic/' + owningTopicId + '/' + owningTopic.slug
     };
   },
 

modules/topic-serialization/test/aggregated-comment-strategy-test.js

@@ -54,7 +54,7 @@ describe('aggregated-comment-strategy', function() {
       body: {},
       sent: "2014-02-01T00:00:00.000Z",
       editedAt: null,
-      uri: forum.uri + "/topic/" + topic._id + "/undefined"
+      uri: forum.uri + "/topic/" + topic._id + "/" + topic.slug
     });
   });
 

modules/topic-serialization/test/aggregated-reply-stategy-test.js

@@ -49,7 +49,7 @@ describe('aggregated-reply-strategy', function() {
       editedAt: null,
       lastChanged: "2014-02-01T00:00:00.000Z",
       lastModified: "2014-02-01T00:00:00.000Z",
-      uri: forum.uri + "/topic/" + topic._id + "/undefined"
+      uri: forum.uri + "/topic/" + topic._id + "/" + topic.slug
     });
   });