Replace 'account' with 'user' everywhere.
authorMichael Orlitzky <michael@orlitzky.com>
Sat, 4 Jan 2014 18:07:57 +0000 (13:07 -0500)
committerMichael Orlitzky <michael@orlitzky.com>
Sat, 4 Jan 2014 18:07:57 +0000 (13:07 -0500)
31 files changed:
bin/mailshears
doc/TODO
lib/common/agendav_plugin.rb
lib/common/davical_plugin.rb
lib/common/dovecot_plugin.rb
lib/common/errors.rb
lib/common/plugin.rb
lib/common/postfixadmin_plugin.rb
lib/common/roundcube_plugin.rb
lib/mv/mv_dummy_runner.rb
lib/mv/mv_plugin.rb
lib/mv/plugins/agendav.rb
lib/mv/plugins/davical.rb
lib/mv/plugins/dovecot.rb
lib/mv/plugins/roundcube.rb
lib/prune/plugins/agendav.rb
lib/prune/plugins/davical.rb
lib/prune/plugins/dovecot.rb
lib/prune/plugins/postfixadmin.rb
lib/prune/plugins/roundcube.rb
lib/prune/prune_dummy_runner.rb
lib/prune/prune_plugin.rb
lib/prune/prune_runner.rb
lib/rm/plugins/agendav.rb
lib/rm/plugins/davical.rb
lib/rm/plugins/dovecot.rb
lib/rm/plugins/postfixadmin.rb
lib/rm/plugins/roundcube.rb
lib/rm/rm_dummy_runner.rb
lib/rm/rm_plugin.rb
lib/rm/rm_runner.rb

index 28c165cee765492bbaeb28f91ae76a54da750631..f7b1963cc8bde0a49cfc388fa29d963cf8e3db55 100755 (executable)
@@ -2,14 +2,14 @@
 #
 # mailshears, to prune unused mail directories.
 #
 #
 # mailshears, to prune unused mail directories.
 #
-# Mail accounts for virtual hosts are stored in SQL, and managed by
+# Mail users for virtual hosts are stored in SQL, and managed by
 # Postfixadmin. However, the physical directories are handled by
 # Postfix/Dovecot and are left untouched by Postfixadmin. This is good
 # for security, but comes at a cost: Postfixadmin can't remove a
 # user's mail directory when his or her account is deleted.
 #
 # Postfixadmin. However, the physical directories are handled by
 # Postfix/Dovecot and are left untouched by Postfixadmin. This is good
 # for security, but comes at a cost: Postfixadmin can't remove a
 # user's mail directory when his or her account is deleted.
 #
-# This program compares the list of filesystem accounts with the ones
-# in the database. It outputs any accounts that exist in the
+# This program compares the list of filesystem users with the ones
+# in the database. It outputs any users that exist in the
 # filesystem, but not the database.
 #
 
 # filesystem, but not the database.
 #
 
@@ -62,7 +62,7 @@ require "#{mode_name}/#{mode_name}_runner"
 require "#{mode_name}/#{mode_name}_dummy_runner"
 
 def make_header(plugin_name)
 require "#{mode_name}/#{mode_name}_dummy_runner"
 
 def make_header(plugin_name)
-  # The header that we output before the list of domains/accounts.
+  # The header that we output before the list of domains/users.
   # Just the path of this script, the current time, and the plugin name.
   exe = File.basename($PROGRAM_NAME)
   header = "#{exe}, "
   # Just the path of this script, the current time, and the plugin name.
   exe = File.basename($PROGRAM_NAME)
   header = "#{exe}, "
index 4f9c0f2bf366f78192dafee6a2b2949a1d032a1c..dd6db29f6879b9407badb7eb1048dcccaf65fbb8 100644 (file)
--- a/doc/TODO
+++ b/doc/TODO
@@ -1,10 +1,8 @@
-* Error reporting sucks, and when a domain or account doesn't exist we
-  should be able to say so. The describe_domain/account functions
+* Error reporting sucks, and when a domain or user doesn't exist we
+  should be able to say so. The describe_domain/user functions
   should also work better. For plugins that don't implement domains,
   should also work better. For plugins that don't implement domains,
-  we can return a (count of?) list of accounts, or fall back to the
-  account deletion descriptions.
-
-* Rename "account" to "user" everywhere.
+  we can return a (count of?) list of users, or fall back to the
+  user deletion descriptions.
 
 * Implement "mv".
 
 
 * Implement "mv".
 
index a29f3ae84a7df1d712badacfc53bc3d3f245b345..7522b2cfce630a7340b69fadd5d7279ea4b9ffd9 100644 (file)
@@ -23,9 +23,9 @@ module AgendavPlugin
   end
 
 
   end
 
 
-  def describe_account(account)
-    if self.user_exists(account)
-      return "Username: #{account}"
+  def describe_user(user)
+    if self.user_exists(user)
+      return "Username: #{user}"
     else
       return 'User not found'
     end
     else
       return 'User not found'
     end
index afd1d883e66115c32964fb9f9cd61f58c0622a93..a1dcba07b690b1f07eace23804597d9c61db977b 100644 (file)
@@ -22,8 +22,8 @@ module DavicalPlugin
   end
 
 
   end
 
 
-  def describe_account(account)
-    principal_id = self.get_principal_id(account)
+  def describe_user(user)
+    principal_id = self.get_principal_id(user)
 
     if principal_id.nil?
       return 'User not found'
 
     if principal_id.nil?
       return 'User not found'
@@ -35,7 +35,7 @@ module DavicalPlugin
 
   protected;
 
 
   protected;
 
-  def get_principal_id(account)
+  def get_principal_id(user)
     principal_id = nil
 
     begin
     principal_id = nil
 
     begin
@@ -52,7 +52,7 @@ module DavicalPlugin
       sql_query += "      ON principal.user_no = usr.user_no) "
       sql_query += "WHERE usr.username = $1;"
 
       sql_query += "      ON principal.user_no = usr.user_no) "
       sql_query += "WHERE usr.username = $1;"
 
-      connection.query(sql_query, [account]) do |result|
+      connection.query(sql_query, [user]) do |result|
         if result.num_tuples > 0
           principal_id = result[0]['principal_id']
         end
         if result.num_tuples > 0
           principal_id = result[0]['principal_id']
         end
index ab5a40930400af057229ded188d40516cc88b866..02e92957cf6994df0e01ebc1a67fa4d07ffbe591 100644 (file)
@@ -19,11 +19,11 @@ module DovecotPlugin
     end
   end
 
     end
   end
 
-  def describe_account(account)
+  def describe_user(user)
     begin
     begin
-      account_path = get_account_path(account)
-      return account_path
-    rescue NonexistentAccountError => e
+      user_path = get_user_path(user)
+      return user_path
+    rescue NonexistentUserError => e
       return "Doesn't exist: #{e.to_s}"
     end
   end
       return "Doesn't exist: #{e.to_s}"
     end
   end
@@ -45,30 +45,30 @@ module DovecotPlugin
   end
 
 
   end
 
 
-  def get_account_path(account)
-    # Return the filesystem path of this account's mailbox.
-    # Only works if the account exists!
-    if not account.include?('@')
-      msg = "#{account}: Accounts must contain an '@' symbol."
-      raise InvalidAccountError.new(msg)
+  def get_user_path(user)
+    # Return the filesystem path of this user's mailbox.
+    # Only works if the user exists!
+    if not user.include?('@')
+      msg = "#{user}: Users must contain an '@' symbol."
+      raise InvalidUserError.new(msg)
     end
 
     end
 
-    account_parts = account.split('@')
-    user_part = account_parts[0]
-    domain_part = account_parts[1]
+    user_parts = user.split('@')
+    local_part = user_parts[0]
+    domain_part = user_parts[1]
 
     begin
       domain_path = get_domain_path(domain_part)
     rescue NonexistentDomainError
 
     begin
       domain_path = get_domain_path(domain_part)
     rescue NonexistentDomainError
-      raise NonexistentAccountError.new(account)
+      raise NonexistentUserError.new(user)
     end
 
     end
 
-    account_path = File.join(domain_path, user_part)
+    user_path = File.join(domain_path, local_part)
 
 
-    if File.directory?(account_path)
-      return account_path
+    if File.directory?(user_path)
+      return user_path
     else
     else
-      raise NonexistentAccountError.new(account)
+      raise NonexistentUserError.new(user)
     end
   end
 
     end
   end
 
@@ -78,25 +78,25 @@ module DovecotPlugin
   end
 
   def list_domains_users(domains)
   end
 
   def list_domains_users(domains)
-    accounts = []
+    users = []
 
     domains.each do |domain|
       begin
         # Throws a NonexistentDomainError if the domain's path
         # doesn't exist on the filesystem. In this case, we want
 
     domains.each do |domain|
       begin
         # Throws a NonexistentDomainError if the domain's path
         # doesn't exist on the filesystem. In this case, we want
-        # to report zero accounts.
+        # to report zero users.
         domain_path = get_domain_path(domain)
         usernames = Filesystem.get_subdirs(domain_path)
 
         usernames.each do |username|
         domain_path = get_domain_path(domain)
         usernames = Filesystem.get_subdirs(domain_path)
 
         usernames.each do |username|
-          accounts << "#{username}@#{domain}"
+          users << "#{username}@#{domain}"
         end
       rescue NonexistentDomainError
         # Party hard.
       end
     end
 
         end
       rescue NonexistentDomainError
         # Party hard.
       end
     end
 
-    return accounts
+    return users
   end
 
 
   end
 
 
index 3607b8527d772fbb4fcf79f6062f21bb08accd9c..d0bcfe200ef14c187c29ac6bfe26ba8cb3be0be6 100644 (file)
@@ -4,14 +4,14 @@ class DatabaseError < StandardError
 end
 
 
 end
 
 
-# Perhaps surprisingly, used to indicate that an account name is
+# Perhaps surprisingly, used to indicate that an user name is
 # invalid.
 # invalid.
-class InvalidAccountError < StandardError
+class InvalidUserError < StandardError
 end
 
 
 end
 
 
-# Used to indicate that an account does not exist on the filesystem.
-class NonexistentAccountError < StandardError
+# Used to indicate that an user does not exist on the filesystem.
+class NonexistentUserError < StandardError
 end
 
 
 end
 
 
index 77d80c469e7b1e94f794b84b0c532f632765f77e..b0b57505873665a856dcf7ade0eafc4b82cf266a 100644 (file)
@@ -21,8 +21,8 @@ module Plugin
     raise NotImplementedError
   end
 
     raise NotImplementedError
   end
 
-  def describe_account(account)
-    # Provide a "description" of the account. This is output along
+  def describe_user(user)
+    # Provide a "description" of the user. This is output along
     # with the domain name and can be anything of use to the system
     # administrator.
     raise NotImplementedError
     # with the domain name and can be anything of use to the system
     # administrator.
     raise NotImplementedError
index 7a1985202cc343af0301518dc9a9e94abf03c799..5556d5a7c23a15866a0a7fb17e6cbb77f2a21f1d 100644 (file)
@@ -17,9 +17,9 @@ module PostfixadminPlugin
   end
 
 
   end
 
 
-  def describe_account(account)
+  def describe_user(user)
     # There's no other unique identifier in PostfixAdmin
     # There's no other unique identifier in PostfixAdmin
-    return account
+    return user
   end
 
 
   end
 
 
@@ -58,7 +58,7 @@ module PostfixadminPlugin
 
 
   def list_users()
 
 
   def list_users()
-    accounts = []
+    users = []
 
     # Just assume PostgreSQL for now.
     begin
 
     # Just assume PostgreSQL for now.
     begin
@@ -74,7 +74,7 @@ module PostfixadminPlugin
       # deliver to that address; it's not an alias."
       sql_query = 'SELECT username FROM mailbox;'
       connection.query(sql_query) do |result|
       # deliver to that address; it's not an alias."
       sql_query = 'SELECT username FROM mailbox;'
       connection.query(sql_query) do |result|
-        accounts = result.field_values('username')
+        users = result.field_values('username')
       end
       connection.close()
     rescue PGError => e
       end
       connection.close()
     rescue PGError => e
@@ -82,7 +82,7 @@ module PostfixadminPlugin
       raise DatabaseError.new(e)
     end
 
       raise DatabaseError.new(e)
     end
 
-    return accounts
+    return users
   end
 
 
   end
 
 
index 8575a11eae4e3ee214ba6a7d1cc317a0c018951a..0f80f72dd5ab33bad582fe15298aa4287f7e4ed7 100644 (file)
@@ -21,8 +21,8 @@ module RoundcubePlugin
     return domain
   end
 
     return domain
   end
 
-  def describe_account(account)
-    user_id = self.get_user_id(account)
+  def describe_user(user)
+    user_id = self.get_user_id(user)
 
     if user_id.nil?
       return 'User not found'
 
     if user_id.nil?
       return 'User not found'
@@ -34,7 +34,7 @@ module RoundcubePlugin
 
   protected;
 
 
   protected;
 
-  def get_user_id(account)
+  def get_user_id(user)
     user_id = nil
 
     begin
     user_id = nil
 
     begin
@@ -48,7 +48,7 @@ module RoundcubePlugin
 
       sql_query = "SELECT user_id FROM users WHERE username = $1;"
 
 
       sql_query = "SELECT user_id FROM users WHERE username = $1;"
 
-      connection.query(sql_query, [account]) do |result|
+      connection.query(sql_query, [user]) do |result|
         if result.num_tuples > 0
           user_id = result[0]['user_id']
         end
         if result.num_tuples > 0
           user_id = result[0]['user_id']
         end
index 0d06fc180820397016c5e6c62a1e7c2953dddcc3..ff612ffeb175e7bc4b663c0f18214055f5946890 100644 (file)
@@ -5,7 +5,7 @@ class MvDummyRunner
 
   def run(plugin, src, dst)
     if src.include?('@') then
 
   def run(plugin, src, dst)
     if src.include?('@') then
-      puts "Would move account: #{src} to #{dst}"
+      puts "Would move user: #{src} to #{dst}"
     else
       puts "Would move domain: #{src} to #{dst}"
     end
     else
       puts "Would move domain: #{src} to #{dst}"
     end
index 2ccb9a388572acedcb3dbb67bc36120f06381347..40184422fbd8f9a5601fa3a57a92632163007d47 100644 (file)
@@ -1,6 +1,6 @@
 module MvPlugin
   #
 module MvPlugin
   #
-  # Plugins for moving (renaming) accounts.
+  # Plugins for moving (renaming) users.
   #
 
   def MvPlugin.included(c)
   #
 
   def MvPlugin.included(c)
@@ -20,8 +20,8 @@ module MvPlugin
     raise NotImplementedError
   end
 
     raise NotImplementedError
   end
 
-  def mv_account(from, to)
-    # Rename the given account.
+  def mv_user(from, to)
+    # Rename the given user.
     raise NotImplementedError
   end
 
     raise NotImplementedError
   end
 
index ab12c73f18473e5735d11f62bece80f1386ca75c..112fa2ab52f307e6dcff9fbe3e65f93f213ed7d5 100644 (file)
@@ -13,7 +13,7 @@ class AgendavMv
     # AgenDAV doesn't have a concept of domains.
   end
 
     # AgenDAV doesn't have a concept of domains.
   end
 
-  def mv_account(from, to)
+  def mv_user(from, to)
     sql_queries = ['UPDATE prefs SET username = $1 WHERE username $2;']
     sql_queries << 'UPDATE shared SET user_from = $1 WHERE user_from = $2;'
     sql_queries << 'UPDATE shared SET user_which = $1 WHERE user_which = $2;'
     sql_queries = ['UPDATE prefs SET username = $1 WHERE username $2;']
     sql_queries << 'UPDATE shared SET user_from = $1 WHERE user_from = $2;'
     sql_queries << 'UPDATE shared SET user_which = $1 WHERE user_which = $2;'
index ee47d3039fd9feb71235000ff0f93e4f6dfe69ff..63c185c1c63835f5fdd0f52a81e243d108a45eea 100644 (file)
@@ -17,7 +17,7 @@ class DavicalMv
   end
 
 
   end
 
 
-  def mv_account(from, to)
+  def mv_user(from, to)
     # Delete the given username. DAViCal uses foreign keys properly
     # and only supports postgres, so we let the ON UPDATE CASCADE
     # trigger handle most of the work.
     # Delete the given username. DAViCal uses foreign keys properly
     # and only supports postgres, so we let the ON UPDATE CASCADE
     # trigger handle most of the work.
index 0eb688861442aba8e07e8cad640a92ce7ec30abc..1b0ef79426ef40533d34c6932943f2ab5cb8824d 100644 (file)
@@ -15,9 +15,9 @@ class DovecotMv < Mailstore
     FileUtils.mv(from_path, to_path)
   end
 
     FileUtils.mv(from_path, to_path)
   end
 
-  def mv_account(from, to)
-    from_path = self.get_account_path(from)
-    to_path = self.get_account_path(to)
+  def mv_user(from, to)
+    from_path = self.get_user_path(from)
+    to_path = self.get_user_path(to)
     FileUtils.mv(from_path, to_path)
   end
 
     FileUtils.mv(from_path, to_path)
   end
 
index 50a52c302cd2a25613f4c463eb5232c7a756b055..bac8344607d925c26f8652aa9a141ff87eee736c 100644 (file)
@@ -13,7 +13,7 @@ class RoundcubeMv
     # Roundcube doesn't have a concept of domains.
   end
 
     # Roundcube doesn't have a concept of domains.
   end
 
-  def mv_account(from, to)
+  def mv_user(from, to)
     sql_queries = ['UPDATE users SET username = $1 WHERE username = $2;']
 
     begin
     sql_queries = ['UPDATE users SET username = $1 WHERE username = $2;']
 
     begin
index 083a3e42731c6aeb59391b2d96a8387885a77cd6..43b5c99e883cf4287438297308b43769345806aa 100644 (file)
@@ -10,15 +10,15 @@ class AgendavPrune < AgendavRm
   def get_leftover_domains(db_domains)
     # AgenDAV doesn't have a concept of domains. We could parse the
     # usernames to see what domains are present, but the point is
   def get_leftover_domains(db_domains)
     # AgenDAV doesn't have a concept of domains. We could parse the
     # usernames to see what domains are present, but the point is
-    # moot: all leftover accounts will be pruned anyway.
+    # moot: all leftover users will be pruned anyway.
     return []
   end
 
 
     return []
   end
 
 
-  def get_leftover_accounts(db_accounts)
+  def get_leftover_users(db_users)
     # Get a list of all users who have logged in to AgenDAV.
     # Get a list of all users who have logged in to AgenDAV.
-    ad_accounts = self.list_users()
-    return ad_accounts - db_accounts
+    ad_users = self.list_users()
+    return ad_users - db_users
   end
 
 end
   end
 
 end
index d13684eb26414a4cb6cd5f44c6cadbd9550ad274..3c5ff0e3401a7087c6a56810f3958866323d92af 100644 (file)
@@ -14,15 +14,15 @@ class DavicalPrune < DavicalRm
   def get_leftover_domains(db_domains)
     # DAViCal doesn't have a concept of domains. We could parse the
     # usernames to see what domains are present, but the point is
   def get_leftover_domains(db_domains)
     # DAViCal doesn't have a concept of domains. We could parse the
     # usernames to see what domains are present, but the point is
-    # moot: all leftover accounts will be pruned anyway.
+    # moot: all leftover users will be pruned anyway.
     return []
   end
 
 
     return []
   end
 
 
-  def get_leftover_accounts(db_accounts)
+  def get_leftover_users(db_users)
     # Get a list of all users who have logged in to DAViCal.
     # Get a list of all users who have logged in to DAViCal.
-    davical_accounts = self.list_users()
-    return davical_accounts - db_accounts
+    davical_users = self.list_users()
+    return davical_users - db_users
   end
 
 
   end
 
 
index 88e5c1cb875351293758c945608f8a1a5f363590..bf96420d209753da032d77bd4cbcb7e71d3d5f3e 100644 (file)
@@ -14,13 +14,13 @@ class DovecotPrune < DovecotRm
     return (fs_domains - db_domains)
   end
 
     return (fs_domains - db_domains)
   end
 
-  def get_leftover_accounts(db_accounts)
-    # Get the list of accounts according to the filesystem.
+  def get_leftover_users(db_users)
+    # Get the list of users according to the filesystem.
     fs_domains = self.list_domains()
     fs_domains = self.list_domains()
-    fs_accounts = self.list_domains_users(fs_domains)
+    fs_users = self.list_domains_users(fs_domains)
 
 
-    # And return the accounts on the filesystem that aren't in the DB.
-    return (fs_accounts - db_accounts)
+    # And return the users on the filesystem that aren't in the DB.
+    return (fs_users - db_users)
   end
 
 end
   end
 
 end
index 62e564710166468e142ddabc5fe9f6d3c404bf2e..395fb240cd47e96c6751fc163d762b9f9d8f7467 100644 (file)
@@ -2,7 +2,7 @@ require 'prune/prune_plugin'
 require 'rm/plugins/postfixadmin'
 
 class PostfixadminPrune < PostfixadminRm
 require 'rm/plugins/postfixadmin'
 
 class PostfixadminPrune < PostfixadminRm
-  # We don't need the ability to remove "left over" accounts or
+  # We don't need the ability to remove "left over" users or
   # domains, since "left over" is with respect to what's in
   # PostfixAdmin. In other words, the other plugins check themselves
   # against PostfixAdmin, it doesn't make sense to check PostfixAdmin
   # domains, since "left over" is with respect to what's in
   # PostfixAdmin. In other words, the other plugins check themselves
   # against PostfixAdmin, it doesn't make sense to check PostfixAdmin
index 1ad8cdb786b989c3c86d9be1b868a9f4e85c4992..1bb782700748644905b0fb1141ddd9eb5fb63127 100644 (file)
@@ -11,15 +11,15 @@ class RoundcubePrune < RoundcubeRm
   def get_leftover_domains(db_domains)
     # Roundcube doesn't have a concept of domains. We could parse the
     # usernames to see what domains are present, but the point is
   def get_leftover_domains(db_domains)
     # Roundcube doesn't have a concept of domains. We could parse the
     # usernames to see what domains are present, but the point is
-    # moot: all leftover accounts will be pruned anyway.
+    # moot: all leftover users will be pruned anyway.
     return []
   end
 
 
     return []
   end
 
 
-  def get_leftover_accounts(db_accounts)
+  def get_leftover_users(db_users)
     # Get a list of all users who have logged in to Roundcube.
     # Get a list of all users who have logged in to Roundcube.
-    rc_accounts = self.list_users()
-    return rc_accounts - db_accounts
+    rc_users = self.list_users()
+    return rc_users - db_users
   end
 
 end
   end
 
 end
index 158ed5f005bef8ceefdfe6c5a0c4469137ddb510..6cb789fcee6c4243f7e4a819eaf55fc8f8102e76 100644 (file)
@@ -16,11 +16,11 @@ class PruneDummyRunner
     db_users = pfa.list_users()
     db_domains = pfa.list_domains()
 
     db_users = pfa.list_users()
     db_domains = pfa.list_domains()
 
-    leftover_users = plugin.get_leftover_accounts(db_users)
+    leftover_users = plugin.get_leftover_users(db_users)
     leftover_domains = plugin.get_leftover_domains(db_domains)
 
     leftover_users.each do |user|
     leftover_domains = plugin.get_leftover_domains(db_domains)
 
     leftover_users.each do |user|
-      user_description = plugin.describe_account(user)
+      user_description = plugin.describe_user(user)
       report(plugin, "Would remove user: #{user} (#{user_description})")
     end
 
       report(plugin, "Would remove user: #{user} (#{user_description})")
     end
 
index 2776704a4135b5be897945a6683d4bb24dbe5d61..e47a5e4aafbd6370fcf07aca76e30b8804fa34c0 100644 (file)
@@ -4,8 +4,8 @@ module PrunePlugin
   include RmPlugin
 
   #
   include RmPlugin
 
   #
-  # Plugins for the removal of leftover non-PostfixAdmin accounts,
-  # i.e. after an account has been removed from the PostfixAdmin
+  # Plugins for the removal of leftover non-PostfixAdmin users,
+  # i.e. after an user has been removed from the PostfixAdmin
   # database.
   #
 
   # database.
   #
 
@@ -27,8 +27,8 @@ module PrunePlugin
     raise NotImplementedError
   end
 
     raise NotImplementedError
   end
 
-  def get_leftover_accounts(db_accounts)
-    # Given a list of accounts, determine which accounts belonging to
+  def get_leftover_users(db_users)
+    # Given a list of users, determine which users belonging to
     # this plugin are not contained in the given list.
     raise NotImplementedError
   end
     # this plugin are not contained in the given list.
     raise NotImplementedError
   end
index acc8828ba11be426a0c0ad90b1fe991c25f7154a..7ea61919ca46981c4284c4744854355756f9e3be 100644 (file)
@@ -16,12 +16,12 @@ class PruneRunner
     db_users = pfa.list_users()
     db_domains = pfa.list_domains()
 
     db_users = pfa.list_users()
     db_domains = pfa.list_domains()
 
-    leftover_users = plugin.get_leftover_accounts(db_users)
+    leftover_users = plugin.get_leftover_users(db_users)
     leftover_domains = plugin.get_leftover_domains(db_domains)
 
     leftover_users.each do |user|
     leftover_domains = plugin.get_leftover_domains(db_domains)
 
     leftover_users.each do |user|
-      user_description = plugin.describe_account(user)
-      plugin.delete_account(user)
+      user_description = plugin.describe_users(user)
+      plugin.delete_user(user)
       report(plugin, "Removed user: #{user} (#{user_description})")
     end
 
       report(plugin, "Removed user: #{user} (#{user_description})")
     end
 
index 2b2952f01a9254e01331b7a663d632762b8eb0ea..0f4e790f6f4007f124eed570db7d52651f767d8a 100644 (file)
@@ -9,10 +9,10 @@ class AgendavRm
   include RmPlugin
 
 
   include RmPlugin
 
 
-  def delete_account(account)
+  def delete_user(user)
     # Delete the given username and any records in other tables
     # belonging to it.
     # Delete the given username and any records in other tables
     # belonging to it.
-    raise NonexistentAccountError.new(account) if not user_exists(account)
+    raise NonexistentUserError.new(user) if not user_exists(user)
 
     sql_queries = ['DELETE FROM prefs WHERE username = $1;']
     sql_queries << 'DELETE FROM shared WHERE user_from = $1;'
 
     sql_queries = ['DELETE FROM prefs WHERE username = $1;']
     sql_queries << 'DELETE FROM shared WHERE user_from = $1;'
@@ -27,7 +27,7 @@ class AgendavRm
                                   @db_pass)
 
       sql_queries.each do |sql_query|
                                   @db_pass)
 
       sql_queries.each do |sql_query|
-        connection.query(sql_query, [account])
+        connection.query(sql_query, [user])
       end
 
       connection.close()
       end
 
       connection.close()
index 04d5fbf024dde17e47375afa486d4359c897c9dd..f7d116e0e783a08364701c97334668d04ca14d45 100644 (file)
@@ -12,11 +12,11 @@ class DavicalRm
   include RmPlugin
 
 
   include RmPlugin
 
 
-  def delete_account(account)
+  def delete_user(user)
     # Delete the given username. DAViCal uses foreign keys properly
     # and only supports postgres, so we let the ON DELETE CASCADE
     # trigger handle most of the work.
     # Delete the given username. DAViCal uses foreign keys properly
     # and only supports postgres, so we let the ON DELETE CASCADE
     # trigger handle most of the work.
-    raise NonexistentAccountError.new(account) if not user_exists(account)
+    raise NonexistentUserError.new(user) if not user_exists(user)
 
     sql_queries = ['DELETE FROM usr WHERE username = $1']
 
 
     sql_queries = ['DELETE FROM usr WHERE username = $1']
 
@@ -30,7 +30,7 @@ class DavicalRm
                                   @db_pass)
 
       sql_queries.each do |sql_query|
                                   @db_pass)
 
       sql_queries.each do |sql_query|
-        connection.query(sql_query, [account])
+        connection.query(sql_query, [user])
       end
 
       connection.close()
       end
 
       connection.close()
index 4283e9c2267f9b0088febcd4d5b45ea423ddb660..837a7fd68b4da94c3c8363d64035622c410796be 100644 (file)
@@ -16,10 +16,10 @@ class DovecotRm
     FileUtils.rm_r(domain_path)
   end
 
     FileUtils.rm_r(domain_path)
   end
 
-  def delete_account(account)
+  def delete_user(user)
     # Will raise an exception if the path doesn't exist.
     # Will raise an exception if the path doesn't exist.
-    account_path = self.get_account_path(account)
-    FileUtils.rm_r(account_path)
+    user_path = self.get_user_path(user)
+    FileUtils.rm_r(user_path)
   end
 
 end
   end
 
 end
index 3883e866103e71dff7cf75ecb37681cebca7ca11..94526bcd25bfc058a1c05bd92081caeb16fa5caa 100644 (file)
@@ -9,11 +9,11 @@ class PostfixadminRm
   include RmPlugin
 
 
   include RmPlugin
 
 
-  def delete_account(account)
-    raise NonexistentAccountError.new(account) if not user_exists(account)
+  def delete_user(user)
+    raise NonexistentUserError.new(user) if not user_exists(user)
 
     sql_queries = ['DELETE FROM alias WHERE address = $1;']
 
     sql_queries = ['DELETE FROM alias WHERE address = $1;']
-    # Wipe out any aliases pointed at our account.
+    # Wipe out any aliases pointed at our user.
     sql_queries << "UPDATE alias SET goto=REPLACE(goto, $1, '');"
     sql_queries << 'DELETE FROM mailbox WHERE username = $1;'
     sql_queries << 'DELETE FROM quota WHERE username = $1;'
     sql_queries << "UPDATE alias SET goto=REPLACE(goto, $1, '');"
     sql_queries << 'DELETE FROM mailbox WHERE username = $1;'
     sql_queries << 'DELETE FROM quota WHERE username = $1;'
@@ -33,7 +33,7 @@ class PostfixadminRm
                                   @db_pass)
 
       sql_queries.each do |sql_query|
                                   @db_pass)
 
       sql_queries.each do |sql_query|
-        connection.query(sql_query, [account])
+        connection.query(sql_query, [user])
       end
 
       # The earlier alias update query will leave things like
       end
 
       # The earlier alias update query will leave things like
index 3dbee8ea61b948033f6ffd8df94ba86d31535e24..23782d20a575471820aa279f7ce47b346e68f2e2 100644 (file)
@@ -8,12 +8,12 @@ class RoundcubeRm
   include RoundcubePlugin
   include RmPlugin
 
   include RoundcubePlugin
   include RmPlugin
 
-  def delete_account(account)
+  def delete_user(user)
     # Delete the given username and any records in other tables
     # belonging to it.
     # Delete the given username and any records in other tables
     # belonging to it.
-    raise NonexistentAccountError.new(account) if not user_exists(account)
+    raise NonexistentUserError.new(user) if not user_exists(user)
 
 
-    user_id = self.get_user_id(account)
+    user_id = self.get_user_id(user)
 
     # The Roundcube developers were nice enough to include
     # DBMS-specific install and upgrade scripts, so Postgres can take
 
     # The Roundcube developers were nice enough to include
     # DBMS-specific install and upgrade scripts, so Postgres can take
index 091d32881bb9940f5f72c5ff2f67c3ba0030010e..60cf23ab8b32c12eefd06fce87a659c697689f0d 100644 (file)
@@ -6,7 +6,7 @@ class RmDummyRunner
   def run(plugin, *targets)
     targets.each do |target|
       if target.include?('@') then
   def run(plugin, *targets)
     targets.each do |target|
       if target.include?('@') then
-        user_description = plugin.describe_account(target)
+        user_description = plugin.describe_user(target)
         report(plugin, "Would remove user: #{user} (#{user_description})")
       else
         domain_description = plugin.describe_domain(target)
         report(plugin, "Would remove user: #{user} (#{user_description})")
       else
         domain_description = plugin.describe_domain(target)
index 49c340f7988f1fc75d1243de6d437795be0ac365..fc0e7a081d97f602124dc9ccaef2ba53b60aa1d6 100644 (file)
@@ -1,6 +1,6 @@
 module RmPlugin
   #
 module RmPlugin
   #
-  # Plugins for the removal of accounts.
+  # Plugins for the removal of users.
   #
 
   def RmPlugin.included(c)
   #
 
   def RmPlugin.included(c)
@@ -24,12 +24,12 @@ module RmPlugin
     raise NonexistentDomainError.new(domain) if usernames.empty?
 
     usernames.each do |u|
     raise NonexistentDomainError.new(domain) if usernames.empty?
 
     usernames.each do |u|
-      delete_account(u)
+      delete_user(u)
     end
   end
 
     end
   end
 
-  def delete_account(account)
-    # Delete the given account.
+  def delete_user(user)
+    # Delete the given user.
     raise NotImplementedError
   end
 
     raise NotImplementedError
   end
 
index 114aabc355ad1c215f0b95e3cf1f20a2a626ea4d..b953898061691cf70f049dd5217579980351c31c 100644 (file)
@@ -5,17 +5,17 @@ class RmRunner
 
   def run(plugin, *targets)
     targets.each do |target|
 
   def run(plugin, *targets)
     targets.each do |target|
-      # Why think too hard? An account has an @, a domain doesn't.
+      # Why think too hard? An user has an @, a domain doesn't.
       if target.include?('@') then
         begin
       if target.include?('@') then
         begin
-          account_description = plugin.describe_account(target)
-          plugin.delete_account(target)
-          report(plugin, "Removed account: #{target} (#{account_description})")
+          user_description = plugin.describe_user(target)
+          plugin.delete_user(target)
+          report(plugin, "Removed user: #{target} (#{user_description})")
 
 
-        rescue NonexistentAccountError => e
-          report(plugin, "Account not found: #{e.to_s}")
+        rescue NonexistentUserError => e
+          report(plugin, "User not found: #{e.to_s}")
         rescue StandardError => e
         rescue StandardError => e
-          report(plugin, "There was an error removing the account: #{e.to_s}")
+          report(plugin, "There was an error removing the user: #{e.to_s}")
           Kernel.exit(ExitCodes::DATABASE_ERROR)
         end
       else
           Kernel.exit(ExitCodes::DATABASE_ERROR)
         end
       else
@@ -24,9 +24,9 @@ class RmRunner
           plugin.delete_domain(target)
           report(plugin, "Removed domain: #{target} (#{domain_description})")
 
           plugin.delete_domain(target)
           report(plugin, "Removed domain: #{target} (#{domain_description})")
 
-        rescue NonexistentAccountError => e
+        rescue NonexistentUserError => e
           # Can happen in the usernames.each... block.
           # Can happen in the usernames.each... block.
-          report(plugin, "Account not found: #{e.to_s}")
+          report(plugin, "User not found: #{e.to_s}")
         rescue NonexistentDomainError => e
           report(plugin, "Domain not found: #{e.to_s}")
         rescue StandardError => e
         rescue NonexistentDomainError => e
           report(plugin, "Domain not found: #{e.to_s}")
         rescue StandardError => e