rattfieldnz / safe-urls

tests/Libraries/Traits/StaticCallingTraitTest.php

Spacing +1 added lines, -1 removed lines

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

tests/SafeUrlsTest.php

Spacing +17 added lines, -17 removed lines

@@ -43,29 +43,29 @@ 
         $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);
 
         $this->assertEquals($expected, $actual);
     }
 
-    public function testRemoveAllUrls(){
+    public function testRemoveAllUrls() {
         $this->safeUrls->add($this->urlsToTest);
 
-        $expected = [];
+        $expected = [ ];
         $this->safeUrls->remove($this->urlsToTest);
         $actual = $this->safeUrls->getCurrentUrls(true);
 
         $this->assertEquals($expected, $actual);
     }
 
-    public function testExecute(){
+    public function testExecute() {
 
         // Only testing with returned HTTP status, as Google Safebrowsing API not working as
         // documented. See tests\Libraries\Curl\CurlTest testMalwareSocialEngineeringAnyPlatformUrl().
@@ -74,24 +74,24 @@ 
 
         $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);
 
     }
 
-    public function testGetResults(){
+    public function testGetResults() {
 
         $this->testExecute();
     }
 
-    public function testCheck(){
+    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);
     }
@@ -99,11 +99,11 @@ 
     /**
      * @expectedException \ErrorException
      */
-    public function testCheckThrowErrorException(){
+    public function testCheckThrowErrorException() {
         $this->safeUrls->add($this->urlsToTest);
 
         $safeUrlsMock = $this->getMockBuilder(SafeUrls::class)
-            ->setMethods(['callStatic'])
+            ->setMethods([ 'callStatic' ])
             ->getMock();
 
         $safeUrlsMock->method("callStatic")
@@ -114,7 +114,7 @@ 
 
     }
 
-    public function testIsDangerous(){
+    public function testIsDangerous() {
         $this->safeUrls->add($this->urlsToTest);
         $this->safeUrls->execute();
 

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;
     }

src/Libraries/Traits/StaticCalling.php

Spacing +1 added lines, -1 removed lines

@@ -20,6 +20,6 @@
         $parameters = func_get_args();
         $parameters = array_slice($parameters, 2); // Remove $className and $methodName
 
-        return call_user_func_array($className . '::' . $methodName, $parameters);
+        return call_user_func_array($className.'::'.$methodName, $parameters);
     }
 }

src/Libraries/Data/Data.php

Spacing +4 added lines, -4 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;
@@ -66,7 +66,7 @@ 
      */
     public static function googleApiUrl()
     {
-        return 'https://safebrowsing.googleapis.com/v4/threatMatches:find?key=' .
+        return 'https://safebrowsing.googleapis.com/v4/threatMatches:find?key='.
             Config::googleApiKey();
     }
 }

src/SafeUrls.php

Spacing +10 added lines, -10 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 = [ ];
     }
 
     /**
@@ -66,12 +66,12 @@ 
      *
      * @return array|string The set of results, in JSON.
      */
-    public function checkCallStatic(array $urls){
+    public function checkCallStatic(array $urls) {
 
         return $this->callStatic(SafeUrls::class, "check", $urls);
     }
 
-    public function execute(){
+    public function execute() {
         $this->results = SafeUrls::check($this->urls);
         return $this;
     }
@@ -111,11 +111,11 @@ 
      *
      * @return array|string
      */
-    public function getCurrentUrls(bool $asArray = false){
-        return $asArray == false ? json_encode($this->urls): $this->urls;
+    public function getCurrentUrls(bool $asArray = false) {
+        return $asArray == false ? json_encode($this->urls) : $this->urls;
     }
 
-    public function getResults(){
+    public function getResults() {
         return $this->results;
     }
     /**
@@ -128,9 +128,9 @@ 
      */
     public function isDangerous(string $url): bool
     {
-        $data = json_decode((string)$this->results);
-        $matches = empty($data->response["matches"]) ? null : $data->response["matches"];
-        if(empty($matches)){
+        $data = json_decode((string) $this->results);
+        $matches = empty($data->response[ "matches" ]) ? null : $data->response[ "matches" ];
+        if (empty($matches)) {
             return false;
         }
         foreach ($matches as $result) {