Description:
fixed user confirmation bug
git-svn-id: http://theory.cpe.ku.ac.th/grader/web/trunk@305 6386c4cd-e34a-4fa8-8920-d93eb39b512e
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Author
jittat
-
r160:0138848b0c5b - - 5 files changed: 23 inserted, 12 deleted
| @@ -5,17 +5,17 | |||||
|
|
5 | before_filter :authenticate, :except => [:index, :login] |
|
5 | before_filter :authenticate, :except => [:index, :login] |
|
|
6 | before_filter :check_viewability, :except => [:index, :login] |
|
6 | before_filter :check_viewability, :except => [:index, :login] |
|
|
7 |
|
7 | ||
|
|
8 | - # |
|
8 | + # COMMENTED OUT: filter in each action instead |
|
|
9 | - # COMMENT OUT: filter in each action instead |
|
||
|
|
10 | - # |
|
||
|
|
11 |
|
|
9 | # before_filter :verify_time_limit, :only => [:submit] |
|
|
12 |
|
10 | ||
|
|
13 | verify :method => :post, :only => [:submit], |
|
11 | verify :method => :post, :only => [:submit], |
|
|
14 | :redirect_to => { :action => :index } |
|
12 | :redirect_to => { :action => :index } |
|
|
15 |
|
13 | ||
|
|
16 |
- |
|
14 | + # COMMENT OUT: only need when having high load |
|
|
17 |
|
|
15 | # caches_action :index, :login |
|
|
18 |
|
16 | ||
|
|
|
17 | + # NOTE: This method is not actually needed, 'config/routes.rb' has | ||
|
|
|
18 | + # assigned action login as a default action. | ||
|
|
19 | def index |
|
19 | def index |
|
|
20 | redirect_to :action => 'login' |
|
20 | redirect_to :action => 'login' |
|
|
21 | end |
|
21 | end |
| @@ -25,9 +25,17 | |||||
|
|
25 | reset_session |
|
25 | reset_session |
|
|
26 | flash[:notice] = saved_notice |
|
26 | flash[:notice] = saved_notice |
|
|
27 |
|
27 | ||
|
|
|
28 | + # EXPERIMENT: | ||
|
|
|
29 | + # Hide login if in single user mode and the url does not | ||
|
|
|
30 | + # explicitly specify /login | ||
|
|
28 | # |
|
31 | # |
|
|
29 | - # These are for site administrator login |
|
32 | + # logger.info "PATH: #{request.path}" |
|
|
30 | - # |
|
33 | + # if Configuration['system.single_user_mode'] and |
|
|
|
34 | + # request.path!='/main/login' | ||
|
|
|
35 | + # @hidelogin = true | ||
|
|
|
36 | + # end | ||
|
|
|
37 | + | ||
|
|
|
38 | + # Site administrator login | ||
|
|
31 | @countries = Country.find(:all, :include => :sites) |
|
39 | @countries = Country.find(:all, :include => :sites) |
|
|
32 | @country_select = @countries.collect { |c| [c.name, c.id] } |
|
40 | @country_select = @countries.collect { |c| [c.name, c.id] } |
|
|
33 |
|
41 | ||
| @@ -55,11 +55,11 | |||||
|
|
55 | def confirm |
|
55 | def confirm |
|
|
56 | login = params[:login] |
|
56 | login = params[:login] |
|
|
57 | key = params[:activation] |
|
57 | key = params[:activation] |
|
|
58 | - user = User.find_by_login(login) |
|
58 | + @user = User.find_by_login(login) |
|
|
59 | - if (user) and (user.verify_activation_key(key)) |
|
59 | + if (@user) and (@user.verify_activation_key(key)) |
|
|
60 | - if user.valid? # check uniquenss of email |
|
60 | + if @user.valid? # check uniquenss of email |
|
|
61 | - user.activated = true |
|
61 | + @user.activated = true |
|
|
62 | - user.save |
|
62 | + @user.save |
|
|
63 | @result = :successful |
|
63 | @result = :successful |
|
|
64 | else |
|
64 | else |
|
|
65 | @result = :email_used |
|
65 | @result = :email_used |
| @@ -128,6 +128,7 | |||||
|
|
128 | end |
|
128 | end |
|
|
129 |
|
129 | ||
|
|
130 | def enough_time_interval_between_same_email_registrations |
|
130 | def enough_time_interval_between_same_email_registrations |
|
|
|
131 | + return if !self.new_record? | ||
|
|
131 | open_user = User.find_by_email(self.email, |
|
132 | open_user = User.find_by_email(self.email, |
|
|
132 | :order => 'created_at DESC') |
|
133 | :order => 'created_at DESC') |
|
|
133 | if open_user and open_user.created_at and |
|
134 | if open_user and open_user.created_at and |
| @@ -8,6 +8,8 | |||||
|
|
8 |
|
8 | ||
|
|
9 | %b= Configuration['ui.front.welcome_message'] |
|
9 | %b= Configuration['ui.front.welcome_message'] |
|
|
10 | %br/ |
|
10 | %br/ |
|
|
|
11 | + | ||
|
|
|
12 | + - if !@hidelogin | ||
|
|
11 | Please login to see the problem list. |
|
13 | Please login to see the problem list. |
|
|
12 | %br/ |
|
14 | %br/ |
|
|
13 | %br/ |
|
15 | %br/ |
| @@ -27,7 +29,6 | |||||
|
|
27 | %td{:align => "right"} Password: |
|
29 | %td{:align => "right"} Password: |
|
|
28 | %td= password_field_tag |
|
30 | %td= password_field_tag |
|
|
29 | = submit_tag 'Login' |
|
31 | = submit_tag 'Login' |
|
|
30 | - |
|
||
|
|
31 | %br/ |
|
32 | %br/ |
|
|
32 |
|
33 | ||
|
|
33 | - if Configuration['system.online_registration'] |
|
34 | - if Configuration['system.online_registration'] |
| @@ -83,6 +83,7 | |||||
|
|
83 | User.should_receive(:find_by_login). |
|
83 | User.should_receive(:find_by_login). |
|
|
84 | with(login). |
|
84 | with(login). |
|
|
85 | and_return(@john) |
|
85 | and_return(@john) |
|
|
|
86 | + User.should_not_receive(:find_by_email) | ||
|
|
86 | @john.should_receive(:valid?).and_return(true) |
|
87 | @john.should_receive(:valid?).and_return(true) |
|
|
87 | @john.should_receive(:activated=).with(true) |
|
88 | @john.should_receive(:activated=).with(true) |
|
|
88 | @john.should_receive(:save).and_return(true) |
|
89 | @john.should_receive(:save).and_return(true) |
You need to be logged in to leave comments.
Login now