Description:
add license agreement checkbox
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Author
-
r535:a0fe4acdb4ed - - 3 files changed: 10 inserted, 2 deleted
| @@ -4,13 +4,16 | |||
|
|
4 | 4 | # show login screen |
|
|
5 | 5 | reset_session |
|
|
6 | 6 | redirect_to :controller => 'main', :action => 'login' |
|
|
7 | 7 | end |
|
|
8 | 8 | |
|
|
9 | 9 | def login |
|
|
10 | - if user = User.authenticate(params[:login], params[:password]) | |
|
|
10 | + if (!GraderConfiguration['right.bypass_agreement']) and (!params[:accept_agree]) | |
|
|
11 | + flash[:notice] = 'You must accept the agreement before logging in' | |
|
|
12 | + redirect_to :controller => 'main', :action => 'login' | |
|
|
13 | + elsif user = User.authenticate(params[:login], params[:password]) | |
|
|
11 | 14 | session[:user_id] = user.id |
|
|
12 | 15 | session[:admin] = user.admin? |
|
|
13 | 16 | |
|
|
14 | 17 | # clear forced logout flag for multicontests contest change |
|
|
15 | 18 | if GraderConfiguration.multicontests? |
|
|
16 | 19 | contest_stat = user.contest_stat |
| @@ -19,12 +19,17 | |||
|
|
19 | 19 | ="#{t 'login_label'}:" |
|
|
20 | 20 | %td= text_field_tag 'login' |
|
|
21 | 21 | %tr |
|
|
22 | 22 | %td{:align => "right"} |
|
|
23 | 23 | ="#{t 'password_label'}:" |
|
|
24 | 24 | %td= password_field_tag |
|
|
25 | + - unless GraderConfiguration['right.bypass_agreement'] | |
|
|
26 | + %tr | |
|
|
27 | + %td{:align => "right"}= check_box_tag 'accept_agree' | |
|
|
28 | + %td ยอมรับข้อตกลงการใช้งาน | |
|
|
29 | + | |
|
|
25 | 30 | = submit_tag t('login.login_submit') |
|
|
26 | 31 | %br/ |
|
|
27 | 32 | |
|
|
28 | 33 | - if GraderConfiguration['system.online_registration'] |
|
|
29 | 34 | =t 'login.participation' |
|
|
30 | 35 | %b |
| @@ -8,13 +8,13 | |||
|
|
8 | 8 | # system, you should be using db:schema:load, not running all the migrations |
|
|
9 | 9 | # from scratch. The latter is a flawed and unsustainable approach (the more migrations |
|
|
10 | 10 | # you'll amass, the slower it'll run and the greater likelihood for issues). |
|
|
11 | 11 | # |
|
|
12 | 12 | # It's strongly recommended to check this file into your version control system. |
|
|
13 | 13 | |
|
|
14 |
- ActiveRecord::Schema.define(:version => 20150914 |
|
|
|
14 | + ActiveRecord::Schema.define(:version => 20150914223258) do | |
|
|
15 | 15 | |
|
|
16 | 16 | create_table "announcements", :force => true do |t| |
|
|
17 | 17 | t.string "author" |
|
|
18 | 18 | t.text "body", :limit => 16777215 |
|
|
19 | 19 | t.boolean "published" |
|
|
20 | 20 | t.datetime "created_at", :null => false |
You need to be logged in to leave comments.
Login now