nocebo/diaspora
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

invitation emails stripped of whitespace from form

Browse source
This commit is contained in:
danielvincent 2010-12-10 18:10:24 -08:00
parent 2c5d5d9328
commit 6f62161442
2 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/controllers/invitations_controller.rb
View file

@ -19,7 +19,7 @@ class InvitationsController < Devise::InvitationsController
end end
aspect = params[:user].delete(:aspects) aspect = params[:user].delete(:aspects)
message = params[:user].delete(:invite_messages) message = params[:user].delete(:invite_messages)
emails = params[:user][:email].split(/, */) emails = params[:user][:email].to_s.gsub(/\s/, '').split(/, */)
good_emails, bad_emails = emails.partition{|e| e.try(:match, Devise.email_regexp)} good_emails, bad_emails = emails.partition{|e| e.try(:match, Devise.email_regexp)}

5
spec/controllers/invitations_controller_spec.rb
View file

@ -49,6 +49,11 @@ describe InvitationsController do
post :create, :user => @invite.merge(:email => "foofoofoofoo@example.com, mbs@gmail.com") post :create, :user => @invite.merge(:email => "foofoofoofoo@example.com, mbs@gmail.com")
end end
it 'can handle a comma seperated list of emails with whitespace' do
Resque.should_receive(:enqueue).twice()
post :create, :user => @invite.merge(:email => "foofoofoofoo@example.com , mbs@gmail.com")
end
it 'displays a message that tells you how many invites were sent, and which REJECTED' do it 'displays a message that tells you how many invites were sent, and which REJECTED' do
post :create, :user => @invite.merge(:email => "mbs@gmail.com, foo@bar.com, foo.com, lala@foo, cool@bar.com") post :create, :user => @invite.merge(:email => "mbs@gmail.com, foo@bar.com, foo.com, lala@foo, cool@bar.com")
flash[:error].should_not be_empty flash[:error].should_not be_empty