FIX: regression and attempts to have clearer naming (#7701)

discourse · Jun 5, 2019 · df6d46b · df6d46b
1 parent 0fd7f74
commit df6d46b
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/app/assets/javascripts/discourse/models/post.js.es6 b/app/assets/javascripts/discourse/models/post.js.es6
@@ -55,8 +55,12 @@ const Post = RestModel.extend({
   },
 
   @computed("post_number", "topic_id", "topic.slug")
-  url(postNr, topicId, slug) {
-    return postUrl(slug || slug, topicId || this.get("topic.id"), postNr);
+  url(post_number, topic_id, topicSlug) {
+    return postUrl(
+      topicSlug || this.topic_slug,
+      topic_id || this.get("topic.id"),
+      post_number
+    );
   },
 
   // Don't drop the /1