owncloud / core

			throw new \InvalidArgumentException('String expected');

		}

		$blocks = explode('-', $guid);

		if(count($blocks) !== 5) {

			/*

			 * Why not throw an Exception instead? This method is a utility

			 * called only when trying to figure out whether a "missing" known

			 * LDAP user was or was not renamed on the LDAP server. And this

			 * even on the use case that a reverse lookup is needed (UUID known,

			 * not DN), i.e. when finding users (search dialog, users page,

			 * login, …) this will not be fired. This occurs only if shares from

			 * a users are supposed to be mounted who cannot be found. Throwing

			 * an exception here would kill the experience for a valid, acting

			 * user. Instead we write a log message.

			 */

			\OC::$server->getLogger()->info(

				'Passed string does not resemble a valid GUID. Known UUID ' .

				'({uuid}) probably does not match UUID configuration.',

				[ 'app' => 'user_ldap', 'uuid' => $guid ]

			);

			return $guid;

		}

		for($i=0; $i < 3; $i++) {

			$pairs = str_split($blocks[$i], 2);

			$pairs = array_reverse($pairs);

	*/

	public function deleteUser($uid) {

		$marked = $this->ocConfig->getUserValue($uid, 'user_ldap', 'isDeleted', 0);

		if(intval($marked) === 0) {

			\OC::$server->getLogger()->notice(

				'User '.$uid . ' is not marked as deleted, not cleaning up.',

				array('app' => 'user_ldap'));

			return false;

		}

		\OC::$server->getLogger()->info('Cleaning up after user ' . $uid,

			array('app' => 'user_ldap'));