Download as
Browse Code »

Commit e791cacefee98bf9035c1176e2973fbd6e5f7012

Authored by Daniela Feitosa
1 parent 7bac7ef7
Exists in master and in 29 other branches add_member_task_reject_details, admin_visible_profile, article-list-template, community_notifications, contact_admin_translation, event_fixes, fix_comments_pagination, fix_rails4_organization_ratings, fix_sign_up_form, follow_step_fix, forum_topic_creation, mirror_block_improvements, multi_env_on_remote_user, new_security, noosfero_spb_ci, organization_ratings_improvements, organization_ratings_link_to_profile_stable-spb-1.3, organization_ratings_translations_fix, profile_api_improvements, ratings_minor_fixes, remote_user_fix, send_email_to_admins, stable-spb-1.3, stable-spb-1.3-fixes, stable-spb-1.4, stable-spb-1.5, suggest_rejected_value, web_steps_improvements, xss_terminate_custom_options

LoginBlock must have the same conditions as login view 

If a plugin doesn't allow registration, the signup button won't appear
If a plugin doesn't allow password recovery, the "Forgot my password" button won't appear

Also added notices after redirections

(ActionItem2423)
Showing 2 changed files with 14 additions and 6 deletions   Show diff stats
app/controllers/public/account_controller.rb
  Diff comments   View file @e791cac
@@ -59,6 +59,7 @@ class AccountController < ApplicationController @@ -59,6 +59,7 @@ class AccountController < ApplicationController
59 def signup 59 def signup
60 if @plugins.dispatch(:allow_user_registration).include?(false) 60 if @plugins.dispatch(:allow_user_registration).include?(false)
61 redirect_back_or_default(:controller => 'home') 61 redirect_back_or_default(:controller => 'home')
  62 + session[:notice] = _("This environment doesn't allow user registration.")
62 end 63 end
63 64
64 @invitation_code = params[:invitation_code] 65 @invitation_code = params[:invitation_code]
@@ -132,6 +133,7 @@ class AccountController < ApplicationController @@ -132,6 +133,7 @@ class AccountController < ApplicationController
132 def forgot_password 133 def forgot_password
133 if @plugins.dispatch(:allow_password_recovery).include?(false) 134 if @plugins.dispatch(:allow_password_recovery).include?(false)
134 redirect_back_or_default(:controller => 'home') 135 redirect_back_or_default(:controller => 'home')
  136 + session[:notice] = _("This environment doesn't allow password recovery.")
135 end 137 end
136 @change_password = ChangePassword.new(params[:change_password]) 138 @change_password = ChangePassword.new(params[:change_password])
137 139
app/views/account/login_block.rhtml
  Diff comments   View file @e791cac
@@ -16,18 +16,24 @@ @@ -16,18 +16,24 @@
16 16
17 <%= f.password_field :password %> 17 <%= f.password_field :password %>
18 18
  19 + <%= @plugins.dispatch(:login_extra_contents).collect { |content| instance_eval(&content) }.join("") %>
  20 +
19 <% button_bar do %> 21 <% button_bar do %>
20 <%= submit_button( 'login', _('Log in') )%> 22 <%= submit_button( 'login', _('Log in') )%>
21 - <%= link_to content_tag( 'span', _('New user') ),  
22 - { :controller => 'account', :action => 'signup' },  
23 - :class => 'button with-text icon-add' %> 23 + <% unless @plugins.dispatch(:allow_user_registration).include?(false) %>
  24 + <%= link_to content_tag( 'span', _('New user') ),
  25 + { :controller => 'account', :action => 'signup' },
  26 + :class => 'button with-text icon-add' %>
  27 + <% end %>
24 <% end %> 28 <% end %>
25 29
26 <% end %> 30 <% end %>
27 31
28 - <p class="forgot-passwd">  
29 - <%= link_to _("I forgot my password!"), :controller => 'account', :action => 'forgot_password' %>  
30 - </p> 32 + <% unless @plugins.dispatch(:allow_password_recovery).include?(false) %>
  33 + <p class="forgot-passwd">
  34 + <%= link_to _("I forgot my password!"), :controller => 'account', :action => 'forgot_password' %>
  35 + </p>
  36 + <% end %>
31 37
32 </div> 38 </div>
33 39