admins: fix user lock and unlock && competition enroll error message

app/assets/javascripts/admins/users/index.js

@@ -15,7 +15,9 @@ $(document).on('turbolinks:load', function(){
       var $unlockAction = $lockAction.siblings('.unlock-action');
 
       var userId = $lockAction.data('id');
-
+      customConfirm({
+        content: '确认加锁吗？',
+        ok: function(){
           $.ajax({
             url: '/admins/users/' + userId + '/lock',
             method: 'POST',
@@ -26,6 +28,8 @@ $(document).on('turbolinks:load', function(){
               $unlockAction.show();
             }
           });
+        }
+      });
     });
 
     // unlock user
@@ -34,7 +38,9 @@ $(document).on('turbolinks:load', function(){
       var $lockAction = $unlockAction.siblings('.lock-action');
 
       var userId = $unlockAction.data('id');
-
+      customConfirm({
+        content: '确认解锁吗？',
+        ok: function () {
           $.ajax({
             url: '/admins/users/' + userId + '/unlock',
             method: 'POST',
@@ -45,6 +51,8 @@ $(document).on('turbolinks:load', function(){
               $unlockAction.hide();
             }
           });
+        }
+      })
     });
 
     // active user
@@ -54,7 +62,9 @@ $(document).on('turbolinks:load', function(){
       var $lockAction = $activeAction.siblings('.lock-action');
 
       var userId = $activeAction.data('id');
-
+      customConfirm({
+        content: '确认激活吗？',
+        ok: function () {
           $.ajax({
             url: '/admins/users/' + userId + '/unlock',
             method: 'POST',
@@ -66,6 +76,8 @@ $(document).on('turbolinks:load', function(){
               $unlockAction.hide();
             }
           });
+        }
+      })
     });
 
     // ***************** reward grade modal *****************

app/controllers/admins/users_controller.rb

@@ -32,13 +32,13 @@ class Admins::UsersController < Admins::BaseController
   end
 
   def lock
-    User.find(params[:user_id]).lock!
+    User.find(params[:id]).lock!
 
     render_ok
   end
 
   def unlock
-    User.find(params[:user_id]).activate!
+    User.find(params[:id]).activate!
 
     render_ok
   end

app/views/admins/users/shared/_user_list.html.erb

@@ -33,14 +33,14 @@
 
             <%= javascript_void_link('奖励', class: 'action reward-grade-action', data: { toggle: 'modal', target: '.admin-users-reward-grade-modal', id: user.id }) %>
 
-            <%= javascript_void_link '解锁', class: 'action unlock-action', data: { id: user.id, confirm: '确认解锁吗？' }, style: user.locked? ? '' : 'display: none;' %>
+            <%= javascript_void_link '解锁', class: 'action unlock-action', data: { id: user.id }, style: user.locked? ? '' : 'display: none;' %>
 
             <% if user.registered? %>
-              <%= javascript_void_link '激活', class: 'action active-action', data: { id: user.id, confirm: '确认激活吗？' } %>
+              <%= javascript_void_link '激活', class: 'action active-action', data: { id: user.id } %>
             <% end %>
 
             <% if user.id != current_user.id %>
-              <%= javascript_void_link '加锁', class: 'action lock-action', data: { id: user.id, confirm: '确认加锁吗？' }, style: user.locked? || user.registered? ? 'display: none;' : '' %>
+              <%= javascript_void_link '加锁', class: 'action lock-action', data: { id: user.id }, style: user.locked? || user.registered? ? 'display: none;' : '' %>
             <% end %>
 
             <%= delete_link '删除', admins_user_path(user, element: ".user-item-#{user.id}"), class: 'delete-user-action' %>

config/locales/forms/save_team_form.zh-CN.yml

@@ -17,9 +17,9 @@
             teacher_ids:
               enroll_forbidden: "本竞赛的参赛者限定为：学生"
               invalid_count: "教师数量应为%{minimum}~%{maximum}人"
-              enrolled: "教师 ${names} 已加入其它战队了"
+              enrolled: "教师 %{names} 已加入其它战队了"
             member_ids:
               enroll_forbidden: "本竞赛的参赛者限定为：教师"
               invalid_count: "队员数量应为%{minimum}~%{maximum}人"
-              enrolled: "队员 ${names} 已加入其它战队了"
+              enrolled: "队员 %{names} 已加入其它战队了"