digest_custom_html を HTML として扱われるように修正 (元: digest.html.erb のオーバーライド)

david · 2025 年 4 月 28 日午後 5:49

メソッドオーバーライドの結果に .html_safe を追加してもらえませんか？ERBテンプレート内にある必要はないと思います。

RailsとEmberの両方における一般的な目標は、「この文字列はHTMLとして安全である」ということを、作成/生成ポイントのできるだけ近くに配置することです。これにより、開発者はHTMLが実際に安全であることを確認する必要がある（つまり、ユーザー入力はエスケープされている）ことを明確に理解できます。

あなたがやっていることは問題ありません（テストされている限りは）。しかし、これらのメソッドの「意図された」使用方法ではありません。もしそれが意図的なプラグインAPIであれば、他のものと一緒に plugin/instance.rb にあるはずです。

このメソッドの意図された使用方法は、対応する翻訳キーにマークダウンを配置することです。

github.com/discourse/discourse

app/helpers/user_notifications_helper.rb

e62066ed4


      
          
          def digest_custom_html(position_key)
            digest_custom "user_notifications.digest.custom.html.#{position_key}"
          end
          
          def digest_custom_text(position_key)
            digest_custom "user_notifications.digest.custom.text.#{position_key}"
          end
          
          def digest_custom(i18n_key)
            PrettyText.format_for_email(I18n.t(i18n_key)).html_safe
          end
          
          def show_image_with_url(url)
            !(url.nil? || url.downcase.end_with?('svg'))
          end
          
          def email_image_url(basename)
            UrlHelper.absolute("#{Discourse.base_uri}/images/emails/#{basename}")
          end

（そこにある html_safe にも注意してください。これは、オーバーライドで必要となるのと同じテクニックです。）

トピック		返信	表示
How to change the style of the email template of the digest? Dev activity-summary	20	5668	2020 年 4 月 18 日
MJML for responsive digest and notification emails and other questions Feature	1	482	2020 年 3 月 20 日
Adding custom content to summary emails Feature email	14	5651	2023 年 6 月 6 日
Customize text is stripping out table Bug	1	552	2021 年 11 月 27 日
Best way to modify topics/show.html.erb Dev	2	1354	2018 年 3 月 8 日

digest_custom_html を HTML として扱われるように修正 (元: digest.html.erb のオーバーライド)

関連トピック