Download as
Browse Code »

Commit 15016ae68dd2e59ce1b12b0be6e9870d16219569

Authored by Drew
1 parent 26bdeb21
Exists in master and in 4 other branches 6-8-stable, 7-0-stable, 7-0-stable-spb, spb-stable

resolved issue 244 and updated test

Showing 2 changed files with 2 additions and 2 deletions   Show diff stats
app/models/user.rb
  Diff comments   View file @15016ae
@@ -30,7 +30,7 @@ class User < ActiveRecord::Base @@ -30,7 +30,7 @@ class User < ActiveRecord::Base
30 scope :not_in_project, lambda { |project| where("id not in (:ids)", :ids => project.users.map(&:id) ) } 30 scope :not_in_project, lambda { |project| where("id not in (:ids)", :ids => project.users.map(&:id) ) }
31 31
32 def identifier 32 def identifier
33 - email.gsub "@", "_" 33 + email.gsub /[@.]/, "_"
34 end 34 end
35 35
36 def is_admin? 36 def is_admin?
spec/models/user_spec.rb
  Diff comments   View file @15016ae
@@ -16,7 +16,7 @@ describe User do @@ -16,7 +16,7 @@ describe User do
16 16
17 it "should return valid identifier" do 17 it "should return valid identifier" do
18 user = User.new(:email => "test@mail.com") 18 user = User.new(:email => "test@mail.com")
19 - user.identifier.should == "test_mail.com" 19 + user.identifier.should == "test_mail_com"
20 end 20 end
21 21
22 it "should have authentication token" do 22 it "should have authentication token" do