noosfero | profile: get redirection config from template if blank (!1704)

Gabriel Silva gitlab at mg.gitlab.com
Wed Aug 14 21:51:34 BRT 2019




Gabriel Silva started a new discussion on app/models/profile.rb: https://gitlab.com/noosfero/noosfero/merge_requests/1704#note_203834622

>  
>    validates_inclusion_of :redirection_after_login, :in => Environment.login_redirection_options.keys, :allow_nil => true
>    def preferred_login_redirection
> -    redirection_after_login.blank? ? environment.redirection_after_login : redirection_after_login
> +    redirection_after_login.blank? ? (template.present? && template.preferred_login_redirection) || environment.redirection_after_login : redirection_after_login

Would you make the following change?

```suggestion:-0+0
    redirection_after_login.blank? ? (template.present? && template.redirection_after_login) || environment.redirection_after_login : redirection_after_login
```

Just to make sure we don't fall in an infinite loop.

-- 
Reply to this email directly or view it on GitLab: https://gitlab.com/noosfero/noosfero/merge_requests/1704#note_203834622
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/20190815/a261c184/attachment.html>


More information about the Noosfero-dev mailing list