diff --git a/lib/composer_messages_finder.rb b/lib/composer_messages_finder.rb index f05fa862698..983081a3aa2 100644 --- a/lib/composer_messages_finder.rb +++ b/lib/composer_messages_finder.rb @@ -17,7 +17,7 @@ class ComposerMessagesFinder count = @user.created_topic_count education_key = :education_new_topic else - count = @user.topic_reply_count + count = @user.post_count education_key = :education_new_reply end @@ -59,8 +59,8 @@ class ComposerMessagesFinder # For users who are not new @user.has_trust_level?(:basic) && - # And who have posted enough topics - (@user.topic_reply_count > SiteSetting.educate_until_posts) && + # And who have posted enough + (@user.post_count > SiteSetting.educate_until_posts) && # And who haven't been notified about sequential replies already (!UserHistory.exists_for_user?(@user, :notified_about_sequential_replies)) diff --git a/spec/components/composer_messages_finder_spec.rb b/spec/components/composer_messages_finder_spec.rb index 35e16b318d7..4945168e6c0 100644 --- a/spec/components/composer_messages_finder_spec.rb +++ b/spec/components/composer_messages_finder_spec.rb @@ -46,12 +46,12 @@ describe ComposerMessagesFinder do end it "returns a message for a user who has not posted any topics" do - user.expects(:topic_reply_count).returns(10) + user.expects(:post_count).returns(10) finder.check_education_message.should be_present end it "returns no message when the user has posted enough topics" do - user.expects(:topic_reply_count).returns(11) + user.expects(:post_count).returns(11) finder.check_education_message.should be_blank end end @@ -109,7 +109,7 @@ describe ComposerMessagesFinder do before do SiteSetting.stubs(:educate_until_posts).returns(10) - user.topic_reply_count = 11 + user.stubs(:post_count).returns(11) Fabricate(:post, topic: topic, user: user) Fabricate(:post, topic: topic, user: user) @@ -135,7 +135,7 @@ describe ComposerMessagesFinder do end it "does not give a message to users who are still in the 'education' phase" do - user.topic_reply_count = 10 + user.stubs(:post_count).returns(10) finder.check_sequential_replies.should be_blank end