phpBB/config/default/container/services_console.yml

@@ -281,17 +281,13 @@ services:
         tags:
             - { name: console.command }
 
-    console.command.user.delete_id:
-        class: phpbb\console\command\user\delete_id
+    console.command.user.delete_ids:
+        class: phpbb\console\command\user\delete_ids
         arguments:
-            - '@dbal.conn'
             - '@language'
             - '@log'
             - '@user'
             - '@user_loader'
-            - '%tables.bots%'
-            - '%tables.user_group%'
-            - '%tables.users%'
             - '%core.root_path%'
             - '%core.php_ext%'
         tags:

phpBB/language/en/cli.php

@@ -108,8 +108,6 @@ $lang = array_merge($lang, array(
 	'CLI_DESCRIPTION_USER_ADD_OPTION_NOTIFY'	=> 'Send account activation email to the new user (not sent by default)',
 	'CLI_DESCRIPTION_USER_DELETE'				=> 'Delete a user account.',
 	'CLI_DESCRIPTION_USER_DELETE_USERNAME'		=> 'Username of the user to delete',
-	'CLI_DESCRIPTION_USER_DELETE_ID'			=> 'Delete user accounts by ID.',
-	'CLI_DESCRIPTION_USER_DELETE_ID_OPTION_ID'	=> 'User IDs of the users to delete',
 	'CLI_DESCRIPTION_USER_DELETE_OPTION_POSTS'	=> 'Delete all posts by the user. Without this option, the user’s posts will be retained.',
 	'CLI_DESCRIPTION_USER_RECLEAN'				=> 'Re-clean usernames.',
 
@@ -159,9 +157,8 @@ $lang = array_merge($lang, array(
 
 	'CLI_USER_ADD_SUCCESS'			=> 'Successfully added user %s.',
 	'CLI_USER_DELETE_CONFIRM'		=> 'Are you sure you want to delete ‘%s’? [y/N]',
-	'CLI_USER_DELETE_ID_CONFIRM'	=> 'Are you sure you want to delete the user IDs ‘%s’? [y/N]',
-	'CLI_USER_DELETE_ID_SUCCESS'	=> 'Successfully deleted user IDs.',
-	'CLI_USER_DELETE_ID_START'		=> 'Deleting users by ID',
+	'CLI_USER_DELETE_IDS_CONFIRM'	=> 'Are you sure you want to delete the user IDs ‘%s’? [y/N]',
+	'CLI_USER_DELETE_IDS_SUCCESS'	=> 'Successfully deleted user IDs.',
 	'CLI_USER_DELETE_NONE'			=> 'No users were deleted by user ID.',
 	'CLI_USER_RECLEAN_START'		=> 'Re-cleaning usernames',
 	'CLI_USER_RECLEAN_DONE'			=> [

phpBB/phpbb/console/command/user/delete_ids.php

@@ -14,7 +14,6 @@
 namespace phpbb\console\command\user;
 
 use phpbb\console\command\command;
-use phpbb\db\driver\driver_interface;
 use phpbb\language\language;
 use phpbb\log\log_interface;
 use phpbb\user;
@@ -26,11 +25,8 @@ use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Question\ConfirmationQuestion;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
-class delete_id extends command
+class delete_ids extends command
 {
-	/** @var driver_interface */
-	protected $db;
-
 	/** @var language */
 	protected $language;
 
@@ -40,15 +36,6 @@ class delete_id extends command
 	/** @var user_loader */
 	protected $user_loader;
 
-	/** @var string Bots table */
-	protected $bots_table;
-
-	/** @var string User group table */
-	protected $user_group_table;
-
-	/** @var string Users table */
-	protected $users_table;
-
 	/** @var string phpBB root path */
 	protected $phpbb_root_path;
 
@@ -58,27 +45,18 @@ class delete_id extends command
 	/**
 	 * Construct method
 	 *
-	 * @param driver_interface $db
 	 * @param language         $language
 	 * @param log_interface    $log
 	 * @param user             $user
 	 * @param user_loader      $user_loader
-	 * @param string           $bots_table
-	 * @param string           $user_group_table
-	 * @param string           $users_table
 	 * @param string           $phpbb_root_path
 	 * @param string           $php_ext
 	 */
-	public function __construct(driver_interface $db, language $language, log_interface $log, user $user, user_loader $user_loader,
-								string $bots_table, string $user_group_table, string $users_table, string $phpbb_root_path, string $php_ext)
+	public function __construct(language $language, log_interface $log, user $user, user_loader $user_loader, string $phpbb_root_path, string $php_ext)
 	{
-		$this->db = $db;
 		$this->language = $language;
 		$this->log = $log;
 		$this->user_loader = $user_loader;
-		$this->bots_table = $bots_table;
-		$this->user_group_table = $user_group_table;
-		$this->users_table = $users_table;
 		$this->phpbb_root_path = $phpbb_root_path;
 		$this->php_ext = $php_ext;
 
@@ -94,12 +72,12 @@ class delete_id extends command
 	protected function configure(): void
 	{
 		$this
-			->setName('user:delete_id')
-			->setDescription($this->language->lang('CLI_DESCRIPTION_USER_DELETE_ID'))
+			->setName('user:delete_ids')
+			->setDescription($this->language->lang('CLI_DESCRIPTION_USER_DELETE_IDS'))
 			->addArgument(
 				'user_ids',
 				InputArgument::REQUIRED | InputArgument::IS_ARRAY,
-				$this->language->lang('CLI_DESCRIPTION_USER_DELETE_ID_OPTION_ID')
+				$this->language->lang('CLI_DESCRIPTION_USER_DELETE_IDS_LIST')
 			)
 			->addOption(
 				'delete-posts',
@@ -132,10 +110,6 @@ class delete_id extends command
 		{
 			$this->user_loader->load_users($user_ids);
 
-			$progress = $this->create_progress_bar(count($user_ids), $io, $output);
-			$progress->setMessage($this->language->lang('CLI_USER_DELETE_ID_START'));
-			$progress->start();
-
 			foreach ($user_ids as $user_id)
 			{
 				$user_row = $this->user_loader->get_user($user_id);
@@ -143,34 +117,23 @@ class delete_id extends command
 				// Skip anonymous user
 				if ($user_row['user_id'] == ANONYMOUS)
 				{
-					$progress->advance();
 					continue;
 				}
-				else if ($user_row['user_type'] == USER_IGNORE)
-				{
-					$this->delete_bot_user($user_row);
-				}
-				else
-				{
-					if (!function_exists('user_delete'))
-					{
-						require($this->phpbb_root_path . 'includes/functions_user.' . $this->php_ext);
-					}
 
-					user_delete($mode, $user_row['user_id'], $user_row['username']);
-
-					$this->log->add('admin', ANONYMOUS, '', 'LOG_USER_DELETED', false, array($user_row['username']));
+				if (!function_exists('user_delete'))
+				{
+					require($this->phpbb_root_path . 'includes/functions_user.' . $this->php_ext);
 				}
 
-				$progress->advance();
+				user_delete($mode, $user_row['user_id'], $user_row['username']);
+
+				$this->log->add('admin', ANONYMOUS, '', 'LOG_USER_DELETED', false, array($user_row['username']));
 				$deleted_users++;
 			}
 
-			$progress->finish();
-
 			if ($deleted_users > 0)
 			{
-				$io->success($this->language->lang('CLI_USER_DELETE_ID_SUCCESS'));
+				$io->success($this->language->lang('CLI_USER_DELETE_IDS_SUCCESS'));
 			}
 		}
 
@@ -197,7 +160,7 @@ class delete_id extends command
 		if (count($user_ids) > 0)
 		{
 			$question = new ConfirmationQuestion(
-				$this->language->lang('CLI_USER_DELETE_ID_CONFIRM', implode(',', $user_ids)),
+				$this->language->lang('CLI_USER_DELETE_IDS_CONFIRM', implode(',', $user_ids)),
 				false
 			);
 
@@ -207,25 +170,4 @@ class delete_id extends command
 			}
 		}
 	}
-
-	/**
-	 * Deletes a bot user
-	 *
-	 * @param array $user_row
-	 * @return void
-	 */
-	protected function delete_bot_user(array $user_row): void
-	{
-		$sql = 'DELETE FROM ' . $this->bots_table . '
-					WHERE user_id = ' . (int) $user_row['user_id'];
-		$this->db->sql_query($sql);
-
-		$delete_tables = [$this->user_group_table, $this->users_table];
-		foreach ($delete_tables as $table)
-		{
-			$sql = "DELETE FROM $table
-				WHERE user_id = " . (int) $user_row['user_id'];
-			$this->db->sql_query($sql);
-		}
-	}
 }

tests/console/user/delete_ids_test.php

@@ -13,7 +13,7 @@
 
 use Symfony\Component\Console\Application;
 use Symfony\Component\Console\Tester\CommandTester;
-use phpbb\console\command\user\delete_id;
+use phpbb\console\command\user\delete_ids;
 
 require_once __DIR__ . '/base.php';
 
@@ -22,20 +22,16 @@ class phpbb_console_user_delete_ids_test extends phpbb_console_user_base
 	public function get_command_tester()
 	{
 		$application = new Application();
-		$application->add(new delete_id(
-			$this->db,
+		$application->add(new delete_ids(
 			$this->language,
 			$this->log,
 			$this->user,
 			$this->user_loader,
-			BOTS_TABLE,
-			USER_GROUP_TABLE,
-			USERS_TABLE,
 			$this->phpbb_root_path,
 			$this->php_ext
 		));
 
-		$command = $application->find('user:delete_id');
+		$command = $application->find('user:delete_ids');
 		$this->command_name = $command->getName();
 		$this->question = $command->getHelper('question');
 
@@ -56,42 +52,7 @@ class phpbb_console_user_delete_ids_test extends phpbb_console_user_base
 
 		$this->assertNull($this->get_user_id('Test'));
 		$this->assertNull($this->get_user_id('Test 2'));
-		$this->assertStringContainsString('CLI_USER_DELETE_ID_SUCCESS', $command_tester->getDisplay());
-	}
-
-	public function test_delete_one()
-	{
-		$command_tester = $this->get_command_tester();
-
-		$command_tester->setInputs(['yes', '']);
-
-		$command_tester->execute(array(
-			'command'			=> $this->command_name,
-			'user_ids'			=> [3],
-			'--delete-posts'	=> false,
-		));
-
-		$this->assertNull($this->get_user_id('Test'));
-		$this->assertNotNull($this->get_user_id('Test 2'));
-		$this->assertStringContainsString('CLI_USER_DELETE_ID_SUCCESS', $command_tester->getDisplay());
-	}
-
-	public function test_delete_bot()
-	{
-		$command_tester = $this->get_command_tester();
-
-		$this->assertNotNull($this->get_user_id('Test Bot'));
-
-		$command_tester->setInputs(['yes', '']);
-
-		$command_tester->execute(array(
-			'command'			=> $this->command_name,
-			'user_ids'			=> [6],
-			'--delete-posts'	=> false,
-		));
-
-		$this->assertNull($this->get_user_id('Test Bot'));
-		$this->assertStringContainsString('CLI_USER_DELETE_ID_SUCCESS', $command_tester->getDisplay());
+		$this->assertStringContainsString('CLI_USER_DELETE_IDS_SUCCESS', $command_tester->getDisplay());
 	}
 
 	public function test_delete_non_user()

tests/console/user/fixtures/config.xml

@@ -47,14 +47,6 @@
 			<value></value>
 			<value>0</value>
 		</row>
-		<row>
-			<value>6</value>
-			<value></value>
-			<value>Test Bot</value>
-			<value>Test Bot</value>
-			<value></value>
-			<value>2</value>
-		</row>
 	</table>
 	<table name="phpbb_groups">
 		<column>group_id</column>
@@ -67,23 +59,5 @@
 			<value>3</value>
 			<value>foobar</value>
 		</row>
-		<row>
-			<value>6</value>
-			<value>BOTS</value>
-			<value>3</value>
-			<value></value>
-		</row>
-	</table>
-	<table name="phpbb_user_group">
-		<column>group_id</column>
-		<column>user_id</column>
-		<column>group_leader</column>
-		<column>user_pending</column>
-		<row>
-			<value>6</value>
-			<value>6</value>
-			<value>0</value>
-			<value>0</value>
-		</row>
 	</table>
 </dataset>