mdaniels5757 / waca

includes/Pages/PageListFlaggedComments.php

Braces +5 added lines, -10 removed lines

@@ -61,28 +61,23 @@
 
                 if ($request->getReserved() === $currentUser->getId()) {
                     $data['hiddenText'] = false;
-                }
-                else {
+                } else {
                     $data['hiddenText'] = true;
                 }
             }
 
             if ($object->getVisibility() == 'requester' || $object->getVisibility() == 'user') {
                 $data['hidden'] = false;
-            }
-            elseif ($object->getVisibility() == 'admin') {
+            } elseif ($object->getVisibility() == 'admin') {
                 if ($seeRestrictedComments) {
                     $data['hidden'] = false;
-                }
-                else {
+                } else {
                     $data['hidden'] = true;
                 }
-            }
-            elseif ($object->getVisibility() == 'checkuser') {
+            } elseif ($object->getVisibility() == 'checkuser') {
                 if ($seeCheckuserComments) {
                     $data['hidden'] = false;
-                }
-                else {
+                } else {
                     $data['hidden'] = true;
                 }
             }

includes/Pages/PageRequestFormManagement.php

Braces +6 added lines, -9 removed lines

@@ -56,7 +56,8 @@ 
         $this->setTemplate('form-management/main.tpl');
     }
 
-    protected function preview() {
+    protected function preview()
+    {
         $previewContent = WebRequest::getSessionContext('preview');
 
         $renderer = new MarkdownRenderingHelper();
@@ -129,8 +130,7 @@ 
                 $form->save();
                 Logger::requestFormCreated($database, $form);
                 $this->redirect('requestFormManagement');
-            }
-            else {
+            } else {
                 $this->populateFromObject($form);
                 WebRequest::setSessionContext('preview', [
                     'main' => $form->getFormContent(),
@@ -142,8 +142,7 @@ 
                 $this->assign('createMode', true);
                 $this->setTemplate('form-management/edit.tpl');
             }
-        }
-        else {
+        } else {
             $this->populateFromObject(new RequestForm());
             WebRequest::setSessionContext('preview', null);
             $this->assign('hidePreview', true);
@@ -237,8 +236,7 @@ 
                 Logger::requestFormEdited($database, $form);
                 $form->save();
                 $this->redirect('requestFormManagement');
-            }
-            else {
+            } else {
                 $this->populateFromObject($form);
                 WebRequest::setSessionContext('preview', [
                     'main' => $form->getFormContent(),
@@ -250,8 +248,7 @@ 
                 $this->assign('createMode', false);
                 $this->setTemplate('form-management/edit.tpl');
             }
-        }
-        else {
+        } else {
             $this->populateFromObject($form);
             WebRequest::setSessionContext('preview', [
                 'main' => $form->getFormContent(),

includes/Pages/UserAuth/MultiFactor/PageMultiFactor.php

Braces +10 added lines, -20 removed lines

@@ -91,13 +91,11 @@ 
                 }
 
                 $this->redirect('multiFactor');
-            }
-            else {
+            } else {
                 SessionAlert::error('Error enabling YubiKey OTP - invalid credentials.');
                 $this->redirect('multiFactor');
             }
-        }
-        else {
+        } else {
             if ($otpCredentialProvider->userIsEnrolled($currentUser->getId())) {
                 // user is not enrolled, we shouldn't have got here.
                 throw new ApplicationLogicException('User is already enrolled in the selected MFA mechanism');
@@ -161,8 +159,7 @@ 
                     $this->setTemplate('mfa/enableTotpEnroll.tpl');
 
                     return;
-                }
-                else {
+                } else {
                     SessionAlert::error('Error enabling TOTP - invalid credentials.');
                     $this->redirect('multiFactor');
 
@@ -187,8 +184,7 @@ 
                             $this->setTemplate('mfa/regenScratchTokens.tpl');
                             return;
                         }
-                    }
-                    else {
+                    } else {
                         $otpCredentialProvider->deleteCredential($currentUser);
                         SessionAlert::error('Error enabling TOTP: invalid token provided');
                     }
@@ -196,8 +192,7 @@ 
 
                     $this->redirect('multiFactor');
                     return;
-                }
-                else {
+                } else {
                     SessionAlert::error('Error enabling TOTP - no enrollment found or enrollment expired.');
                     $this->redirect('multiFactor');
 
@@ -207,8 +202,7 @@ 
 
             // urgh, dunno what happened, but it's not something expected.
             throw new ApplicationLogicException();
-        }
-        else {
+        } else {
             if ($otpCredentialProvider->userIsEnrolled($currentUser->getId())) {
                 // user is not enrolled, we shouldn't have got here.
                 throw new ApplicationLogicException('User is already enrolled in the selected MFA mechanism');
@@ -258,13 +252,11 @@ 
                 $tokens = $otpCredentialProvider->getTokens();
                 $this->assign('tokens', $tokens);
                 $this->setTemplate('mfa/regenScratchTokens.tpl');
-            }
-            else {
+            } else {
                 SessionAlert::error('Error refreshing scratch tokens - invalid credentials.');
                 $this->redirect('multiFactor');
             }
-        }
-        else {
+        } else {
             $this->assignCSRFToken();
 
             $this->assign('alertmessage', 'To regenerate your emergency scratch tokens, please prove you are who you say you are by providing your tool password below. Note that continuing will invalidate all remaining scratch tokens, and provide a set of new ones.');
@@ -302,13 +294,11 @@ 
                 $otpCredentialProvider->deleteCredential($currentUser);
                 SessionAlert::success('Disabled ' . $factorType . '.');
                 $this->redirect('multiFactor');
-            }
-            else {
+            } else {
                 SessionAlert::error('Error disabling ' . $factorType . ' - invalid credentials.');
                 $this->redirect('multiFactor');
             }
-        }
-        else {
+        } else {
             if (!$otpCredentialProvider->userIsEnrolled($currentUser->getId())) {
                 // user is not enrolled, we shouldn't have got here.
                 throw new ApplicationLogicException('User is not enrolled in the selected MFA mechanism');

includes/Pages/RequestAction/PageBreakReservation.php

Braces +3 added lines, -6 removed lines

@@ -33,13 +33,11 @@ 
 
         if ($currentUser->getId() === $request->getReserved()) {
             $this->doUnreserve($request, $database);
-        }
-        else {
+        } else {
             // not the same user!
             if ($this->barrierTest('force', $currentUser)) {
                 $this->doBreakReserve($request, $database);
-            }
-            else {
+            } else {
                 throw new AccessDeniedException($this->getSecurityManager(), $this->getDomainAccessManager());
             }
         }
@@ -83,8 +81,7 @@ 
             $this->skipAlerts();
 
             $this->setTemplate("confirmations/breakreserve.tpl");
-        }
-        else {
+        } else {
             $request->setReserved(null);
             $request->setUpdateVersion(WebRequest::postInt('updateversion'));
             $request->save();

includes/Pages/RequestAction/PageCustomClose.php

Braces +2 added lines, -4 removed lines

@@ -54,8 +54,7 @@ 
             if ($success) {
                 $this->redirect();
             }
-        }
-        else {
+        } else {
             $this->assignCSRFToken();
             $this->showCustomCloseForm($database, $request);
         }
@@ -311,8 +310,7 @@ 
         if ($action == EmailTemplate::ACTION_CREATED) {
             $logCloseType = 'custom-y';
             $notificationCloseType = "Custom, Created";
-        }
-        else {
+        } else {
             $logCloseType = 'custom-n';
             $notificationCloseType = "Custom, Not Created";
         }

includes/Pages/PageDomainManagement.php

Braces +3 added lines, -6 removed lines

@@ -83,8 +83,7 @@ 
 
             Logger::domainCreated($database, $domain);
             $this->redirect('domainManagement');
-        }
-        else {
+        } else {
             $this->assignCSRFToken();
 
             $this->assign('shortName', '');
@@ -129,8 +128,7 @@ 
                 && $template->getPreloadOnly() === false
                 && $template->getDefaultAction() === EmailTemplate::ACTION_CREATED) {
                 $domain->setDefaultClose(WebRequest::postInt('defaultClose'));
-            }
-            else {
+            } else {
                 SessionAlert::warning("Chosen email template is not valid for use as the default creation template");
             }
 
@@ -146,8 +144,7 @@ 
 
             Logger::domainEdited($database, $domain);
             $this->redirect('domainManagement');
-        }
-        else {
+        } else {
             $this->assignCSRFToken();
 
             $templates = EmailTemplate::getActiveNonpreloadTemplates(

includes/Pages/PageSearch.php

Braces +1 added lines, -2 removed lines

@@ -111,8 +111,7 @@
             list($defaultSort, $defaultSortDirection) = WebRequest::requestListDefaultSort();
             $this->assign('defaultSort', $defaultSort);
             $this->assign('defaultSortDirection', $defaultSortDirection);
-        }
-        else {
+        } else {
             $this->assign('type', 'name');
             $this->assign('hasResultset', false);
             $this->assign('limit', 50);

includes/Pages/PageEmailManagement.php

Braces +3 added lines, -6 removed lines

@@ -117,8 +117,7 @@ 
             SessionAlert::success("Email template has been saved successfully.");
 
             $this->redirect('emailManagement');
-        }
-        else {
+        } else {
             $this->assignCSRFToken();
             $this->assign('id', $template->getId());
             $this->assign('emailTemplate', $template);
@@ -158,8 +157,7 @@ 
             $template->setDefaultAction(EmailTemplate::ACTION_CREATED);
             $template->setActive(true);
             $template->setPreloadOnly(false);
-        }
-        else {
+        } else {
             $defaultAction = WebRequest::postString('defaultaction');
             switch ($defaultAction) {
                 case EmailTemplate::ACTION_NONE:
@@ -212,8 +210,7 @@ 
             SessionAlert::success("Email template has been saved successfully.");
 
             $this->redirect('emailManagement');
-        }
-        else {
+        } else {
             $this->assignCSRFToken();
             $this->assign('id', -1);
             $this->assign('emailTemplate', new EmailTemplate());

includes/Pages/PageQueueManagement.php

Braces +4 added lines, -8 removed lines

@@ -92,15 +92,13 @@ 
                 $queue->save();
                 Logger::requestQueueCreated($database, $queue);
                 $this->redirect('queueManagement');
-            }
-            else {
+            } else {
                 $this->populateFromObject($queue);
 
                 $this->assign('createMode', true);
                 $this->setTemplate('queue-management/edit.tpl');
             }
-        }
-        else {
+        } else {
             $this->assign('header', null);
             $this->assign('displayName', null);
             $this->assign('apiName', null);
@@ -167,15 +165,13 @@ 
                 Logger::requestQueueEdited($database, $queue);
                 $queue->save();
                 $this->redirect('queueManagement');
-            }
-            else {
+            } else {
                 $this->populateFromObject($queue);
 
                 $this->assign('createMode', false);
                 $this->setTemplate('queue-management/edit.tpl');
             }
-        }
-        else {
+        } else {
             $this->populateFromObject($queue);
 
             $this->assign('createMode', false);