signify-nz / silverstripe-security-headers

src/Forms/GridField/GridFieldDeleteRelationsButton.php

Spacing +1 added lines, -1 removed lines

@@ -559,7 +559,7 @@
         }
         $data = urldecode($data);
 
-        $data = preg_replace_callback('/(?:^|(?<=&))[^=[]+/', function ($match) {
+        $data = preg_replace_callback('/(?:^|(?<=&))[^=[]+/', function($match) {
             return bin2hex(urldecode($match[0]));
         }, $data);
 

src/Controllers/CSPViolationsController.php

Spacing +1 added lines, -1 removed lines

@@ -143,7 +143,7 @@
                         // Firefox doesn't report the disposition.
                         return 'unknown';
                     }
-                    return ;
+                    return;
                 case self::BLOCKED_URI:
                     return $cspReport['blocked-uri'];
                 case self::EFFECTIVE_DIRECTIVE:

tests/SecurityHeaderMiddlewareTest.php

Spacing +3 added lines, -3 removed lines

@@ -99,7 +99,7 @@ 
                     ],
                 ],
             ],
-            function () use ($testURI) {
+            function() use ($testURI) {
                 $defaultUri = Director::absoluteURL(SecurityHeaderMiddleware::config()->get('report_uri'));
                 $response = $this->getResponse();
                 $csp = $response->getHeader('Content-Security-Policy');
@@ -129,7 +129,7 @@ 
                     'use_report_to' => true,
                 ],
             ],
-            function () {
+            function() {
                 $response = $this->getResponse();
                 $csp = $response->getHeader('Content-Security-Policy');
                 $reportHeaderExists = $response->getHeader('Report-To') !== null;
@@ -174,7 +174,7 @@ 
                     'use_report_to' => true,
                 ],
             ],
-            function () {
+            function() {
                 $defaultEndpoint = SecurityHeaderMiddleware::config()->get('report_to_group');
                 $defaultUri = Director::absoluteURL(SecurityHeaderMiddleware::config()->get('report_uri'));
                 $response = $this->getResponse();

src/Jobs/RemoveOldCSPViolationsJob.php

Braces +1 added lines, -2 removed lines

@@ -56,8 +56,7 @@
                 $report->delete();
                 $delta++;
             }
-        }
-        finally {
+        } finally {
             DB::get_conn()->transactionEnd();
         }
 

src/Jobs/RemoveUnreferencedCSPDocumentJob.php

Braces +1 added lines, -2 removed lines

@@ -49,8 +49,7 @@
                 }
                 ++$delta;
             }
-        }
-        finally {
+        } finally {
             DB::get_conn()->transactionEnd();
         }
 

src/Middleware/SecurityHeaderMiddleware.php

Spacing +2 added lines, -2 removed lines

@@ -212,7 +212,7 @@ 
             'max_age' => 1800,
             'endpoints' => [[
                 'url' => $this->getReportURI(),
-            ],],
+            ], ],
             'include_subdomains' => $this->getIncludeSubdomains(),
         ];
         return json_encode($header);
@@ -222,7 +222,7 @@ 
     {
         if ($this->isReporting()) {
             // Add or update report-uri directive.
-            if($cspHeader) {
+            if ($cspHeader) {
                 if (strpos($cspHeader, 'report-uri')) {
                     $cspHeader = str_replace('report-uri', $this->getReportURIDirective(), $cspHeader);
                 } else {

Braces +1 added lines, -2 removed lines

@@ -228,8 +228,7 @@
                 } else {
                     $cspHeader = rtrim($cspHeader, ';') . "; {$this->getReportURIDirective()};";
                 }
-            }
-            else {
+            } else {
                 $cspHeader = $this->getReportURIDirective() . ';';
             }
             // Add report-to directive.