From 85cb434b2c8776924dc30a754bcc4fc711958c9e Mon Sep 17 00:00:00 2001
From: evazion <noizave@gmail.com>
Date: Sun, 2 Oct 2022 03:42:51 -0500
Subject: [PATCH] users: fix bug in invalid username deletion script.

Fix a bug in script/fixes/115_delete_invalid_users.rb where certain
usernames containing punctuation weren't deleted.
---
 script/fixes/115_delete_invalid_users.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/script/fixes/115_delete_invalid_users.rb b/script/fixes/115_delete_invalid_users.rb
index 9d987ec35..f83adc7db 100755
--- a/script/fixes/115_delete_invalid_users.rb
+++ b/script/fixes/115_delete_invalid_users.rb
@@ -14,8 +14,8 @@ def delete(user)
 
     puts "[WARN] id=#{user.id} user='#{user.name}' email='#{user.email_address.address}'"
   elsif ENV.fetch("DELETE", "false").truthy?
-    UserDeletion.new(user: user, deleter: User.owner).delete!
     puts "[DELETE] id=#{user.id} user='#{user.name}'"
+    UserDeletion.new(user: user, deleter: User.owner).delete!
   end
 end
 
@@ -47,7 +47,7 @@ with_confirmation do
     delete(user)
   end
 
-  users.where_regex(:name, "[`~!@#$%^&*()+={}\[\]|\\:;'\"<>,?/]").find_each do |user|
+  users.where_regex(:name, "[][`~!@#$%^&*()+={}|\\:;'\"<>,?/]").find_each do |user|
     delete(user)
   end