From d95728dd16b952890a7ac040b794af315f80d41f Mon Sep 17 00:00:00 2001 From: jeremylan Date: Thu, 17 Mar 2016 17:43:21 +1100 Subject: [PATCH] Fixed anonymizer when 'full name required' setting is on When the setting 'full name required' is on the anonymizer was trying to set the user name to nil and this caused the user name and email to remain not anonymized. Now in this scenario the user name is set to the anonimized username and the email is anonymized correctly. --- app/services/user_anonymizer.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/services/user_anonymizer.rb b/app/services/user_anonymizer.rb index e9ff162dbc6..ac692d28d76 100644 --- a/app/services/user_anonymizer.rb +++ b/app/services/user_anonymizer.rb @@ -20,7 +20,7 @@ class UserAnonymizer @user.reload @user.password = SecureRandom.hex @user.email = "#{@user.username}@example.com" - @user.name = nil + @user.name = SiteSetting.full_name_required ? @user.username : nil @user.date_of_birth = nil @user.title = nil @user.uploaded_avatar_id = nil