Completed
Branch newinternal (830cd7)
by Simon
03:46
created
includes/Pages/Request/PageConfirmEmail.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -56,14 +56,14 @@
 block discarded – undo
56 56
 		try {
57 57
 			$request->save();
58 58
 		}
59
-		catch(OptimisticLockFailedException $ex){
59
+		catch (OptimisticLockFailedException $ex) {
60 60
 			// Okay. Someone's edited this in the time between us loading this page and doing the checks, and us getting
61 61
 			// to saving the page. We *do not* want to show an optimistic lock failure, the most likely problem is they
62 62
 			// double-loaded this page (see #255). Let's confirm this, and bomb out with a success message if it's the
63 63
 			// case.
64 64
 
65 65
 			$request = Request::getById($id, $this->getDatabase());
66
-			if($request->getEmailConfirm() === 'Confirmed') {
66
+			if ($request->getEmailConfirm() === 'Confirmed') {
67 67
 				// we've already done the sanity checks above
68 68
 
69 69
 				$this->redirect('requestSubmitted');
Please login to merge, or discard this patch.
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -56,7 +56,7 @@
 block discarded – undo
56 56
 		try {
57 57
 			$request->save();
58 58
 		}
59
-		catch(OptimisticLockFailedException $ex){
59
+		catch(OptimisticLockFailedException $ex) {
60 60
 			// Okay. Someone's edited this in the time between us loading this page and doing the checks, and us getting
61 61
 			// to saving the page. We *do not* want to show an optimistic lock failure, the most likely problem is they
62 62
 			// double-loaded this page (see #255). Let's confirm this, and bomb out with a success message if it's the
Please login to merge, or discard this patch.