FIX: staff/admin shouldn’t be able to create uncategorized topics (#7077)

This commit is contained in:
Joffrey JAFFEUX 2019-02-28 15:51:13 +01:00 committed by GitHub
parent 3517103398
commit 1cd64f68f1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 30 additions and 4 deletions

View File

@ -369,8 +369,7 @@ const Composer = RestModel.extend({
return (
canCategorize &&
!categoryId &&
!this.siteSettings.allow_uncategorized_topics &&
!this.user.get("admin")
!this.siteSettings.allow_uncategorized_topics
);
},

View File

@ -95,8 +95,7 @@ class Topic < ActiveRecord::Base
if: Proc.new { |t|
(t.new_record? || t.category_id_changed?) &&
!SiteSetting.allow_uncategorized_topics &&
(t.archetype.nil? || t.regular?) &&
(!t.user_id || !t.user.staff?)
(t.archetype.nil? || t.regular?)
}
validates :featured_link, allow_nil: true, url: true

View File

@ -999,6 +999,34 @@ describe PostsController do
expect(JSON.parse(response.body)["errors"]).to include(I18n.t(:spamming_host))
end
context "allow_uncategorized_topics is false" do
before do
SiteSetting.allow_uncategorized_topics = false
end
it "cant create an uncategorized post" do
post "/posts.json", params: {
raw: "a new post with no category",
title: "a new post with no category"
}
expect(response).not_to be_successful
end
context "as staff" do
before do
sign_in(Fabricate(:admin))
end
it "cant create an uncategorized post" do
post "/posts.json", params: {
raw: "a new post with no category",
title: "a new post with no category"
}
expect(response).not_to be_successful
end
end
end
end
end