cafe-grader-web

Commit r812:e2597379728c public

parent | child
Description:
fix bugs when user creation error
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

r812:e2597379728c - - 1 file changed: 1 inserted, 1 deleted

Show file before | Show file after | Raw diff | Download diff | Show whitespace Increase context Show commentsHide comments
@@ -48,49 +48,49
48 48 @user.save
49 49 redirect_to action: 'index', page: params[:page]
50 50 end
51 51
52 52 def create_from_list
53 53 lines = params[:user_list]
54 54
55 55
56 56 res = User.create_from_list(lines)
57 57 error_logins = res[:error_logins]
58 58 error_msg = res[:first_error]
59 59 ok_user = res[:created_users]
60 60
61 61
62 62 #add to group
63 63 if params[:add_to_group]
64 64 group = Group.find_by(id: params[:group_id])&.add_users_skip_existing(ok_user)
65 65 end
66 66
67 67 # show flash
68 68 if ok_user.count > 0
69 69 flash[:success] = "#{ok_user.count} user(s) was created or updated successfully"
70 70 end
71 71 if error_logins.size > 0
72 - flash[:error] = "Following user(s) failed to be created: " + error_note.join(', ') + ". The error of the first failed one are: " + error_msg;
72 + flash[:error] = "Following user(s) failed to be created: " + error_logins.join(', ') + ". The error of the first failed one are: " + error_msg;
73 73 end
74 74 redirect_to :action => 'index'
75 75 end
76 76
77 77 def edit
78 78 @user = User.find(params[:id])
79 79 end
80 80
81 81 def update
82 82 @user = User.find(params[:id])
83 83 if @user.update_attributes(user_params)
84 84 flash[:notice] = 'User was successfully updated.'
85 85 redirect_to :action => 'show', :id => @user
86 86 else
87 87 render :action => 'edit'
88 88 end
89 89 end
90 90
91 91 def destroy
92 92 User.find(params[:id]).destroy
93 93 redirect_to :action => 'index'
94 94 end
95 95
96 96 def user_stat
You need to be logged in to leave comments. Login now