Completed
Push — master ( 040c1d...fcafe6 )
by Paul
03:34
created
modules/u2f.php 1 patch
Braces   +7 added lines, -9 removed lines patch added patch discarded remove patch
@@ -215,10 +215,12 @@  discard block
 block discarded – undo
215 215
 
216 216
 			if (property_exists($response, 'errorCode'))
217 217
 			{
218
-				if ($response->errorCode == 4) // errorCode 4 means that this device wasn't registered
218
+				if ($response->errorCode == 4) {
219
+					// errorCode 4 means that this device wasn't registered
219 220
 				{
220 221
 					throw new AccessDeniedHttpException($this->user->lang('TFA_NOT_REGISTERED'));
221 222
 				}
223
+				}
222 224
 				throw new BadRequestHttpException($this->user->lang('TFA_SOMETHING_WENT_WRONG'));
223 225
 			}
224 226
 			$result = new AuthenticationResponse($response->signatureData, $response->clientData, $response->keyHandle, $response->errorCode);
@@ -232,12 +234,10 @@  discard block
 block discarded – undo
232 234
 
233 235
 			$sql = 'UPDATE ' . $this->registration_table . ' SET ' . $this->db->sql_build_array('UPDATE', $sql_ary) . ' WHERE registration_id = ' . (int)$reg->id;
234 236
 			$this->db->sql_query($sql);
235
-		}
236
-		catch (U2fError $error)
237
+		} catch (U2fError $error)
237 238
 		{
238 239
 			$this->createError($error);
239
-		}
240
-		catch (\InvalidArgumentException $invalid)
240
+		} catch (\InvalidArgumentException $invalid)
241 241
 		{
242 242
 			throw new BadRequestHttpException($this->user->lang('TFA_SOMETHING_WENT_WRONG') . '<br />' . $invalid->getMessage(), $invalid);
243 243
 		}
@@ -274,8 +274,7 @@  discard block
 block discarded – undo
274 274
 		if ($count == 0)
275 275
 		{
276 276
 			trigger_error('TFA_UNABLE_TO_UPDATE_SESSION');
277
-		}
278
-		else if ($count > 1)
277
+		} else if ($count > 1)
279 278
 		{
280 279
 			// Reset sessions table. We had multiple sessions with same ID!!!
281 280
 			$sql_ary['u2f_request'] = '';
@@ -319,8 +318,7 @@  discard block
 block discarded – undo
319 318
 			);
320 319
 
321 320
 			$this->update_session($sql_ary);
322
-		}
323
-		catch (U2fError $err)
321
+		} catch (U2fError $err)
324 322
 		{
325 323
 			$this->createError($err);
326 324
 		}
Please login to merge, or discard this patch.
ucp/tfa_module.php 1 patch
Braces   +3 added lines, -6 removed lines patch added patch discarded remove patch
@@ -101,13 +101,11 @@  discard block
 block discarded – undo
101 101
 				meta_refresh(3, $this->u_action);
102 102
 				$message = $this->user->lang['TFA_KEY_ADDED'] . '<br /><br />' . sprintf($this->user->lang['RETURN_UCP'], '<a href="' . $this->u_action . '">', '</a>');
103 103
 				trigger_error($message);
104
-			}
105
-			else
104
+			} else
106 105
 			{
107 106
 				$error[] = $this->user->lang('TFA_MODULE_NOT_FOUND', $class);
108 107
 			}
109
-		}
110
-		catch (\Exception $e)
108
+		} catch (\Exception $e)
111 109
 		{
112 110
 			$error[] = $e->getMessage();
113 111
 		}
@@ -130,8 +128,7 @@  discard block
 block discarded – undo
130 128
 			if (!check_form_key('ucp_tfa_keys'))
131 129
 			{
132 130
 				$error[] = 'FORM_INVALID';
133
-			}
134
-			else
131
+			} else
135 132
 			{
136 133
 				switch ($mode)
137 134
 				{
Please login to merge, or discard this patch.