Code Duplication    Length = 6-19 lines in 3 locations

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/Access.php 1 location

@@ 1629-1647 (lines=19) @@
1626
			throw new \InvalidArgumentException('String expected');
1627
		}
1628
		$blocks = explode('-', $guid);
1629
		if(count($blocks) !== 5) {
1630
			/*
1631
			 * Why not throw an Exception instead? This method is a utility
1632
			 * called only when trying to figure out whether a "missing" known
1633
			 * LDAP user was or was not renamed on the LDAP server. And this
1634
			 * even on the use case that a reverse lookup is needed (UUID known,
1635
			 * not DN), i.e. when finding users (search dialog, users page,
1636
			 * login, …) this will not be fired. This occurs only if shares from
1637
			 * a users are supposed to be mounted who cannot be found. Throwing
1638
			 * an exception here would kill the experience for a valid, acting
1639
			 * user. Instead we write a log message.
1640
			 */
1641
			\OC::$server->getLogger()->info(
1642
				'Passed string does not resemble a valid GUID. Known UUID ' .
1643
				'({uuid}) probably does not match UUID configuration.',
1644
				[ 'app' => 'user_ldap', 'uuid' => $guid ]
1645
			);
1646
			return $guid;
1647
		}
1648
		for($i=0; $i < 3; $i++) {
1649
			$pairs = str_split($blocks[$i], 2);
1650
			$pairs = array_reverse($pairs);

apps/user_ldap/lib/User_LDAP.php 1 location

@@ 368-373 (lines=6) @@
365
	*/
366
	public function deleteUser($uid) {
367
		$marked = $this->ocConfig->getUserValue($uid, 'user_ldap', 'isDeleted', 0);
368
		if(intval($marked) === 0) {
369
			\OC::$server->getLogger()->notice(
370
				'User '.$uid . ' is not marked as deleted, not cleaning up.',
371
				array('app' => 'user_ldap'));
372
			return false;
373
		}
374
		\OC::$server->getLogger()->info('Cleaning up after user ' . $uid,
375
			array('app' => 'user_ldap'));
376