From c0205d7d26f29470877fc573672000cdcbfd1298 Mon Sep 17 00:00:00 2001
From: "marius.balteanu" <marius.balteanu@zitec.com>
Date: Fri, 18 Mar 2022 23:56:29 +0200
Subject: [PATCH] Add tasks to prune registered users after a certain number of
 days (#30998).

---
 app/models/user.rb     |  4 ++++
 lib/tasks/redmine.rake | 16 ++++++++++++++++
 test/unit/user_test.rb | 10 ++++++++++
 3 files changed, 30 insertions(+)

diff --git a/app/models/user.rb b/app/models/user.rb
index eac3d82ae..425dbf26b 100644
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -874,6 +874,10 @@ class User < Principal
     project_ids.map(&:to_i)
   end
 
+  def self.prune(age)
+    User.where("created_on < ? AND status = ?", Time.now - age, STATUS_REGISTERED).destroy_all
+  end
+
   protected
 
   def validate_password_length
diff --git a/lib/tasks/redmine.rake b/lib/tasks/redmine.rake
index 112a9c5d3..83740c86a 100644
--- a/lib/tasks/redmine.rake
+++ b/lib/tasks/redmine.rake
@@ -40,6 +40,22 @@ namespace :redmine do
     end
   end
 
+  namespace :users do
+    desc 'Removes registered users that have not been activated after a number of days. Use DAYS to set the number of days, defaults to 30 days.'
+    task :prune => :environment do
+      days = 30
+      env_days = ENV['DAYS']
+      if env_days
+        if env_days.to_i <= 0
+          abort "Invalid DAYS #{env_days} given. The value must be a integer."
+        else
+          days = env_days.to_i
+        end
+      end
+      User.prune(days.days)
+    end
+  end
+
   namespace :watchers do
     desc 'Removes watchers from what they can no longer view.'
     task :prune => :environment do
diff --git a/test/unit/user_test.rb b/test/unit/user_test.rb
index 6590ce2f7..552419163 100644
--- a/test/unit/user_test.rb
+++ b/test/unit/user_test.rb
@@ -1348,4 +1348,14 @@ class UserTest < ActiveSupport::TestCase
     cv2a.reload
     assert_equal @dlopper.id.to_s, cv2a.value
   end
+
+  def test_prune_should_destroy_unactivated_old_users
+    User.generate!(:status => User::STATUS_REGISTERED, :created_on => 6.days.ago)
+    User.generate!(:status => User::STATUS_REGISTERED, :created_on => 7.day.ago)
+    User.generate!(:status => User::STATUS_REGISTERED)
+
+    assert_difference 'User.count', -2 do
+      User.prune(7)
+    end
+  end
 end
-- 
2.35.1