Description:
add enabled option for user
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r670:f90ec3934e15 - - 1 file changed: 11 inserted, 13 deleted
@@ -30,67 +30,65 | |||||
|
30 |
|
30 | ||
|
31 | def authorization_by_roles(allowed_roles) |
|
31 | def authorization_by_roles(allowed_roles) |
|
32 | return false unless authenticate |
|
32 | return false unless authenticate |
|
33 | user = User.find(session[:user_id]) |
|
33 | user = User.find(session[:user_id]) |
|
34 | unless user.roles.detect { |role| allowed_roles.member?(role.name) } |
|
34 | unless user.roles.detect { |role| allowed_roles.member?(role.name) } |
|
35 | unauthorized_redirect |
|
35 | unauthorized_redirect |
|
36 | return false |
|
36 | return false |
|
37 | end |
|
37 | end |
|
38 | end |
|
38 | end |
|
39 |
|
39 | ||
|
40 | def testcase_authorization |
|
40 | def testcase_authorization |
|
41 | #admin always has privileged |
|
41 | #admin always has privileged |
|
42 | - puts "haha" |
|
||
|
43 | if @current_user.admin? |
|
42 | if @current_user.admin? |
|
44 | return true |
|
43 | return true |
|
45 | end |
|
44 | end |
|
46 |
|
45 | ||
|
47 | - puts "hehe" |
|
||
|
48 | - puts GraderConfiguration["right.view_testcase"] |
|
||
|
49 | unauthorized_redirect unless GraderConfiguration["right.view_testcase"] |
|
46 | unauthorized_redirect unless GraderConfiguration["right.view_testcase"] |
|
50 | end |
|
47 | end |
|
51 |
|
48 | ||
|
52 | protected |
|
49 | protected |
|
53 |
|
50 | ||
|
54 | def authenticate |
|
51 | def authenticate |
|
55 | unless session[:user_id] |
|
52 | unless session[:user_id] |
|
56 | flash[:notice] = 'You need to login' |
|
53 | flash[:notice] = 'You need to login' |
|
57 | if GraderConfiguration[SINGLE_USER_MODE_CONF_KEY] |
|
54 | if GraderConfiguration[SINGLE_USER_MODE_CONF_KEY] |
|
58 | flash[:notice] = 'You need to login but you cannot log in at this time' |
|
55 | flash[:notice] = 'You need to login but you cannot log in at this time' |
|
59 | end |
|
56 | end |
|
60 | redirect_to :controller => 'main', :action => 'login' |
|
57 | redirect_to :controller => 'main', :action => 'login' |
|
61 | return false |
|
58 | return false |
|
62 | end |
|
59 | end |
|
63 |
|
60 | ||
|
|
61 | + | ||
|
64 | # check if run in single user mode |
|
62 | # check if run in single user mode |
|
65 | if GraderConfiguration[SINGLE_USER_MODE_CONF_KEY] |
|
63 | if GraderConfiguration[SINGLE_USER_MODE_CONF_KEY] |
|
66 | - user = User.find_by_id(session[:user_id]) |
|
64 | + if @current_user==nil or (not @current_user.admin?) |
|
67 | - if user==nil or (not user.admin?) |
|
||
|
68 | flash[:notice] = 'You cannot log in at this time' |
|
65 | flash[:notice] = 'You cannot log in at this time' |
|
69 | redirect_to :controller => 'main', :action => 'login' |
|
66 | redirect_to :controller => 'main', :action => 'login' |
|
70 | return false |
|
67 | return false |
|
71 | end |
|
68 | end |
|
72 | - unless user.enabled? |
|
||
|
73 | - flash[:notice] = 'Your account is disabled' |
|
||
|
74 | - redirect_to :controller => 'main', :action => 'login' |
|
||
|
75 | - return false |
|
||
|
76 | - end |
|
||
|
77 | return true |
|
69 | return true |
|
78 | end |
|
70 | end |
|
79 |
|
71 | ||
|
|
72 | + # check if the user is enabled | ||
|
|
73 | + unless @current_user.enabled? or @current_user.admin? | ||
|
|
74 | + flash[:notice] = 'Your account is disabled' | ||
|
|
75 | + redirect_to :controller => 'main', :action => 'login' | ||
|
|
76 | + return false | ||
|
|
77 | + end | ||
|
|
78 | + | ||
|
80 | if GraderConfiguration.multicontests? |
|
79 | if GraderConfiguration.multicontests? |
|
81 | - user = User.find(session[:user_id]) |
|
80 | + return true if @current_user.admin? |
|
82 | - return true if user.admin? |
|
||
|
83 | begin |
|
81 | begin |
|
84 | - if user.contest_stat(true).forced_logout |
|
82 | + if @current_user.contest_stat(true).forced_logout |
|
85 | flash[:notice] = 'You have been automatically logged out.' |
|
83 | flash[:notice] = 'You have been automatically logged out.' |
|
86 | redirect_to :controller => 'main', :action => 'index' |
|
84 | redirect_to :controller => 'main', :action => 'index' |
|
87 | end |
|
85 | end |
|
88 | rescue |
|
86 | rescue |
|
89 | end |
|
87 | end |
|
90 | end |
|
88 | end |
|
91 | return true |
|
89 | return true |
|
92 | end |
|
90 | end |
|
93 |
|
91 | ||
|
94 | def authenticate_by_ip_address |
|
92 | def authenticate_by_ip_address |
|
95 | #this assume that we have already authenticate normally |
|
93 | #this assume that we have already authenticate normally |
|
96 | unless GraderConfiguration[MULTIPLE_IP_LOGIN_CONF_KEY] |
|
94 | unless GraderConfiguration[MULTIPLE_IP_LOGIN_CONF_KEY] |
You need to be logged in to leave comments.
Login now