Некатегоризированные элементы отображаются на мобильных устройствах

pfaffman · 14.Апрель.2021 17:30:25

У меня есть сайт, где опция «Разрешить некатегоризированные темы» отключена, но для незарегистрированного пользователя (или для администратора) на Android-устройстве категория «Без категории» всё равно отображается. Я только что выполнил перестроение. Компоненты темы этого не объясняют. Проблема сохраняется в безопасном режиме. Установлены только официальные плагины.

На компьютере я эту проблему не вижу.

https://www.druidforum.org/

sam · 14.Апрель.2021 23:36:17

Вы очистили все темы из категории «Без категории»?

pfaffman · 15.Апрель.2021 00:23:28

Да! Это касается только темы «О нас», ничего больше. Мне трудно понять, как это вообще возможно с учётом спецификаций, которые, как я предполагаю, действуют. И это работает только на мобильных устройствах, не на десктопе.

codinghorror · 19.Апрель.2021 01:36:19

Можете ли вы воспроизвести это, @tshenry? Похоже, что только на мобильных устройствах?

tshenry · 19.Апрель.2021 23:01:03

Я не думаю, что мы ожидаем, что тема «О нас» останется в категории «Без категории», когда она отключена. В описании настройки сайта «Разрешить темы без категории» есть предупреждение (выделено курсивом):

ВНИМАНИЕ: Если есть любые темы без категории, вы должны перевыбрать их перед отключением этой опции.

Попробуйте переместить тему «О нас» в активную категорию. На моём тестовом сайте это сработало.

pfaffman · 19.Апрель.2021 23:08:18

А, понятно. Это помогло.

Это довольно запутанно, ведь обычно запрещено удалять тему «О» (но, полагаю, это не совсем тема «О», поскольку это не категория). Но я удалил её, и теперь «Без категории» больше не отображается. Пришлось делать это с телефона, иначе я бы не смог её найти.

«Без категории» очень запутывает, так как это НЕ категория. Я действительно считаю, что эта опция должна быть отключена по умолчанию.

codinghorror · 20.Апрель.2021 20:30:39

Нет, потому что я не считаю нужным с самого начала навязывать искусственную классификацию «Живое, Растительное, Минеральное» каждому отдельному объекту в мире. Извините.

Обратите внимание на предупреждение, которое там указано не просто так:

ВНИМАНИЕ: Если есть любые темы без категории, вы должны их переклассифицировать перед отключением этой функции.

Тем не менее, можем ли мы привести это поведение к единому стандарту @zogstrip, чтобы оно было одинаковым на мобильных устройствах и на компьютере?

pfaffman · 20.Апрель.2021 20:43:59

Да. Я понимаю этот аргумент и раньше с ним соглашался, но считаю, что категория по умолчанию имела бы больше смысла, чем отсутствие категории. Но я постараюсь не предлагать это снова, если только однажды мы не окажемся в одном месте с .

zogstrip · 21.Апрель.2021 09:14:18

@nbianca, можешь добавить в свой список задачу по выяснению того, почему категория «Без категории» отображалась на мобильном устройстве, хотя на десктопе её не было?

nbianca · 21.Апрель.2021 19:13:21

На десктопе мы отображаем категории и темы отдельно, а на мобильных устройствах — объединённо. Учитывая это, именно эта строка вызывает ошибку:

github.com/discourse/discourse

app/models/category_list.rb

main


      
                DismissedTopicUser.lookup_for(@guardian.current_user, @all_topics)
              @dismissed_topic_users_lookup.include?(topic.id)
            else
              false
            end
          end
          
          def find_categories
            query = Category.includes(CategoryList.included_associations).secured(@guardian)
          
            if SiteSetting.content_localization_enabled
              locale = I18n.locale.to_s
              query =
                query.joins(
                  ActiveRecord::Base.sanitize_sql_array(
                    [
                      "LEFT JOIN category_localizations cl ON cl.category_id = categories.id AND cl.locale = ?",
                      locale,
                    ],
                  ),
                )

Условие c.displayable_topics.blank? всегда истинно на десктопе, поскольку темы загружаются отдельно, поэтому displayable_topics всегда пуст. На мобильных устройствах оно истинно только в том случае, если в категории действительно нет тем.

У нас есть два решения:

отображать «без категории», если есть хотя бы одна тема, независимо от значения настройки «разрешать темы без категории»
скрывать «без категории», даже если есть тема, если настройка «разрешать темы без категории» отключена

Второе решение проще в реализации, так как достаточно просто убрать вторую часть условия.

pfaffman · 21.Апрель.2021 19:26:59

Я считаю, что скрытие тем из категории «Без категории» (странно) менее запутанно, чем отсутствие скрытия этой категории при включении настройки. Если вы отключите «Без категории», вы сразу заметите, что все эти темы исчезли. Но если при отключении ничего не происходит, то это кажется ошибкой.

codinghorror · 21.Апрель.2021 20:14:34

Давайте выберем более простой вариант, так как мы близки к релизу, и я не хочу делать ничего слишком рискованного.

sam · 22.Апрель.2021 00:19:14

Мой голос здесь — «удалить как можно больше специального кода без категории». Чем его больше, тем запутаннее всё становится.

Поэтому я голосую за:

показывать без категории, если есть хотя бы одна тема, независимо от значения настройки «разрешить темы без категории»

При условии, что реализация удалит специальный магический код из Discourse и мы будем иметь меньше крайних случаев.

В общем, выбирайте решение, которое удаляет больше кода из Discourse.

nbianca · 29.Апрель.2021 12:33:15

Я отправил исправление для ошибки в первом сообщении:

Я выбрал первое решение, так как оно удаляет часть кода из Discourse. Второе же, наоборот, добавляет код для определения наличия хотя бы одной темы в категории «Без категории».

Я также проверил все использования category.uncategorized? и нашёл несколько мест, которые могут оказаться ненужными, если мы сделаем категорию «Без категории» менее особенной и более похожей на обычную категорию:

github.com/discourse/discourse

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	8	1046	09.07.2018
Allow_uncategorized_topics doesn't work on mobile Support	5	502	14.07.2019
Disabled Uncategorized shows in category dropdowns Bug	8	311	25.08.2025
Uncategorized shows up in Categories menu and results in 404 Bug	14	2363	21.07.2022
Non-staff users are able to access uncategorized even when disabled Bug	14	1771	23.07.2019

Некатегоризированные элементы отображаются на мобильных устройствах

Связанные темы