From e355a55e3c691399cf120e3e21815c8ea623dae9 Mon Sep 17 00:00:00 2001 From: Jean-Philippe Lang Date: Sun, 20 Jan 2013 13:54:42 +0000 Subject: [PATCH] Don't use && return. git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@11221 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- app/controllers/account_controller.rb | 6 +++--- app/controllers/projects_controller.rb | 3 ++- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/app/controllers/account_controller.rb b/app/controllers/account_controller.rb index 3c4b7b7cf..281e44b82 100644 --- a/app/controllers/account_controller.rb +++ b/app/controllers/account_controller.rb @@ -44,7 +44,7 @@ class AccountController < ApplicationController # Lets user choose a new password def lost_password - redirect_to(home_url) && return unless Setting.lost_password? + (redirect_to(home_url); return) unless Setting.lost_password? if params[:token] @token = Token.find_by_action_and_value("recovery", params[:token].to_s) if @token.nil? || @token.expired? @@ -94,7 +94,7 @@ class AccountController < ApplicationController # User self-registration def register - redirect_to(home_url) && return unless Setting.self_registration? || session[:auth_source_registration] + (redirect_to(home_url); return) unless Setting.self_registration? || session[:auth_source_registration] if request.get? session[:auth_source_registration] = nil @user = User.new(:language => current_language.to_s) @@ -176,7 +176,7 @@ class AccountController < ApplicationController user = User.find_or_initialize_by_identity_url(identity_url) if user.new_record? # Self-registration off - redirect_to(home_url) && return unless Setting.self_registration? + (redirect_to(home_url); return) unless Setting.self_registration? # Create on the fly user.login = registration['nickname'] unless registration['nickname'].nil? diff --git a/app/controllers/projects_controller.rb b/app/controllers/projects_controller.rb index a3527a550..df1d63776 100644 --- a/app/controllers/projects_controller.rb +++ b/app/controllers/projects_controller.rb @@ -141,7 +141,8 @@ class ProjectsController < ApplicationController def show if params[:jump] # try to redirect to the requested menu item - redirect_to_project_menu_item(@project, params[:jump]) && return + redirect_to_project_menu_item(@project, params[:jump]) + return end @users_by_role = @project.users_by_role -- 2.39.5