Настройки «Исключить эти теги из сводных писем» не работают в превью сводного письма?

Внести вклад Ошибка
1

Я исключил некоторые теги в новых настройках и попытался просмотреть электронное письмо с дайджестом. В превью отображаются некоторые дублирующиеся темы, а также темы с этими тегами.

2 лайка
2

Проверяю это на локальном dev-сайте, похоже, это баг. Код ожидает, что настройка digest_suppress_tags будет списком ID тегов, разделённых вертикальной чертой, но на самом деле это список названий тегов:

Вот как выглядит эта настройка на моём сайте:

SiteSetting.digest_suppress_tags
=> "foo|bar"

Код, исключающий теги из дайджеста, ожидает, что значение настройки будет "1|2" (ID тегов).

Обратите внимание, что настройка digest_suppress_categories работает без проблем. Категория-селектор в настройках сохраняет ID категорий, а не их названия.

3 лайка
5

Это уже исправлено в:

Спасибо за отчёт @th21 и за отличный анализ и отладку @simon!

4 лайка