31 Mar, 2016
7 commits
-
Signed-off-by: Alexandre Barbosa <alexandreab@live.com> Signed-off-by: ArthurJahn <stutrzbecher@gmail.com> Signed-off-by: Marcos Ronaldo <marcos.rpj2@gmail.com>
-
Improve register performance - Send activation code with delayed job - Avoid unecessary user save after update person Signed-off-by: Gustavo Jaruga <darksshades@gmail.com> Signed-off-by: Marcos Ronaldo <marcos.rpj2@gmail.com> See merge request !829
-
Adds custom_fields details to moderation tasks Adds custom_values to community when moderated; Adds option to show custom_field in moderation task; Signed-off-by: Alexandre Barbosa <alexandreab@live.com> Signed-off-by: Arthur Jahn <stutrzbecher@gmail.com> Signed-off-by: Marcos Ronaldo <marcos.rpj2@gmail.com> See merge request !825 -
fix to sql injections vulnerabilities identified using brakeman See merge request !832
-
Signed-off-by: Lucas Kanashiro <kanashiro.duarte@gmail.com> Sign-off-by: Macartur Sousa <macartur.sc@gmail.com>
-
Uses custom fields for create community moderation Uses custom fields for user creation moderation Signed-off-by: Alexandre Barbosa <alexandreab@live.com> Signed-off-by: Arthur Jahn <stutrzbecher@gmail.com> Signed-off-by: Marcos Ronaldo <marcos.rpj2@gmail.com>
29 Mar, 2016
8 commits
-
Remove HTML generation from container_block plugin It now uses BoxesHelper logic instead of HTML generation within the model violating MVC. As it is a class within a module (`ContainerBlockPlugin::ContainerBlock`) `BoxesHelper` required fixes to handle the module prefix when searching for the proper view to render. Previously it would wrongly search for `blocks/container_block_plugin/container.html.erb` instead of `blocks/container.html.erb`. In case gitlab build does not start, here is the Travis build: https://travis-ci.org/rafamanzo/noosfero/builds/118607978 See merge request !827
-
Remove HTML generation from community_block plugin It now relies on BoxesHelper logic and thus the method got removed. As it was the only subject of unit tests, The unit tests for this class have been removed (most of the logic is tested through functional tests). It's dangerous to go alone. Have this travis build just in case: https://travis-ci.org/rafamanzo/noosfero/builds/118601061 See merge request !826
-
Allow other Profile types to use blocks Allow other Profile types (besides Person, Community and Enterprise) to use blocks See merge request !742
-
It now relies on BoxesHelper logic and thus the method got removed. As it was the only subject of unit tests, The unit tests for this class have been removed (most of the logic is tested through functional tests).
-
Signed-off-by: Rodrigo Souto <rodrigo@colivre.coop.br>
-
- Send activation code with delayed job - Avoid unecessary user save after update person Signed-off-by: Gustavo Jaruga <darksshades@gmail.com> Signed-off-by: Marcos Ronaldo <marcos.rpj2@gmail.com>
25 Mar, 2016
1 commit
-
It now uses BoxesHelper logic instead of HTML generation within the model violating MVC. As it is a class within a module (`ContainerBlockPlugin::ContainerBlock`) `BoxesHelper` required fixes to handle the module prefix when searching for the proper view to render. Previously it would wrongly search for `blocks/container_block_plugin/container.html.erb` instead of `blocks/container.html.erb`.
24 Mar, 2016
10 commits
-
Makes max_size uniform between UploadedFile and Thumbnail closes #152 See merge request !804
-
Improve oauth client This MR is related to oauth-client plugin. Fix a bug in edit of providers and improve documentation. Making this plugin up-to-date. See merge request !821
23 Mar, 2016
10 commits
-
Decouple people block plugin Model html generation As a follow up for !803. This applies the same refactor to PeopleBlockPlugin removing HTML generation from models. Here is a build: https://travis-ci.org/rafamanzo/noosfero/builds/117720302 As it involved many changes, I figured it would be worth of its own MR. I'll work on other block plugins refactor following. Please let me know if you do not want one MR for each. See merge request !817
-
Signed-off-by: Lucas Kanashiro <kanashiro.duarte@gmail.com> Signed-off-by: Macartur Sousa <macartur.sc@gmail.com>
-
Signed-off-by: Lucas Kanashiro <kanashiro.duarte@gmail.com> Signed-off-by: Macartur Sousa <macartur.sc@gmail.com>
-
Keep filters after close tasks See merge request !798
-
- Rename TasksHelper file - Referencing TasksHelper in TaskController - Refactoring task_action/url helper method - Manage tasks feature - Update SuggetstArticle step - Adds functional test for keeping tasks filter Signed-off-by: Alexandre Barbosa <alexandreab@live.com> Signed-off-by: Joenio Costa <joenio@colivre.coop.br> Signed-off-by: Ábner Silva de Oliveira <abner.oliveira@serpro.gov.br> Signed-off-by: Gustavo Jaruga <darkshades@gmail.com> Signed-off-by: Marcos Ronaldo <marcos.rpj2@gmail.com>
-
This method was producing view code from within the model which is a violation of MVC. Part of the code was extracted into the new helper and most of it became a view. This follow the standard for using BoxesHelper#render_block_footer.
-
This method was producing view code from within the model which is a violation of MVC. Part of the code was extracted into the new helper and most of it became a view. This follow the standard for using BoxesHelper#render_block_content.
-
The method for build profile_image_link was part of ApplicationHelper but will also be necessary for building proper tests on a upcoming refactor for PeopleBlockPlugin. As including ApplicationHelper on unit tests has already caused issues with unwanted block behaviour regarding their visibility, this is a move in the direction of avoiding such issues before they happen and allow reliable future tests.
-
comment: make explicit fail-safe relation with profile and environment This tests are necessary because comment estabilishes a polymorphic relation with the source, which creates harsh conditions to determine how to relate it with the environment. See merge request !819
22 Mar, 2016
4 commits
-
This tests are necessary because comment estabilishes a polymorphic relation with the source, which creates harsh conditions to determine how to relate it with the environment.