Merge pull request #2 from CloCkWeRX/issue-412

Added stripping users emails from whitespaces
fatfreecrm · Dec 5, 2016 · 7db549a · 7db549a
2 parents 6d8e508 + ceb8a6e
commit 7db549a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/app/controllers/admin/users_controller.rb b/app/controllers/admin/users_controller.rb
@@ -44,6 +44,7 @@ def edit
   # POST /admin/users.xml                                                  AJAX
   #----------------------------------------------------------------------------
   def create
+    params[:user][:email].try(:strip!)
     params[:user][:password_confirmation] = nil if params[:user][:password_confirmation].blank?
     @user = User.new(user_params)
     @user.save_without_session_maintenance
@@ -55,6 +56,7 @@ def create
   # PUT /admin/users/1.xml                                                 AJAX
   #----------------------------------------------------------------------------
   def update
+    params[:user][:email].try(:strip!)
     params[:user][:password_confirmation] = nil if params[:user][:password_confirmation].blank?
     @user = User.find(params[:id])
     @user.attributes = user_params

diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
@@ -134,6 +134,7 @@ def opportunities_overview
   protected
 
   def user_params
+    params[:user][:email].try(:strip!)
     params[:user].permit(
       :username,
       :email,