noosfero | WIP: Rails 5.1.6 Upgrade (!1438)

[Álax Alves]

Álax Alves commented on a 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) }

Well, now that I've tried to revert this, I realized why it had to be done this way. Changing to 
``` ruby
sanitizer = HTML::FullSanitizer.new
self.tag_list.map!{|i| strip_tag_name sanitizer.sanitize(i) }
```
causes `rake db:data:minimal`  to break with the following output:

![image](/uploads/159baf88d72a31f864558ef9905dde2b/image.png)

-- 
Reply to this email directly or view it on GitLab: https://gitlab.com/noosfero/noosfero/merge_requests/1438#note_81565864
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/20180614/133800ac/attachment-0001.html>