posts: allow new accounts to remove tags.
Remove the rule that new accounts can't remove tags (except for tagme and *_request tags) until their account is a week old.
This commit is contained in:
@@ -377,12 +377,6 @@ class Post < ApplicationRecord
|
||||
def update_tag_post_counts
|
||||
decrement_tags = tag_array_was - tag_array
|
||||
|
||||
decrement_tags_except_requests = decrement_tags.reject {|tag| tag == "tagme" || tag.end_with?("_request")}
|
||||
if !decrement_tags_except_requests.empty? && !CurrentUser.is_builder? && CurrentUser.created_at > 1.week.ago
|
||||
self.errors.add(:updater_id, "must have an account at least 1 week old to remove tags")
|
||||
return false
|
||||
end
|
||||
|
||||
increment_tags = tag_array - tag_array_was
|
||||
if increment_tags.any?
|
||||
Tag.increment_post_counts(increment_tags)
|
||||
|
||||
@@ -569,24 +569,6 @@ class PostTest < ActiveSupport::TestCase
|
||||
@post = FactoryBot.create(:post)
|
||||
end
|
||||
|
||||
context "as a new user" do
|
||||
setup do
|
||||
@post.update(:tag_string => "aaa bbb ccc ddd tagme")
|
||||
CurrentUser.user = FactoryBot.create(:user)
|
||||
end
|
||||
|
||||
should "not allow you to remove tags" do
|
||||
@post.update(tag_string: "aaa")
|
||||
assert_equal(["You must have an account at least 1 week old to remove tags"], @post.errors.full_messages)
|
||||
end
|
||||
|
||||
should "allow you to remove request tags" do
|
||||
@post.update(tag_string: "aaa bbb ccc ddd")
|
||||
@post.reload
|
||||
assert_equal("aaa bbb ccc ddd", @post.tag_string)
|
||||
end
|
||||
end
|
||||
|
||||
context "with a banned artist" do
|
||||
setup do
|
||||
CurrentUser.scoped(FactoryBot.create(:admin_user)) do
|
||||
|
||||
Reference in New Issue
Block a user