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'])

ucp/tfa_module.php

Braces +4 added lines, -8 removed lines

@@ -108,18 +108,15 @@ 
 				if ($module->can_register())
 				{
 					$this->tpl_name = $module->register_start();
-				}
-				else
+				} else
 				{
 					$error[] = 'TFA_MODULE_NO_REGISTER';
 				}
-			}
-			else
+			} else
 			{
 				$error[] = $this->user->lang('TFA_MODULE_NOT_FOUND', $class);
 			}
-		}
-		catch (\Exception $e)
+		} catch (\Exception $e)
 		{
 			$error[] = $e->getMessage();
 		}
@@ -151,8 +148,7 @@ 
 					if (!check_form_key('ucp_tfa_keys'))
 					{
 						$error[] = 'FORM_INVALID';
-					}
-					else
+					} else
 					{
 						if ($this->request->variable('md', false, false, \phpbb\request\request_interface::POST))
 						{

modules/u2f.php

Braces +7 added lines, -9 removed lines

@@ -152,8 +152,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;
 		}
@@ -234,10 +233,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); // Do not need to include errorCode, as we already handled it.
@@ -251,12 +252,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);
 		}
@@ -351,8 +350,7 @@ 
 			);
 
 			$this->update_session($sql_ary);
-		}
-		catch (U2fError $err)
+		} catch (U2fError $err)
 		{
 			$this->createError($err);
 		}

controller/main_controller.php

Braces +1 added lines, -2 removed lines

@@ -155,8 +155,7 @@
 				$this->template->assign_var('S_ERROR', $this->user->lang('TFA_INCORRECT_KEY'));
 				$this->session_helper->generate_page($user_id, $admin, $auto_login, $viewonline, $redirect);
 			}
-		}
-		catch (BadRequestHttpException $ex) // @TODO: Replace exception with own exception
+		} catch (BadRequestHttpException $ex) // @TODO: Replace exception with own exception
 		{
 			$this->template->assign_var('S_ERROR', $ex->getMessage());
 			$this->session_helper->generate_page($user_id, $admin, $auto_login, $viewonline, $redirect);