cafe-grader-web

Commit r469:b28cd479ede6 public

parent | child
Description:
modify user list creation into user list update
Commit status:
[Not Reviewed]
References:
algo
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

r469:b28cd479ede6 - - 1 file changed: 11 inserted, 5 deleted

Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -63,53 +63,59
63 end
63 end
64 end
64 end
65
65
66 def create_from_list
66 def create_from_list
67 lines = params[:user_list]
67 lines = params[:user_list]
68
68
69 note = []
69 note = []
70
70
71 lines.split("\n").each do |line|
71 lines.split("\n").each do |line|
72 items = line.chomp.split(',')
72 items = line.chomp.split(',')
73 if items.length>=2
73 if items.length>=2
74 login = items[0]
74 login = items[0]
75 full_name = items[1]
75 full_name = items[1]
76
76
77 added_random_password = false
77 added_random_password = false
78 if items.length>=3
78 if items.length>=3
79 password = items[2].chomp(" ")
79 password = items[2].chomp(" ")
80 user_alias = (items.length>=4) ? items[3] : login
80 user_alias = (items.length>=4) ? items[3] : login
81 else
81 else
82 password = random_password
82 password = random_password
83 user_alias = (items.length>=4) ? items[3] : login
83 user_alias = (items.length>=4) ? items[3] : login
84 added_random_password = true
84 added_random_password = true
85 end
85 end
86
86
87 - user = User.new({:login => login,
87 + user = User.find_by_login(login)
88 - :full_name => full_name,
88 + if (user)
89 - :password => password,
89 + user.full_name = full_name
90 - :password_confirmation => password,
90 + user.password = password
91 - :alias => user_alias})
91 + else
92 + user = User.new({:login => login,
93 + :full_name => full_name,
94 + :password => password,
95 + :password_confirmation => password,
96 + :alias => user_alias})
97 + end
92 user.activated = true
98 user.activated = true
93 user.save
99 user.save
94
100
95 if added_random_password
101 if added_random_password
96 note << "'#{login}' (+)"
102 note << "'#{login}' (+)"
97 else
103 else
98 note << login
104 note << login
99 end
105 end
100 end
106 end
101 end
107 end
102 flash[:notice] = 'User(s) ' + note.join(', ') +
108 flash[:notice] = 'User(s) ' + note.join(', ') +
103 ' were successfully created. ' +
109 ' were successfully created. ' +
104 '( (+) - created with random passwords.)'
110 '( (+) - created with random passwords.)'
105 redirect_to :action => 'list'
111 redirect_to :action => 'list'
106 end
112 end
107
113
108 def edit
114 def edit
109 @user = User.find(params[:id])
115 @user = User.find(params[:id])
110 end
116 end
111
117
112 def update
118 def update
113 @user = User.find(params[:id])
119 @user = User.find(params[:id])
114 if @user.update_attributes(params[:user])
120 if @user.update_attributes(params[:user])
115 flash[:notice] = 'User was successfully updated.'
121 flash[:notice] = 'User was successfully updated.'
You need to be logged in to leave comments. Login now