Varias entradas repetidas de resumen en el correo

Error
,
30

He podido reproducir el problema en mi sitio local de Discourse añadiendo una etiqueta a la configuración del sitio digest suppress tags y luego creando un tema con varias etiquetas.

El siguiente código causa el problema: discourse/app/models/topic.rb at main · discourse/discourse · GitHub

    if SiteSetting.digest_suppress_tags.present?
      tag_ids = Tag.where_name(SiteSetting.digest_suppress_tags.split("|")).pluck(:id)
      if tag_ids.present?
        topics =
          topics.joins("LEFT JOIN topic_tags tg ON topics.id = tg.topic_id").where(
            "tg.tag_id NOT IN (?) OR tg.tag_id IS NULL",
            tag_ids,
          )
      end
    end

Editar: No creo que sea necesario un LEFT JOIN aquí. Las columnas de la tabla topic_tags unida no parecen utilizarse más adelante en el método. La solución podría ser tan simple como:

if SiteSetting.digest_suppress_tags.present?
tag_ids = Tag.where_name(SiteSetting.digest_suppress_tags.split("|")).pluck(:id)
  if tag_ids.present?
  topics =
    topics.where.not(id: TopicTag.where(tag_id: tag_ids).select(:topic_id))
  end
end

Dejaré que el equipo de Discourse determine el mejor enfoque. El método for_digest se ejecuta muchas veces y necesita ser eficiente.

4 Me gusta