mirror of
https://github.com/codeninjasllc/discourse.git
synced 2024-11-23 23:58:31 -05:00
Don't include category definitions on Categories page
This commit is contained in:
parent
33661b2d56
commit
350eb82223
3 changed files with 4 additions and 3 deletions
|
@ -1,6 +1,6 @@
|
||||||
GIT
|
GIT
|
||||||
remote: https://github.com/dysania/onebox.git
|
remote: https://github.com/dysania/onebox.git
|
||||||
revision: 3a96eb87c548baf8161362b6b36b58026b5a3946
|
revision: 6a2f6e6a08f183a4df52f9a51187f566b8ae3a00
|
||||||
specs:
|
specs:
|
||||||
onebox (1.1.0)
|
onebox (1.1.0)
|
||||||
hexpress (~> 1.2)
|
hexpress (~> 1.2)
|
||||||
|
|
|
@ -20,7 +20,7 @@ class CategoryFeaturedTopic < ActiveRecord::Base
|
||||||
def self.feature_topics_for(c, existing=nil)
|
def self.feature_topics_for(c, existing=nil)
|
||||||
return if c.blank?
|
return if c.blank?
|
||||||
|
|
||||||
query = TopicQuery.new(self.fake_admin, per_page: SiteSetting.category_featured_topics, except_topic_ids: [c.topic_id], visible: true)
|
query = TopicQuery.new(self.fake_admin, per_page: SiteSetting.category_featured_topics, except_topic_ids: [c.topic_id], visible: true, no_definitions: true)
|
||||||
results = query.list_category(c).topic_ids
|
results = query.list_category(c).topic_ids
|
||||||
|
|
||||||
return if results == existing
|
return if results == existing
|
||||||
|
|
|
@ -19,6 +19,7 @@ class TopicQuery
|
||||||
sort_order
|
sort_order
|
||||||
no_subcategories
|
no_subcategories
|
||||||
sort_descending
|
sort_descending
|
||||||
|
no_definitions
|
||||||
status).map(&:to_sym)
|
status).map(&:to_sym)
|
||||||
|
|
||||||
# Maps `sort_order` to a columns in `topics`
|
# Maps `sort_order` to a columns in `topics`
|
||||||
|
@ -240,7 +241,7 @@ class TopicQuery
|
||||||
result = result.where('categories.name is null or categories.name <> ?', options[:exclude_category]).references(:categories) if options[:exclude_category]
|
result = result.where('categories.name is null or categories.name <> ?', options[:exclude_category]).references(:categories) if options[:exclude_category]
|
||||||
|
|
||||||
# Don't include the category topic unless restricted to that category
|
# Don't include the category topic unless restricted to that category
|
||||||
if options[:category].blank?
|
if options[:category].blank? || options[:no_definitions]
|
||||||
result = result.where('COALESCE(categories.topic_id, 0) <> topics.id')
|
result = result.where('COALESCE(categories.topic_id, 0) <> topics.id')
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue