From 07627084e4307e493f81c458350e8a563ac3a8fc Mon Sep 17 00:00:00 2001
From: Joas Schilling <nickvergessen@owncloud.com>
Date: Thu, 23 Apr 2015 12:32:46 +0200
Subject: [PATCH] Check if the user exists before trying to delete him

---
 core/command/user/delete.php | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/core/command/user/delete.php b/core/command/user/delete.php
index 53952ceb9e..2db80dae08 100644
--- a/core/command/user/delete.php
+++ b/core/command/user/delete.php
@@ -22,19 +22,20 @@
 
 namespace OC\Core\Command\User;
 
+use OCP\IUserManager;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Input\InputArgument;
 
 class Delete extends Command {
-	/** @var \OC\User\Manager */
+	/** @var IUserManager */
 	protected $userManager;
 
 	/**
-	 * @param \OC\User\Manager $userManager
+	 * @param IUserManager $userManager
 	 */
-	public function __construct(\OC\User\Manager $userManager) {
+	public function __construct(IUserManager $userManager) {
 		$this->userManager = $userManager;
 		parent::__construct();
 	}
@@ -51,11 +52,17 @@ class Delete extends Command {
 	}
 
 	protected function execute(InputInterface $input, OutputInterface $output) {
-		$wasSuccessful = $this->userManager->get($input->getArgument('uid'))->delete();
-		if($wasSuccessful === true) {
+		$user = $this->userManager->get($input->getArgument('uid'));
+		if (is_null($user)) {
+			$output->writeln('User does not exist');
+			return;
+		}
+
+		if ($user->delete()) {
 			$output->writeln('The specified user was deleted');
 			return;
 		}
+
 		$output->writeln('<error>The specified could not be deleted. Please check the logs.</error>');
 	}
 }