Code Duplication    Length = 6-19 lines in 3 locations

apps/user_ldap/lib/Access.php 1 location

@@ 1574-1592 (lines=19) @@
1571
			throw new \InvalidArgumentException('String expected');
1572
		}
1573
		$blocks = explode('-', $guid);
1574
		if(count($blocks) !== 5) {
1575
			/*
1576
			 * Why not throw an Exception instead? This method is a utility
1577
			 * called only when trying to figure out whether a "missing" known
1578
			 * LDAP user was or was not renamed on the LDAP server. And this
1579
			 * even on the use case that a reverse lookup is needed (UUID known,
1580
			 * not DN), i.e. when finding users (search dialog, users page,
1581
			 * login, …) this will not be fired. This occurs only if shares from
1582
			 * a users are supposed to be mounted who cannot be found. Throwing
1583
			 * an exception here would kill the experience for a valid, acting
1584
			 * user. Instead we write a log message.
1585
			 */
1586
			\OC::$server->getLogger()->info(
1587
				'Passed string does not resemble a valid GUID. Known UUID ' .
1588
				'({uuid}) probably does not match UUID configuration.',
1589
				[ 'app' => 'user_ldap', 'uuid' => $guid ]
1590
			);
1591
			return $guid;
1592
		}
1593
		for($i=0; $i < 3; $i++) {
1594
			$pairs = str_split($blocks[$i], 2);
1595
			$pairs = array_reverse($pairs);

apps/user_ldap/lib/Mapping/AbstractMapping.php 1 location

@@ 231-240 (lines=10) @@
228
	 * @return bool
229
	 */
230
	public function map($fdn, $name, $uuid) {
231
		if(mb_strlen($fdn) > 255) {
232
			\OC::$server->getLogger()->error(
233
				'Cannot map, because the DN exceeds 255 characters: {dn}',
234
				[
235
					'app' => 'user_ldap',
236
					'dn' => $fdn,
237
				]
238
			);
239
			return false;
240
		}
241
242
		$row = array(
243
			'ldap_dn'        => $fdn,

apps/user_ldap/lib/User_LDAP.php 1 location

@@ 351-356 (lines=6) @@
348
	*/
349
	public function deleteUser($uid) {
350
		$marked = $this->ocConfig->getUserValue($uid, 'user_ldap', 'isDeleted', 0);
351
		if(intval($marked) === 0) {
352
			\OC::$server->getLogger()->notice(
353
				'User '.$uid . ' is not marked as deleted, not cleaning up.',
354
				array('app' => 'user_ldap'));
355
			return false;
356
		}
357
		\OC::$server->getLogger()->info('Cleaning up after user ' . $uid,
358
			array('app' => 'user_ldap'));
359