noosfero | WIP: Rails 5.1.6 Upgrade (!1438)

[Gabriel Silva]

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

>  
>    attr_accessible :content, :reason
>  
> -  belongs_to :reporter, :class_name => 'Person'
> -  belongs_to :abuse_complaint
> -  has_many :reported_images, :dependent => :destroy
> +  belongs_to :reporter, class_name: 'Person', optional: true

Is this relation really optional? Because there is a presence validation right on line 6. I believe we must double check all `optional` flags in `belongs_to` associations.

-- 
Reply to this email directly or view it on GitLab: https://gitlab.com/noosfero/noosfero/merge_requests/1438#note_113521961
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/20181031/7814a59d/attachment-0001.html>