log when staff blocks/unblocks someone

app/models/user_history.rb

@@ -44,7 +44,9 @@ class UserHistory < ActiveRecord::Base
                           change_category_settings: 26,
                           delete_category: 27,
                           create_category: 28,
-                          change_site_text: 29)
+                          change_site_text: 29,
+                          block_user: 30,
+                          unblock_user: 31)
   end
 
   # Staff actions is a subset of all actions, used to audit actions taken by staff users.
@@ -70,7 +72,9 @@ class UserHistory < ActiveRecord::Base
                         :reviewed_post,
                         :change_category_settings,
                         :delete_category,
-                        :create_category]
+                        :create_category,
+                        :block_user,
+                        :unblock_user]
   end
 
   def self.staff_action_ids

app/services/staff_action_logger.rb

@@ -280,6 +280,22 @@ class StaffActionLogger
     }))
   end
 
+  def log_block_user(user, opts={})
+    raise Discourse::InvalidParameters.new(:user) unless user
+    UserHistory.create( params(opts).merge({
+      action: UserHistory.actions[:block_user],
+      target_user_id: user.id
+    }))
+  end
+
+  def log_unblock_user(user, opts={})
+    raise Discourse::InvalidParameters.new(:user) unless user
+    UserHistory.create( params(opts).merge({
+      action: UserHistory.actions[:unblock_user],
+      target_user_id: user.id
+    }))
+  end
+
   private
 
     def params(opts=nil)

app/services/user_blocker.rb

@@ -18,6 +18,7 @@ class UserBlocker
       @user.blocked = true
       if @user.save
         SystemMessage.create(@user, @opts[:message] || :blocked_by_staff)
+        StaffActionLogger.new(@by_user).log_block_user(@user) if @by_user
       end
     else
       false
@@ -34,6 +35,7 @@ class UserBlocker
     @user.blocked = false
     if @user.save
       SystemMessage.create(@user, :unblocked)
+      StaffActionLogger.new(@by_user).log_unblock_user(@user) if @by_user
     end
   end
 

config/locales/client.en.yml

@@ -2280,6 +2280,8 @@ en:
             change_category_settings: "change category settings"
             delete_category: "delete category"
             create_category: "create category"
+            block_user: "block user"
+            unblock_user: "unblock user"
         screened_emails:
           title: "Screened Emails"
           description: "When someone tries to create a new account, the following email addresses will be checked and the registration will be blocked, or some other action performed."