```ruby
module Spree::ReviewsHelper

  def star(the_class)
    "<span class=\"#{the_class}\"> &#10030; </span>"
  end

  def mk_stars(m)
    (1..5).collect {|n| n <= m ? star("lit") : star("unlit") }.join
  end

  def txt_stars(n, show_out_of = true)
    res = I18n.t('star', :count => n)
    res += ' ' + t('out_of_5') if show_out_of
    res
  end

end
```

Use helpers instead of html in helper, interpolate strings instead of concat
user
```ruby
module Spree::ReviewsHelper

  def star(the_class)
    content_tag(:span, " &#10030; ".html_safe, :class => the_class)
  end

  def mk_stars(m)
    (1..5).collect {|n| n <= m ? star("lit") : star("unlit") }.join
  end

  def txt_stars(n, show_out_of = true)
    res = I18n.t('star', :count => n)
    res += " #{t('out_of_5')}" if show_out_of
    res
  end

end
```
assistant