Fix #1009 Replace all special characters in user's identity

tomykaira
1 parent 8b7e404b
Showing 2 changed files with 6 additions and 1 deletions Show diff stats
app/roles/account.rb
spec/models/user_spec.rb
 module Account 
   def identifier
-    email.gsub /[@.]/, "_"
+    email.gsub /[^[:alnum:]]/, "_"
   end
   def is_admin?
@@ -22,6 +22,11 @@ describe User do
     user.identifier.should == "test_mail_com"
   end
+  it "should return identifier without + sign" do
+    user = User.new(:email => "test+foo@mail.com")
+    user.identifier.should == "test_foo_mail_com"
+  end
+
   it "should execute callback when force_random_password specified" do
     user = User.new(:email => "test@mail.com", :force_random_password => true)
     user.should_receive(:generate_password)
1	module Account	1	module Account
2	def identifier	2	def identifier
3	- email.gsub /[@.]/, "_"	3	+ email.gsub /[^[:alnum:]]/, "_"
4	end	4	end
5		5
6	def is_admin?	6	def is_admin?