移动端显示未分类

nbianca · 2021 年4 月 29 日 12:33

我在首帖中提交了针对该 bug 的修复方案：

我选择了第一种方案，因为它会删除 Discourse 中的一些代码。而另一种方案实际上需要添加代码来判断“未分类”类别中是否至少存在一个主题。

我还审查了所有使用 category.uncategorized? 的地方，发现如果我们让“未分类”不再那么特殊，而更像普通类别，那么以下几个位置可能就不再需要了：

app/controllers/list_controller.rb

af6d0342b


      
          @description_meta = if @category.uncategorized?
            I18n.t("category.uncategorized_description", locale: SiteSetting.default_locale)
          elsif @category.description_text.present?
            @category.description_text
          else
            SiteSetting.site_description
          end

github.com/discourse/discourse

app/serializers/basic_category_serializer.rb

3e7f7fdde


      
          def name
            object.uncategorized? ? I18n.t('uncategorized_category_name', locale: SiteSetting.default_locale) : object.name
          end
          
          def description_text
            object.uncategorized? ? I18n.t('category.uncategorized_description', locale: SiteSetting.default_locale) : object.description_text
          end
          
          def description
            object.uncategorized? ? I18n.t('category.uncategorized_description', locale: SiteSetting.default_locale) : object.description
          end
          
          def description_excerpt
            object.uncategorized? ? I18n.t('category.uncategorized_description', locale: SiteSetting.default_locale) : object.description_excerpt
          end

github.com/discourse/discourse

app/models/category.rb

eeaecd4fd


      
                SELECT c2.id FROM categories c2
                LEFT JOIN topics t ON t.id = c2.topic_id AND t.deleted_at IS NULL
                WHERE t.id IS NULL AND c2.topic_id IS NOT NULL
              )
            SQL
          
            DB.exec(sql)
          
            Category
              .joins('LEFT JOIN topics ON categories.topic_id = topics.id AND topics.deleted_at IS NULL')
              .where('categories.id <> ?', SiteSetting.uncategorized_category_id)
              .where(topics: { id: nil })
              .find_each do |category|
              category.create_category_definition
            end
          end
          
          def slug_path
            if self.parent_category_id.present?
              slug_path = self.parent_category.slug_path
              slug_path.push(self.slug_for_url)

这些部分不会干扰其他功能，因此变更风险较低，但仍然并不容易。

话题		回复	浏览量
Disabled Uncategorized category is shown in mobile browsers when unlogged Support	7	1136	2018 年7 月 9 日
Allow_uncategorized_topics doesn't work on mobile Support	4	544	2019 年6 月 14 日
Disabled Uncategorized shows in category dropdowns Bug	9	520	2026 年7 月 10 日
Uncategorized shows up in Categories menu and results in 404 Bug	13	2426	2022 年7 月 21 日
Non-staff users are able to access uncategorized even when disabled Bug	13	1842	2019 年7 月 23 日

移动端显示未分类

相关话题