noosfero | Search infra improvements (#727)

Rodrigo Souto gitlab at mg.gitlab.com
Thu Nov 19 16:03:19 BRST 2015


New comment for Merge Request 727

https://gitlab.com/noosfero/noosfero/merge_requests/727#note_2710043


Rodrigo Souto

I think is alright to be in an instance variable because it's in the scope of a plugin action where the plugin define the base instance variables for the execution. Take a look at how it's used: https://softwarepublico.gov.br/gitlab/softwarepublico/softwarepublico/blob/gov-user-refactoring/src/noosfero-spb/gov_user/lib/gov_user_plugin.rb#L275

If you still think it is no good, we can skip it for now I guess.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listas.softwarelivre.org/pipermail/noosfero-dev/attachments/20151119/ade9cee9/attachment.html>


More information about the Noosfero-dev mailing list