cafe-grader-web

Commit r340:f512a445945b public

parent | child
Description:
fixed form_tag bug in views
Commit status:
[Not Reviewed]
References:
default
Diff options:
Raw Diff | Patch Diff | Download Diff | Ignore 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

r340:f512a445945b - - 2 files changed: 12 inserted, 12 deleted

Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -153,7 +153,7
153 @prefix = params[:prefix] || ''
153 @prefix = params[:prefix] || ''
154 @non_admin_users = User.find_non_admin_with_prefix(@prefix)
154 @non_admin_users = User.find_non_admin_with_prefix(@prefix)
155 @changed = false
155 @changed = false
156 - if request.request_method == :post
156 + if request.request_method == 'POST'
157 @non_admin_users.each do |user|
157 @non_admin_users.each do |user|
158 password = random_password
158 password = random_password
159 user.password = password
159 user.password = password
Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -1,6 +1,6
1 %h1 Random user passwords
1 %h1 Random user passwords
2
2
3 - -if @changed
3 + - if @changed
4 %p
4 %p
5 %b Done!
5 %b Done!
6 Here's a new password list.
6 Here's a new password list.
@@ -12,28 +12,28
12 %th Login
12 %th Login
13 %th Fullname
13 %th Fullname
14 %th Password
14 %th Password
15 - -for u in @non_admin_users
15 + - for u in @non_admin_users
16 %tr
16 %tr
17 %td= u.login
17 %td= u.login
18 %td= u.full_name
18 %td= u.full_name
19 %td
19 %td
20 %tt= u.password
20 %tt= u.password
21
21
22 - -else
22 + - else
23 - -if @prefix!=''
23 + - if @prefix!=''
24 Current prefix:
24 Current prefix:
25 = @prefix
25 = @prefix
26 - -form_tag((url_for :action => 'random_all_passwords'), :method => 'get') do
26 + = form_tag((url_for :action => 'random_all_passwords'), :method => 'get') do
27 Change prefix
27 Change prefix
28 - =text_field_tag 'prefix'
28 + = text_field_tag 'prefix'
29 - =submit_tag 'Change'
29 + = submit_tag 'Change'
30
30
31 This will change passwords of the following users.
31 This will change passwords of the following users.
32 %ul
32 %ul
33 - -for u in @non_admin_users
33 + - for u in @non_admin_users
34 %li= u.login
34 %li= u.login
35
35
36 - -form_tag((url_for :action => 'random_all_passwords'), :method => 'post') do
36 + = form_tag((url_for :action => 'random_all_passwords'), :method => 'post') do
37 - =hidden_field_tag 'prefix', @prefix
37 + = hidden_field_tag 'prefix', @prefix
38 Are you sure?
38 Are you sure?
39 - =submit_tag 'Go ahead'
39 + = submit_tag 'Go ahead'
You need to be logged in to leave comments. Login now