cafe-grader-web

Commit r160:0138848b0c5b public

parent | child
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:
default
Diff options:
Raw Diff | Patch Diff | Download Diff | Show whitespace | Increase context
Comments:
0 Commit comments 0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Side by Side Unified
Expand All Files Collapse All Files Wide Mode Diff
Add another comment
Browse Files

r160:0138848b0c5b - - 5 files changed: 23 inserted, 12 deleted

Show file before | Show file after | Raw diff | Download diff | Show whitespace Increase context Show commentsHide comments
@@ -1,42 +1,50
1 1 class MainController < ApplicationController
2 2
3 3 SYSTEM_MODE_CONF_KEY = 'system.mode'
4 4
5 5 before_filter :authenticate, :except => [:index, :login]
6 6 before_filter :check_viewability, :except => [:index, :login]
7 7
8 - #
9 - # COMMENT OUT: filter in each action instead
10 - #
8 + # COMMENTED OUT: filter in each action instead
11 9 # before_filter :verify_time_limit, :only => [:submit]
12 10
13 11 verify :method => :post, :only => [:submit],
14 12 :redirect_to => { :action => :index }
15 13
16 - # COMMENT OUT, only need when having high load
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 19 def index
20 20 redirect_to :action => 'login'
21 21 end
22 22
23 23 def login
24 24 saved_notice = flash[:notice]
25 25 reset_session
26 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
30 - #
32 + # logger.info "PATH: #{request.path}"
33 + # if Configuration['system.single_user_mode'] and
34 + # request.path!='/main/login'
35 + # @hidelogin = true
36 + # end
37 +
38 + # Site administrator login
31 39 @countries = Country.find(:all, :include => :sites)
32 40 @country_select = @countries.collect { |c| [c.name, c.id] }
33 41
34 42 @country_select_with_all = [['Any',0]]
35 43 @countries.each do |country|
36 44 @country_select_with_all << [country.name, country.id]
37 45 end
38 46
39 47 @site_select = []
40 48 @countries.each do |country|
41 49 country.sites.each do |site|
42 50 @site_select << ["#{site.name}, #{country.name}", site.id]
Show file before | Show file after | Raw diff | Download diff | Show whitespace Increase context Show commentsHide comments
@@ -46,29 +46,29
46 46 else
47 47 render :action => 'email_error', :layout => 'empty'
48 48 end
49 49 else
50 50 @user.errors.add_to_base("Email cannot be blank") if @user.email==''
51 51 render :action => 'new', :layout => 'empty'
52 52 end
53 53 end
54 54
55 55 def confirm
56 56 login = params[:login]
57 57 key = params[:activation]
58 - user = User.find_by_login(login)
59 - if (user) and (user.verify_activation_key(key))
60 - if user.valid? # check uniquenss of email
61 - user.activated = true
62 - user.save
58 + @user = User.find_by_login(login)
59 + if (@user) and (@user.verify_activation_key(key))
60 + if @user.valid? # check uniquenss of email
61 + @user.activated = true
62 + @user.save
63 63 @result = :successful
64 64 else
65 65 @result = :email_used
66 66 end
67 67 else
68 68 @result = :failed
69 69 end
70 70 render :action => 'confirm', :layout => 'empty'
71 71 end
72 72
73 73 protected
74 74
Show file before | Show file after | Raw diff | Download diff | Show whitespace Increase context Show commentsHide comments
@@ -119,20 +119,21
119 119 def self.encrypt(string,salt)
120 120 Digest::SHA1.hexdigest(salt + string)
121 121 end
122 122
123 123 def uniqueness_of_email_from_activated_users
124 124 user = User.activated_users.find_by_email(self.email)
125 125 if user and (user.login != self.login)
126 126 self.errors.add_to_base("Email has already been taken")
127 127 end
128 128 end
129 129
130 130 def enough_time_interval_between_same_email_registrations
131 + return if !self.new_record?
131 132 open_user = User.find_by_email(self.email,
132 133 :order => 'created_at DESC')
133 134 if open_user and open_user.created_at and
134 135 (open_user.created_at > Time.now.gmtime - 5.minutes)
135 136 self.errors.add_to_base("There are already unactivated registrations with this e-mail address (please wait for 5 minutes)")
136 137 end
137 138 end
138 139 end
Show file before | Show file after | Raw diff | Download diff | Show whitespace Increase context Show commentsHide comments
@@ -1,42 +1,43
1 1 %h1= Configuration['ui.front.title']
2 2
3 3 - if @announcements.length!=0
4 4 .announcementbox
5 5 %span{:class => 'title'}
6 6 Announcements
7 7 = render :partial => 'announcement', :collection => @announcements
8 8
9 9 %b= Configuration['ui.front.welcome_message']
10 10 %br/
11 +
12 + - if !@hidelogin
11 13 Please login to see the problem list.
12 14 %br/
13 15 %br/
14 16
15 17 - if flash[:notice]
16 18 %hr/
17 19 %b= flash[:notice]
18 20 %hr/
19 21
20 22 %div{ :style => "border: solid 1px gray; padding: 2px; background: #f0f0f0;"}
21 23 - form_tag :controller => 'login', :action => 'login' do
22 24 %table
23 25 %tr
24 26 %td{:align => "right"} Login:
25 27 %td= text_field_tag 'login'
26 28 %tr
27 29 %td{:align => "right"} Password:
28 30 %td= password_field_tag
29 31 = submit_tag 'Login'
30 -
31 32 %br/
32 33
33 34 - if Configuration['system.online_registration']
34 35 Want to participate?
35 36 %b
36 37 Please
37 38 = link_to 'register.', :controller => :users, :action => :new
38 39 %br/
39 40
40 41 - if (Configuration['system.mode']=='contest') and (Configuration['contest.multisites'])
41 42 %script{:type => 'text/javascript'}
42 43 var siteList = new Array();
Show file before | Show file after | Raw diff | Download diff | Show whitespace Increase context Show commentsHide comments
@@ -74,22 +74,23
74 74 post :register, :login => @john_info[:login],
75 75 :full_name => @john_info[:full_name],
76 76 :email => @john_info[:email]
77 77
78 78 response.should render_template('users/email_error')
79 79 end
80 80
81 81 it "should activate user with valid activation key" do
82 82 login = @john_info[:login]
83 83 User.should_receive(:find_by_login).
84 84 with(login).
85 85 and_return(@john)
86 + User.should_not_receive(:find_by_email)
86 87 @john.should_receive(:valid?).and_return(true)
87 88 @john.should_receive(:activated=).with(true)
88 89 @john.should_receive(:save).and_return(true)
89 90
90 91 get :confirm, :login => login, :activation => @john_activation_key
91 92
92 93 response.should render_template('users/confirm')
93 94 end
94 95
95 96 end
You need to be logged in to leave comments. Login now