noosfero | Fixes newsletter access (!1608)

Rodrigo Souto gitlab at mg.gitlab.com
Thu Sep 13 13:59:04 BRT 2018


Rodrigo Souto started a new discussion on plugins/public_access_restriction/lib/public_access_restriction_plugin.rb:

>        where(reference_article_id: article.id).first.present?
>    end
>  
> -  def show_newsletter environment, profile
> +  def show_newsletter? environment, params, profile

I don't think we should mix these two checks into the same method. Split them in different methods.

-- 
Reply to this email directly or view it on GitLab: https://gitlab.com/noosfero/noosfero/merge_requests/1608#note_101247559
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/20180913/e774d228/attachment.html>


More information about the Noosfero-dev mailing list