Passed
Push — master ( d70b01...159d75 )
by Morris
11:29
created
apps/user_ldap/lib/LDAP.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -52,12 +52,12 @@  discard block
 block discarded – undo
52 52
 	 * @return mixed
53 53
 	 */
54 54
 	public function connect($host, $port) {
55
-		if(strpos($host, '://') === false) {
56
-			$host = 'ldap://' . $host;
55
+		if (strpos($host, '://') === false) {
56
+			$host = 'ldap://'.$host;
57 57
 		}
58
-		if(strpos($host, ':', strpos($host, '://') + 1) === false) {
58
+		if (strpos($host, ':', strpos($host, '://') + 1) === false) {
59 59
 			//ldap_connect ignores port parameter when URLs are passed
60
-			$host .= ':' . $port;
60
+			$host .= ':'.$port;
61 61
 		}
62 62
 		return $this->invokeLDAPMethod('connect', $host);
63 63
 	}
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
 	 */
194 194
 	public function search($link, $baseDN, $filter, $attr, $attrsOnly = 0, $limit = 0) {
195 195
 		$oldHandler = set_error_handler(function($no, $message, $file, $line) use (&$oldHandler) {
196
-			if(strpos($message, 'Partial search results returned: Sizelimit exceeded') !== false) {
196
+			if (strpos($message, 'Partial search results returned: Sizelimit exceeded') !== false) {
197 197
 				return true;
198 198
 			}
199 199
 			$oldHandler($no, $message, $file, $line);
@@ -272,13 +272,13 @@  discard block
 block discarded – undo
272 272
 	 * @return bool
273 273
 	 */
274 274
 	protected function isResultFalse($result) {
275
-		if($result === false) {
275
+		if ($result === false) {
276 276
 			return true;
277 277
 		}
278 278
 
279
-		if($this->curFunc === 'ldap_search' && is_array($result)) {
279
+		if ($this->curFunc === 'ldap_search' && is_array($result)) {
280 280
 			foreach ($result as $singleResult) {
281
-				if($singleResult === false) {
281
+				if ($singleResult === false) {
282 282
 					return true;
283 283
 				}
284 284
 			}
@@ -292,8 +292,8 @@  discard block
 block discarded – undo
292 292
 	 */
293 293
 	protected function invokeLDAPMethod() {
294 294
 		$arguments = func_get_args();
295
-		$func = 'ldap_' . array_shift($arguments);
296
-		if(function_exists($func)) {
295
+		$func = 'ldap_'.array_shift($arguments);
296
+		if (function_exists($func)) {
297 297
 			$this->preFunctionCall($func, $arguments);
298 298
 			$result = call_user_func_array($func, $arguments);
299 299
 			if ($this->isResultFalse($result)) {
@@ -323,12 +323,12 @@  discard block
 block discarded – undo
323 323
 	 */
324 324
 	private function processLDAPError($resource) {
325 325
 		$errorCode = ldap_errno($resource);
326
-		if($errorCode === 0) {
326
+		if ($errorCode === 0) {
327 327
 			return;
328 328
 		}
329
-		$errorMsg  = ldap_error($resource);
329
+		$errorMsg = ldap_error($resource);
330 330
 
331
-		if($this->curFunc === 'ldap_get_entries'
331
+		if ($this->curFunc === 'ldap_get_entries'
332 332
 			&& $errorCode === -4) {
333 333
 		} else if ($errorCode === 32) {
334 334
 			//for now
@@ -344,7 +344,7 @@  discard block
 block discarded – undo
344 344
 			throw new \Exception('LDAP Operations error', $errorCode);
345 345
 		} else if ($errorCode === 19) {
346 346
 			ldap_get_option($this->curArgs[0], LDAP_OPT_ERROR_STRING, $extended_error);
347
-			throw new ConstraintViolationException(!empty($extended_error)?$extended_error:$errorMsg, $errorCode);
347
+			throw new ConstraintViolationException(!empty($extended_error) ? $extended_error : $errorMsg, $errorCode);
348 348
 		} else {
349 349
 			\OC::$server->getLogger()->debug('LDAP error {message} ({code}) after calling {func}', [
350 350
 				'app' => 'user_ldap',
@@ -360,9 +360,9 @@  discard block
 block discarded – undo
360 360
 	 * @throw \Exception
361 361
 	 */
362 362
 	private function postFunctionCall() {
363
-		if($this->isResource($this->curArgs[0])) {
363
+		if ($this->isResource($this->curArgs[0])) {
364 364
 			$resource = $this->curArgs[0];
365
-		} else if(
365
+		} else if (
366 366
 			   $this->curFunc === 'ldap_search'
367 367
 			&& is_array($this->curArgs[0])
368 368
 			&& $this->isResource($this->curArgs[0][0])
Please login to merge, or discard this patch.