Passed
Push — master ( 6fe0c7...cd5575 )
by Simon
14:57 queued 11:13
created
includes/Validation/RequestValidationHelper.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -251,7 +251,7 @@  discard block
 block discarded – undo
251 251
                 /** @var RequestQueue|false $targetQueue */
252 252
                 $targetQueue = RequestQueue::getById($ban->getTargetQueue(), $this->database);
253 253
 
254
-                if ($targetQueue === false ) {
254
+                if ($targetQueue === false) {
255 255
                     $comment = new Comment();
256 256
                     $comment->setDatabase($this->database);
257 257
                     $comment->setRequest($request->getId());
@@ -444,7 +444,7 @@  discard block
 block discarded – undo
444 444
     private function formOverride(Request $request)
445 445
     {
446 446
         $form = $request->getOriginFormObject();
447
-        if($form === null || $form->getOverrideQueue() === null) {
447
+        if ($form === null || $form->getOverrideQueue() === null) {
448 448
             return;
449 449
         }
450 450
 
Please login to merge, or discard this patch.
includes/DataObjects/UserPreference.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
      */
68 68
     public function save()
69 69
     {
70
-        if($this->isNew()) {
70
+        if ($this->isNew()) {
71 71
             // insert
72 72
             $statement = $this->dbObject->prepare(<<<SQL
73 73
                 INSERT INTO `userpreference` (
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
88 88
             else {
89 89
                 throw new Exception($statement->errorInfo());
90 90
             }
91
-        }else{
91
+        } else {
92 92
             // update
93 93
             $statement = $this->dbObject->prepare(<<<SQL
94 94
                 UPDATE `userpreference` SET 
Please login to merge, or discard this patch.
includes/Pages/Request/PageRequestAccount.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -324,7 +324,7 @@
 block discarded – undo
324 324
         {
325 325
             $value = WebRequest::httpHeader($header);
326 326
 
327
-            if($value === null){
327
+            if ($value === null) {
328 328
                 continue;
329 329
             }
330 330
 
Please login to merge, or discard this patch.
includes/Pages/UserAuth/PagePreferences.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -37,9 +37,9 @@
 block discarded – undo
37 37
         if (WebRequest::wasPosted()) {
38 38
             $this->validateCSRFToken();
39 39
 
40
-            $this->setPreference($preferencesManager,PreferenceManager::PREF_EMAIL_SIGNATURE, 'emailSignature');
41
-            $this->setPreferenceWithValue($preferencesManager,PreferenceManager::PREF_SKIP_JS_ABORT, 'skipJsAbort', WebRequest::postBoolean('skipJsAbort') ? 1 : 0);
42
-            $this->setPreferenceWithValue($preferencesManager,PreferenceManager::PREF_QUEUE_HELP, 'showQueueHelp', WebRequest::postBoolean('showQueueHelp') ? 1 : 0);
40
+            $this->setPreference($preferencesManager, PreferenceManager::PREF_EMAIL_SIGNATURE, 'emailSignature');
41
+            $this->setPreferenceWithValue($preferencesManager, PreferenceManager::PREF_SKIP_JS_ABORT, 'skipJsAbort', WebRequest::postBoolean('skipJsAbort') ? 1 : 0);
42
+            $this->setPreferenceWithValue($preferencesManager, PreferenceManager::PREF_QUEUE_HELP, 'showQueueHelp', WebRequest::postBoolean('showQueueHelp') ? 1 : 0);
43 43
             $this->setCreationMode($user, $preferencesManager);
44 44
             $this->setSkin($preferencesManager);
45 45
             $preferencesManager->setGlobalPreference(PreferenceManager::PREF_DEFAULT_DOMAIN, WebRequest::postInt('defaultDomain'));
Please login to merge, or discard this patch.
includes/Fragments/NavigationMenuAccessControl.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -117,13 +117,13 @@
 block discarded – undo
117 117
         $countOfJobQueue = 0;
118 118
 
119 119
         // Count of flagged comments:
120
-        if($this->barrierTest(RoleConfigurationBase::MAIN, $currentUser, PageListFlaggedComments::class)) {
120
+        if ($this->barrierTest(RoleConfigurationBase::MAIN, $currentUser, PageListFlaggedComments::class)) {
121 121
             // We want all flagged comments that haven't been acknowledged if we can visit the page.
122 122
             $countOfFlagged = sizeof(Comment::getFlaggedComments($database, 1)); // FIXME: domains
123 123
         }
124 124
 
125 125
         // Count of failed job queue changes:
126
-        if($this->barrierTest(RoleConfigurationBase::MAIN, $currentUser, PageJobQueue::class)) {
126
+        if ($this->barrierTest(RoleConfigurationBase::MAIN, $currentUser, PageJobQueue::class)) {
127 127
             // We want all failed jobs that haven't been acknowledged if we can visit the page.
128 128
             JobQueueSearchHelper::get($database, 1) // FIXME: domains
129 129
                 ->statusIn([JobQueue::STATUS_FAILED])
Please login to merge, or discard this patch.
includes/Pages/PageDomainSwitch.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@
 block discarded – undo
44 44
         try {
45 45
             $this->getDomainAccessManager()->switchDomain($currentUser, $newDomain);
46 46
         }
47
-        catch(DomainSwitchNotAllowedException $ex){
47
+        catch (DomainSwitchNotAllowedException $ex) {
48 48
             throw new AccessDeniedException($this->getSecurityManager(), $this->getDomainAccessManager());
49 49
         }
50 50
 
Please login to merge, or discard this patch.