For those who are interested, I usually process this easily by:

<pre>
delete from users where status = 2;
</pre>

Not sure if it's really safe while redmine is running though.
--------------------------------------------------------------------------------
How about adding a rake task @redmine:users:prune@ to remove users who have not completed their registration?

Redmine already has a similarly named rake task @redmine:attachments:prune@ (source:tags/4.2.0/lib/tasks/redmine.rake#L20). The rake task deletes files that have been uploaded but not attached to an issue or wiki page.

--------------------------------------------------------------------------------
Go MAEDA wrote:
> How about adding a rake task @redmine:users:prune@ to remove users who have not completed their registration?

+1
I created a rake task @redmine:users:prune@.

<pre><code class="diff">
diff --git a/app/models/user.rb b/app/models/user.rb
index a3d2449d2..af65d5aaa 100644
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -887,6 +887,10 @@ class User < Principal
project_ids.map(&:to_i)
end

+ def self.prune(age=1.day)
+ 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 2bebaf18c..8cdf6f80f 100644
--- a/lib/tasks/redmine.rake
+++ b/lib/tasks/redmine.rake
@@ -40,6 +40,13 @@ namespace :redmine do
end
end

+ namespace :users do
+ desc 'Remove users who could not be activated after one day.'
+ task :prune => :environment do
+ User.prune
+ end
+ end
+
namespace :watchers do
desc 'Removes watchers from what they can no longer view.'
task :prune => :environment do
</code></pre>

--------------------------------------------------------------------------------

--------------------------------------------------------------------------------
Thank you for the patch.

I have changed the patch to remove users registered more than 7 days ago. I think one day is too short.

Suppose that you set "Self-registration" to "manual account activation" and run the rake task every day. When the admin receives a notification that a user has registered, the admin must activate the user within one day or the user will be deleted. All users who register after Friday afternoon may be deleted by Monday morning.
--------------------------------------------------------------------------------
Updated the patch for the trunk r21318. Setting the target version to 5.0.0.
--------------------------------------------------------------------------------
The rake task should accept the number of days as parameter in order to make this configurable. Also, the default value should be at least 7 days or even 30 days. One day is too less.
--------------------------------------------------------------------------------
Go Maeda, what do you think about the following implementation?
--------------------------------------------------------------------------------

--------------------------------------------------------------------------------
Marius BALTEANU wrote:
> Go Maeda, what do you think about the following implementation?

Looks nice, thank you!
--------------------------------------------------------------------------------
Committed the patch. Thank you.
--------------------------------------------------------------------------------