rattfieldnz / safe-urls

src/Libraries/Data/Data.php

Spacing +3 added lines, -3 removed lines

@@ -26,7 +26,7 @@ 
      *
      * @return array The generated payload.
      */
-    public static function payload(array $urls, array $threatTypes = [], array $platformTypes = [], array $threatEntryTypes = []): array
+    public static function payload(array $urls, array $threatTypes = [ ], array $platformTypes = [ ], array $threatEntryTypes = [ ]): array
     {
         return [
             'client' => [
@@ -51,9 +51,9 @@ 
      */
     public static function formatUrls(array $urls): array
     {
-        $formattedUrls = [];
+        $formattedUrls = [ ];
         foreach ($urls as $url) {
-            $formattedUrls[] = ['url' => $url];
+            $formattedUrls[ ] = [ 'url' => $url ];
         }
 
         return $formattedUrls;

src/SafeUrlsServiceProvider.php

Spacing +6 added lines, -6 removed lines

@@ -44,7 +44,7 @@ 
     public function register()
     {
         // Register the service the package provides.
-        $this->app->singleton('safe-urls', function () {
+        $this->app->singleton('safe-urls', function() {
             return new SafeUrls();
         });
     }
@@ -56,7 +56,7 @@ 
      */
     public function provides()
     {
-        return ['safe-urls'];
+        return [ 'safe-urls' ];
     }
 
     /**
@@ -72,7 +72,7 @@ 
         ], 'safe-urls');
 
         // Registering package commands.
-        $this->commands(['safe-urls']);
+        $this->commands([ 'safe-urls' ]);
     }
 
     /**
@@ -85,8 +85,8 @@ 
      */
     protected function mergeConfigFrom($path, $key)
     {
-        $config = $this->app['config']->get($key, []);
-        $this->app['config']->set($key, $this->mergeConfig($config, require $path));
+        $config = $this->app[ 'config' ]->get($key, [ ]);
+        $this->app[ 'config' ]->set($key, $this->mergeConfig($config, require $path));
     }
 
     /**
@@ -110,7 +110,7 @@ 
             if (is_numeric($key)) {
                 continue;
             }
-            $array[$key] = $this->mergeConfig($value, $merging[$key]);
+            $array[ $key ] = $this->mergeConfig($value, $merging[ $key ]);
         }
 
         return $array;

tests/Libraries/Traits/StaticCallingTraitTest.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
             $testClass->callStatic(
                 $className, $methodName, $this->urlsToTest
             ), true
-        )['status'];
+        )[ 'status' ];
 
         $this->assertEquals($expected, $actual);
     }

tests/SafeUrlsTest.php

Spacing +11 added lines, -11 removed lines

@@ -43,12 +43,12 @@ 
         $urlToRemove = 'https://testsafebrowsing.appspot.com/s/phishing.html';
 
         $expected = [
-            $this->urlsToTest[1],
-            $this->urlsToTest[2],
-            $this->urlsToTest[3],
-            $this->urlsToTest[4],
+            $this->urlsToTest[ 1 ],
+            $this->urlsToTest[ 2 ],
+            $this->urlsToTest[ 3 ],
+            $this->urlsToTest[ 4 ],
         ];
-        $this->safeUrls->remove([$urlToRemove]);
+        $this->safeUrls->remove([ $urlToRemove ]);
 
         $actual = $this->safeUrls->getCurrentUrls(true);
 
@@ -59,7 +59,7 @@ 
     {
         $this->safeUrls->add($this->urlsToTest);
 
-        $expected = [];
+        $expected = [ ];
         $this->safeUrls->remove($this->urlsToTest);
         $actual = $this->safeUrls->getCurrentUrls(true);
 
@@ -76,8 +76,8 @@ 
 
         $expected = '{ "status": 200, "response": { "matches": [ { "threatType": "SOCIAL_ENGINEERING", "platformType": "ANY_PLATFORM", "threat": { "url": "https://testsafebrowsing.appspot.com/s/phishing.html" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "MALWARE", "platformType": "ANY_PLATFORM", "threat": { "url": "https://testsafebrowsing.appspot.com/s/malware.html" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "MALWARE", "platformType": "ANY_PLATFORM", "threat": { "url": "http://testsafebrowsing.appspot.com/apiv4/ANY_PLATFORM/MALWARE/URL/" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "SOCIAL_ENGINEERING", "platformType": "ANY_PLATFORM", "threat": { "url": "http://testsafebrowsing.appspot.com/apiv4/ANY_PLATFORM/SOCIAL_ENGINEERING/URL/" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "MALWARE", "platformType": "ANY_PLATFORM", "threat": { "url": "http://malware.testing.google.test/testing/malware/" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "SOCIAL_ENGINEERING", "platformType": "ANY_PLATFORM", "threat": { "url": "http://malware.testing.google.test/testing/malware/" }, "cacheDuration": "300s", "threatEntryType": "URL" } ] } }';
 
-        $expected = json_decode($expected, true)['status'];
-        $actual = json_decode((string) $this->safeUrls->execute()->getResults(), true)['status'];
+        $expected = json_decode($expected, true)[ 'status' ];
+        $actual = json_decode((string) $this->safeUrls->execute()->getResults(), true)[ 'status' ];
 
         $this->assertEquals($expected, $actual);
     }
@@ -90,9 +90,9 @@ 
     public function testCheck()
     {
         $expected = '{ "status": 200, "response": { "matches": [ { "threatType": "SOCIAL_ENGINEERING", "platformType": "ANY_PLATFORM", "threat": { "url": "https://testsafebrowsing.appspot.com/s/phishing.html" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "MALWARE", "platformType": "ANY_PLATFORM", "threat": { "url": "https://testsafebrowsing.appspot.com/s/malware.html" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "MALWARE", "platformType": "ANY_PLATFORM", "threat": { "url": "http://testsafebrowsing.appspot.com/apiv4/ANY_PLATFORM/MALWARE/URL/" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "SOCIAL_ENGINEERING", "platformType": "ANY_PLATFORM", "threat": { "url": "http://testsafebrowsing.appspot.com/apiv4/ANY_PLATFORM/SOCIAL_ENGINEERING/URL/" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "MALWARE", "platformType": "ANY_PLATFORM", "threat": { "url": "http://malware.testing.google.test/testing/malware/" }, "cacheDuration": "300s", "threatEntryType": "URL" }, { "threatType": "SOCIAL_ENGINEERING", "platformType": "ANY_PLATFORM", "threat": { "url": "http://malware.testing.google.test/testing/malware/" }, "cacheDuration": "300s", "threatEntryType": "URL" } ] } }';
-        $expected = json_decode($expected, true)['status'];
+        $expected = json_decode($expected, true)[ 'status' ];
 
-        $actual = json_decode($this->safeUrls::check($this->urlsToTest), true)['status'];
+        $actual = json_decode($this->safeUrls::check($this->urlsToTest), true)[ 'status' ];
 
         $this->assertEquals($expected, $actual);
     }
@@ -105,7 +105,7 @@ 
         $this->safeUrls->add($this->urlsToTest);
 
         $safeUrlsMock = $this->getMockBuilder(SafeUrls::class)
-            ->setMethods(['callStatic'])
+            ->setMethods([ 'callStatic' ])
             ->getMock();
 
         $safeUrlsMock->method('callStatic')

src/SafeUrls.php

Spacing +3 added lines, -3 removed lines

@@ -35,10 +35,10 @@ 
     public function __construct()
     {
         // Initialise the list of urls as an empty array.
-        $this->urls = [];
+        $this->urls = [ ];
 
         // Initialise the list of urls as an empty array.
-        $this->results = [];
+        $this->results = [ ];
     }
 
     /**
@@ -135,7 +135,7 @@ 
     public function isDangerous(string $url): bool
     {
         $data = json_decode((string) $this->results);
-        $matches = empty($data->response['matches']) ? null : $data->response['matches'];
+        $matches = empty($data->response[ 'matches' ]) ? null : $data->response[ 'matches' ];
         if (empty($matches)) {
             return false;
         }