limonte / php-adblock-parser

src/AdblockRule.php

Spacing +5 added lines, -5 removed lines

@@ -43,8 +43,8 @@ 
      */
     public function matchUrl($url)
     {
-        return (boolean)preg_match(
-            '/' . $this->getRegex() . '/',
+        return (boolean) preg_match(
+            '/'.$this->getRegex().'/',
             $url
         );
     }
@@ -113,18 +113,18 @@ 
 
         // | in the end means the end of the address
         if ($this->endsWith($regex, '|')) {
-            $regex = mb_substr($regex, 0, mb_strlen($regex) - 1) . '$';
+            $regex = mb_substr($regex, 0, mb_strlen($regex) - 1).'$';
         }
 
         // || in the beginning means beginning of the domain name
         if ($this->startsWith($regex, '||')) {
             if (mb_strlen($regex) > 2) {
                 // http://tools.ietf.org/html/rfc3986#appendix-B
-                $regex = "^([^:\/?#]+:)?(\/\/([^\/?#]*\.)?)?" . mb_substr($regex, 2);
+                $regex = "^([^:\/?#]+:)?(\/\/([^\/?#]*\.)?)?".mb_substr($regex, 2);
             }
         // | in the beginning means start of the address
         } elseif ($this->startsWith($regex, '|')) {
-            $regex = '^' . mb_substr($regex, 1);
+            $regex = '^'.mb_substr($regex, 1);
         }
 
         // other | symbols should be escaped

src/AdblockParser.php

Doc Comments +1 added lines, -2 removed lines

@@ -69,7 +69,7 @@ 
     /**
      * @param  string  $url
      *
-     * @return integer
+     * @return boolean
      */
     public function shouldBlock($url)
     {
@@ -118,7 +118,6 @@ 
     /**
      * Set external resources cache expire (in days)
      *
-     * @param  integer  $expire
      */
     public function setCacheExpire($expireInDays)
     {

Spacing +8 added lines, -8 removed lines

@@ -7,11 +7,11 @@ 
 
     private $cacheExpire = 1; // 1 day
 
-    const CACHE_FOLDER = __DIR__ . DIRECTORY_SEPARATOR . '..' . DIRECTORY_SEPARATOR . 'cache' . DIRECTORY_SEPARATOR;
+    const CACHE_FOLDER = __DIR__.DIRECTORY_SEPARATOR.'..'.DIRECTORY_SEPARATOR.'cache'.DIRECTORY_SEPARATOR;
 
-    public function __construct($rules = [])
+    public function __construct($rules = [ ])
     {
-        $this->rules = [];
+        $this->rules = [ ];
         $this->addRules($rules);
     }
 
@@ -22,15 +22,15 @@ 
     {
         foreach ($rules as $rule) {
             try {
-                $this->rules[] = new AdblockRule($rule);
+                $this->rules[ ] = new AdblockRule($rule);
             } catch (InvalidRuleException $e) {
                 // Skip invalid rules
             }
         }
 
         // Sort rules, eceptions first
-        usort($this->rules, function ($a, $b) {
-            return (int)$a->isException() < (int)$b->isException();
+        usort($this->rules, function($a, $b) {
+            return (int) $a->isException() < (int) $b->isException();
         });
     }
 
@@ -130,7 +130,7 @@ 
      */
     public function clearCache()
     {
-        foreach (glob(self::CACHE_FOLDER . '*') as $file) {
+        foreach (glob(self::CACHE_FOLDER.'*') as $file) {
             unlink($file);
         }
     }
@@ -142,7 +142,7 @@ 
      */
     private function getCachedResource($url)
     {
-        $cacheFile = self::CACHE_FOLDER . basename($url) . md5($url);
+        $cacheFile = self::CACHE_FOLDER.basename($url).md5($url);
 
         if (file_exists($cacheFile) && (filemtime($cacheFile) > (time() - 60 * 24 * $this->cacheExpire))) {
             // Cache file is less than five minutes old.