paul999 / phpbb_2fa

event/listener.php

Braces +1 added lines, -2 removed lines

@@ -162,8 +162,7 @@
 					// We simply return and continue the login procedure (The normal way :)),
 					// and will disable all pages untill he has added a 2FA key.
 					return;
-				}
-				else
+				} else
 				{
 					$redirect = $this->request->variable('redirect', "{$this->root_path}index.{$this->php_ext}");
 					if ($event['admin'])

controller/main_controller.php

Braces +1 added lines, -2 removed lines

@@ -127,8 +127,7 @@
 		if (!empty($default))
 		{
 			$module = $this->session_helper->findModule($class);
-		}
-		else
+		} else
 		{
 			foreach ($modules as $row)
 			{

ucp/tfa_module.php

Braces +3 added lines, -6 removed lines

@@ -101,13 +101,11 @@ 
 				meta_refresh(3, $this->u_action);
 				$message = $this->user->lang['TFA_KEY_ADDED'] . '<br /><br />' . sprintf($this->user->lang['RETURN_UCP'], '<a href="' . $this->u_action . '">', '</a>');
 				trigger_error($message);
-			}
-			else
+			} else
 			{
 				$error[] = $this->user->lang('TFA_MODULE_NOT_FOUND', $class);
 			}
-		}
-		catch (\Exception $e)
+		} catch (\Exception $e)
 		{
 			$error[] = $e->getMessage();
 		}
@@ -130,8 +128,7 @@ 
 			if (!check_form_key('ucp_tfa_keys'))
 			{
 				$error[] = 'FORM_INVALID';
-			}
-			else
+			} else
 			{
 				switch ($mode)
 				{

modules/u2f.php

Braces +8 added lines, -11 removed lines

@@ -147,8 +147,7 @@ 
 		if (!empty($secure))
 		{
 			return 'on' == strtolower($secure) || '1' == $secure;
-		}
-		else if ('443' == $this->request->server('SERVER_PORT'))
+		} else if ('443' == $this->request->server('SERVER_PORT'))
 		{
 			return true;
 		}
@@ -236,10 +235,12 @@ 
 
 			if (property_exists($response, 'errorCode'))
 			{
-				if ($response->errorCode == 4) // errorCode 4 means that this device wasn't registered
+				if ($response->errorCode == 4) {
+					// errorCode 4 means that this device wasn't registered
 				{
 					throw new AccessDeniedHttpException($this->user->lang('TFA_NOT_REGISTERED'));
 				}
+				}
 				throw new BadRequestHttpException($this->user->lang('TFA_SOMETHING_WENT_WRONG'));
 			}
 			$result = new AuthenticationResponse($response->signatureData, $response->clientData, $response->keyHandle, $response->errorCode);
@@ -253,12 +254,10 @@ 
 
 			$sql = 'UPDATE ' . $this->registration_table . ' SET ' . $this->db->sql_build_array('UPDATE', $sql_ary) . ' WHERE registration_id = ' . (int) $reg->getId();
 			$this->db->sql_query($sql);
-		}
-		catch (U2fError $error)
+		} catch (U2fError $error)
 		{
 			$this->createError($error);
-		}
-		catch (\InvalidArgumentException $invalid)
+		} catch (\InvalidArgumentException $invalid)
 		{
 			throw new BadRequestHttpException($this->user->lang('TFA_SOMETHING_WENT_WRONG') . '<br />' . $invalid->getMessage(), $invalid);
 		}
@@ -295,8 +294,7 @@ 
 		if ($count == 0)
 		{
 			trigger_error('TFA_UNABLE_TO_UPDATE_SESSION');
-		}
-		else if ($count > 1)
+		} else if ($count > 1)
 		{
 			// Reset sessions table. We had multiple sessions with same ID!!!
 			$sql_ary['u2f_request'] = '';
@@ -340,8 +338,7 @@ 
 			);
 
 			$this->update_session($sql_ary);
-		}
-		catch (U2fError $err)
+		} catch (U2fError $err)
 		{
 			$this->createError($err);
 		}