Adjust notifications helper

app/helpers/notifications_helper.rb

@@ -16,32 +16,28 @@ module NotificationsHelper
   end
 
   def render_for_banner(title)
-    @@banner_renderer ||= NotificationTitleRenderer.new({ invert_link_colour: true, bold_all_text: true })
+    banner_renderer ||= NotificationTitleRenderer.new({ invert_link_colour: true, bold_all_text: true })
-    @@banner_markdown ||= Redcarpet::Markdown.new(@@banner_renderer, no_intra_emphasis: true)
+    Redcarpet::Markdown.new(banner_renderer, no_intra_emphasis: true).render(title)
-    @@banner_markdown.render(title)
   end
 
   def render_for_summary(title)
-    @@plain_title_renderer ||= NotificationTitleRenderer.new({ invert_link_colour: false, bold_all_text: false })
+    plain_title_renderer ||= NotificationTitleRenderer.new({ invert_link_colour: false, bold_all_text: false })
-    @@plain_title_markdown ||= Redcarpet::Markdown.new(@@plain_title_renderer, no_intra_emphasis: true)
+    Redcarpet::Markdown.new(plain_title_renderer, no_intra_emphasis: true).render(title)
-    @@plain_title_markdown.render(title)
   end
+end
 
-private
+class NotificationTitleRenderer < Redcarpet::Render::HTML
-
+  def initialize(options = {})
-  class NotificationTitleRenderer < Redcarpet::Render::HTML
+    link_class = "govuk-link"
-    def initialize(options = {})
+    link_class += " govuk-link--inverse" if options[:invert_link_colour]
-      link_class = "govuk-link"
+    @bold = options[:bold_all_text]
-      link_class += " govuk-link--inverse" if options[:invert_link_colour]
+    base_options = { escape_html: true, safe_links_only: true, link_attributes: { class: link_class } }
-      @bold = options[:bold_all_text]
+    super base_options
-      base_options = { escape_html: true, safe_links_only: true, link_attributes: { class: link_class } }
+  end
-      super base_options
-    end
 
-    def paragraph(text)
+  def paragraph(text)
-      return %(<p class="govuk-!-font-weight-bold">#{text}</p>) if @bold
+    return %(<p class="govuk-!-font-weight-bold">#{text}</p>) if @bold
 
-      %(<p>#{text}</p>)
+    %(<p>#{text}</p>)
-    end
   end
 end