noosfero | WIP: Rails 5.1.6 Upgrade (!1438)

[Gabriel Silva]

Gabriel Silva started a new discussion on app/models/concerns/sanitize_tags.rb:

>    private
>  
>    def sanitize_tag_list
> -    sanitizer = HTML::FullSanitizer.new
> -    self.tag_list.map!{|i| strip_tag_name sanitizer.sanitize(i) }
> +    self.tag_list.map!{|i| strip_tag_name (Rails::HTML::FullSanitizer.new).sanitize(i) }

can't we keep using the snippet below?

```ruby
sanitizer = Rails::HTML::FullSanitizer.new
self.tag_list.map!{|i| strip_tag_name sanitizer.sanitize(i) }
```

It looks cleaner.

-- 
Reply to this email directly or view it on GitLab: https://gitlab.com/noosfero/noosfero/merge_requests/1438#note_74813421
You're receiving this email because of your account on gitlab.com.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listas.softwarelivre.org/pipermail/noosfero-dev/attachments/20180524/c365a3ba/attachment-0001.html>