zytzagoo / wp-google-fonts-optimizer

src/GoogleFontsCollection.php

Spacing +41 added lines, -41 removed lines

@@ -7,22 +7,22 @@ 
 class GoogleFontsCollection
 {
 
-    protected $links = [];
+    protected $links = [ ];
 
-    protected $texts = [];
+    protected $texts = [ ];
 
     /**
      * @var array|ZWF\GoogleFont[]
      */
-    protected $fonts = [];
+    protected $fonts = [ ];
 
-    protected $subsetsMap = [];
+    protected $subsetsMap = [ ];
 
-    protected $namedSizes = [];
+    protected $namedSizes = [ ];
 
-    public function __construct(array $urls = [])
+    public function __construct(array $urls = [ ])
     {
-        if (! empty($urls)) {
+        if (!empty($urls)) {
             foreach ($urls as $url) {
                 $this->add($url);
             }
@@ -33,24 +33,24 @@ 
     {
         $this->addLink($url);
 
-        $params = [];
+        $params = [ ];
         parse_str(parse_url($url, PHP_URL_QUERY), $params);
 
-        $bucket = isset($params['text']) ? 'text' : 'complete';
+        $bucket = isset($params[ 'text' ]) ? 'text' : 'complete';
 
         $fontstrings = $this->buildFontStringsFromQueryParams($params);
         foreach ($fontstrings as $font) {
             $this->addFontFromString($font, $bucket);
             if ('text' === $bucket) {
                 $font_family = explode(':', $font);
-                $this->addTextFont($font_family[0], Utils::httpsify($url));
+                $this->addTextFont($font_family[ 0 ], Utils::httpsify($url));
             }
         }
     }
 
     protected function addLink($url)
     {
-        $this->links[] = $url;
+        $this->links[ ] = $url;
     }
 
     protected function addFontFromString($fontstring, $bucket = 'complete')
@@ -60,15 +60,15 @@ 
         $subsets = $font->getSubsetsString();
 
         // Add to bucket list
-        $this->fonts[$bucket][] = $font;
+        $this->fonts[ $bucket ][ ] = $font;
 
         // Keeping a separate map of names => subsets for webfontloader purposes
-        if (! array_key_exists($name, $this->subsetsMap)) {
+        if (!array_key_exists($name, $this->subsetsMap)) {
             // Nothing found yet, create a new key
-            $this->subsetsMap[$name] = $subsets;
+            $this->subsetsMap[ $name ] = $subsets;
         } else {
             // Something is in there already, merge
-            $this->subsetsMap[$name] = array_merge($this->subsetsMap[$name], (array) $subsets);
+            $this->subsetsMap[ $name ] = array_merge($this->subsetsMap[ $name ], (array) $subsets);
         }
 
         // Deduplicate values and don't sort the subsets map
@@ -80,8 +80,8 @@ 
 
     protected function addTextFont($name, $url)
     {
-        $this->texts['name'][] = $name;
-        $this->texts['url'][]  = $url;
+        $this->texts[ 'name' ][ ] = $name;
+        $this->texts[ 'url' ][ ]  = $url;
     }
 
     /**
@@ -117,7 +117,7 @@ 
 
     public function hasText()
     {
-        return (! empty($this->texts));
+        return (!empty($this->texts));
     }
 
     /**
@@ -125,10 +125,10 @@ 
      */
     public function getTextUrls()
     {
-        $urls = [];
+        $urls = [ ];
 
         if ($this->hasText()) {
-            $urls = $this->texts['url'];
+            $urls = $this->texts[ 'url' ];
         }
 
         return $urls;
@@ -139,7 +139,7 @@ 
      */
     public function getTextNames()
     {
-        return $this->texts['name'];
+        return $this->texts[ 'name' ];
     }
 
     /**
@@ -147,7 +147,7 @@ 
      */
     public function getFonts()
     {
-        return $this->fonts['complete'];
+        return $this->fonts[ 'complete' ];
     }
 
     /**
@@ -166,11 +166,11 @@ 
      */
     public function getSubsets()
     {
-        $subsets = [];
+        $subsets = [ ];
 
         $fonts = $this->getFonts();
         foreach ($fonts as $font) {
-            $subsets[] = $font->getSubsets();
+            $subsets[ ] = $font->getSubsets();
         }
 
         $subsets = Utils::arrayFlattenIterative($subsets);
@@ -183,14 +183,14 @@ 
      */
     protected function buildNamedsizesMap()
     {
-        $fonts = [];
+        $fonts = [ ];
 
         foreach ($this->getFonts() as $font) {
             $name = $font->getName();
-            if (! array_key_exists($name, $fonts)) {
-                $fonts[$name] = $font->getSizes();
+            if (!array_key_exists($name, $fonts)) {
+                $fonts[ $name ] = $font->getSizes();
             } else {
-                $fonts[$name] = array_merge($fonts[$name], $font->getSizes());
+                $fonts[ $name ] = array_merge($fonts[ $name ], $font->getSizes());
             }
         }
 
@@ -213,12 +213,12 @@ 
      */
     protected function buildFontStringsFromQueryParams(array $params)
     {
-        $fonts = [];
+        $fonts = [ ];
 
-        foreach (explode('|', $params['family']) as $family) {
+        foreach (explode('|', $params[ 'family' ]) as $family) {
             $font = $this->parseFontStringName($family, $params);
-            if (! empty($font)) {
-                $fonts[] = $font;
+            if (!empty($font)) {
+                $fonts[ ] = $font;
             }
         }
 
@@ -236,12 +236,12 @@ 
         $subset = false;
         $family = explode(':', $family);
 
-        if (isset($params['subset'])) {
+        if (isset($params[ 'subset' ])) {
             // Use the found subset query parameter
-            $subset = $params['subset'];
-        } elseif (isset($family[2])) {
+            $subset = $params[ 'subset' ];
+        } elseif (isset($family[ 2 ])) {
             // Use the subset in the family string if present
-            $subset = $family[2];
+            $subset = $family[ 2 ];
         }
 
         // $family can have a lot of thing specified with separators etc.
@@ -261,19 +261,19 @@ 
      */
     protected function validateFontStringParts(array $family, $subset = false)
     {
-        $parts = [];
+        $parts = [ ];
 
         // First part is the font name, which should always be present
-        $parts[] = $family[0];
+        $parts[ ] = $family[ 0 ];
 
         // Check if sizes are specified
-        if (isset($family[1]) && strlen($family[1]) > 0) {
-            $parts[] = $family[1];
+        if (isset($family[ 1 ]) && strlen($family[ 1 ]) > 0) {
+            $parts[ ] = $family[ 1 ];
         }
 
         // Add the subset if specified
         if ($subset) {
-            $parts[] = $subset;
+            $parts[ ] = $subset;
         }
 
         return $parts;

src/GoogleFont.php

Spacing +12 added lines, -12 removed lines

@@ -7,11 +7,11 @@ 
 
     protected $name = '';
 
-    protected $sizes = [];
+    protected $sizes = [ ];
 
-    protected $subsets = [];
+    protected $subsets = [ ];
 
-    public function __construct($name, array $sizes = [], array $subsets = [])
+    public function __construct($name, array $sizes = [ ], array $subsets = [ ])
     {
         $this->setName($name);
         $this->setSizes($sizes);
@@ -84,11 +84,11 @@ 
         $sizes   = $this->getSizesString();
         $subsets = $this->getSubsetsString();
 
-        if (! empty($sizes)) {
-            $parts[] = $sizes;
+        if (!empty($sizes)) {
+            $parts[ ] = $sizes;
         }
-        if (! empty($subsets)) {
-            $parts[] = $subsets;
+        if (!empty($subsets)) {
+            $parts[ ] = $subsets;
         }
 
         return implode(':', $parts);
@@ -124,14 +124,14 @@ 
     public static function fromString($fontstring)
     {
         $parts = explode(':', $fontstring);
-        $font  = new self($parts[0]);
+        $font  = new self($parts[ 0 ]);
 
-        if (isset($parts[1])) {
-            $font->setSizes($parts[1]);
+        if (isset($parts[ 1 ])) {
+            $font->setSizes($parts[ 1 ]);
         }
 
-        if (isset($parts[2])) {
-            $font->setSubsets($parts[2]);
+        if (isset($parts[ 2 ])) {
+            $font->setSubsets($parts[ 2 ]);
         }
 
         return $font;

src/GoogleFontsOptimizer.php

Spacing +23 added lines, -23 removed lines

@@ -16,14 +16,14 @@ 
     const FILTER_OB_CLEANER  = 'zwf_gfo_clean_ob';
     const DEFAULT_OB_CLEANER = false;
 
-    protected $candidates = [];
+    protected $candidates = [ ];
 
-    protected $enqueued = [];
+    protected $enqueued = [ ];
 
     /**
      * @param array $urls
      */
-    public function setCandidates(array $urls = [])
+    public function setCandidates(array $urls = [ ])
     {
         $this->candidates = $urls;
     }
@@ -52,7 +52,7 @@ 
                  * Scan and optimize requests found in the markup (uses more
                  * memory but works on [almost] any theme)
                  */
-                add_action('template_redirect', [$this, 'startBuffering'], 11);
+                add_action('template_redirect', [ $this, 'startBuffering' ], 11);
                 break;
 
             case self::DEFAULT_OPERATION_MODE:
@@ -61,7 +61,7 @@ 
                  * Scan only things added via wp_enqueue_style (uses slightly
                  * less memory usually, but requires a decently coded theme)
                  */
-                add_filter('print_styles_array', [$this, 'processStylesHandles']);
+                add_filter('print_styles_array', [ $this, 'processStylesHandles' ]);
                 break;
         }
     }
@@ -83,7 +83,7 @@ 
         $candidate_handles = $this->findCandidateHandles($handles);
 
         // Bail if we don't have anything that makes sense for us to continue
-        if (! $this->hasEnoughElements($candidate_handles)) {
+        if (!$this->hasEnoughElements($candidate_handles)) {
             return $handles;
         }
 
@@ -97,14 +97,14 @@ 
         $font_url    = $collection->getCombinedUrl();
         $handle_name = 'zwf-gfo-combined';
         $this->enqueueStyle($handle_name, $font_url);
-        $handles[] = $handle_name;
+        $handles[ ] = $handle_name;
 
         // Process 'text-only' requests if there are any...
         $texts = $collection->getTextUrls();
         foreach ($texts as $k => $url) {
             $handle_name = 'zwf-gfo-combined-txt-' . ($k + 1);
             $this->enqueueStyle($handle_name, $url);
-            $handles[] = $handle_name;
+            $handles[ ] = $handle_name;
         }
 
         // Remove/dequeue the ones we just combined above
@@ -131,14 +131,14 @@ 
     protected function findCandidateHandles(array $handles)
     {
         $handler           = /** @scrutinizer ignore-call */ \wp_styles();
-        $candidate_handles = [];
+        $candidate_handles = [ ];
 
         foreach ($handles as $handle) {
             $dep = $handler->query($handle, 'registered');
             if ($dep) {
                 $url = $dep->src;
                 if (Utils::isGoogleWebFontUrl($url)) {
-                    $candidate_handles[$handle] = $url;
+                    $candidate_handles[ $handle ] = $url;
                 }
             }
         }
@@ -153,7 +153,7 @@ 
      *
      * @return bool
      */
-    protected function hasEnoughElements(array $candidates = [])
+    protected function hasEnoughElements(array $candidates = [ ])
     {
         $enough = true;
 
@@ -184,7 +184,7 @@ 
                 \wp_dequeue_style($handle);
             }
             // @codeCoverageIgnoreEnd
-            unset($this->enqueued[$handle]);
+            unset($this->enqueued[ $handle ]);
         }
     }
 
@@ -199,7 +199,7 @@ 
      *
      * @return void
      */
-    protected function enqueueStyle($handle, $url, $deps = [], $version = null)
+    protected function enqueueStyle($handle, $url, $deps = [ ], $version = null)
     {
         // @codeCoverageIgnoreStart
         if (function_exists('\wp_enqueue_style')) {
@@ -208,7 +208,7 @@ 
         }
         // @codeCoverageIgnoreEnd
 
-        $this->enqueued[$handle] = $url;
+        $this->enqueued[ $handle ] = $url;
     }
 
     /**
@@ -222,8 +222,8 @@ 
     {
         $data = $this->enqueued;
 
-        if (null !== $handle && isset($this->enqueued[$handle])) {
-            $data = $this->enqueued[$handle];
+        if (null !== $handle && isset($this->enqueued[ $handle ])) {
+            $data = $this->enqueued[ $handle ];
         }
 
         return $data;
@@ -241,7 +241,7 @@ 
     public function processMarkup($markup)
     {
         $hrefs = $this->parseMarkupForHrefs($markup);
-        if (! empty($hrefs)) {
+        if (!empty($hrefs)) {
             $this->setCandidates($hrefs);
         }
 
@@ -249,7 +249,7 @@ 
         $candidates = $this->getCandidates();
 
         // Bail and return original markup unmodified if we don't have things to do
-        if (! $this->hasEnoughElements($candidates)) {
+        if (!$this->hasEnoughElements($candidates)) {
             return $markup;
         }
 
@@ -291,11 +291,11 @@ 
      */
     protected function filterHrefsFromCandidateLinkNodes(\DOMNodeList $nodes)
     {
-        $hrefs = [];
+        $hrefs = [ ];
 
         foreach ($nodes as $node) {
             if ($this->isCandidateLink($node)) {
-                $hrefs[] = $node->getAttribute('href');
+                $hrefs[ ] = $node->getAttribute('href');
             }
         }
 
@@ -366,7 +366,7 @@ 
              */
 
             // Start our own buffering
-            $started = ob_start([$this, 'endBuffering']);
+            $started = ob_start([ $this, 'endBuffering' ]);
         }
 
         return $started;
@@ -375,7 +375,7 @@ 
     public function endBuffering($markup)
     {
         // Bail early on things we don't want to parse
-        if (! $this->isMarkupDoable($markup)) {
+        if (!$this->isMarkupDoable($markup)) {
             return $markup;
         }
 
@@ -460,7 +460,7 @@ 
         $html5 = Utils::hasHtml5Doctype($content);
         $xsl   = Utils::hasXslStylesheet($content);
 
-        return (($html || $html5) && ! $xsl);
+        return (($html || $html5) && !$xsl);
     }
 
     /**