Merge pull request #63 from tswestendorp/remove-quotes-vue-related-patch

Don't strip quotes around Vue prop values

Author: Renato Marinho

src/Middleware/RemoveQuotes.php

@@ -7,17 +7,17 @@ class RemoveQuotes extends PageSpeed
     public function apply($buffer)
     {
         $replace = [
-            '/src="(.*?)"/' => 'src=$1',
-            '/width="(.*?)"/' => 'width=$1',
-            '/height="(.*?)"/' => 'height=$1',
-            '/name="(.*?)"/' => 'name=$1',
-            '/charset="(.*?)"/' => 'charset=$1',
-            //'/href="(.*?)"/' => 'href=$1',
-            '/align="(.*?)"/' => 'align=$1',
-            '/border="(.*?)"/' => 'border=$1',
-            '/crossorigin="(.*?)"/' => 'crossorigin=$1',
-            //'/rel="(.*?)"/' => 'rel=$1',
-            '/type="(.*?)"/' => 'type=$1',
+            '/ src="(.*?)"/' => ' src=$1',
+            '/ width="(.*?)"/' => ' width=$1',
+            '/ height="(.*?)"/' => ' height=$1',
+            '/ name="(.*?)"/' => ' name=$1',
+            '/ charset="(.*?)"/' => ' charset=$1',
+            //'/ href="(.*?)"/' => ' href=$1',
+            '/ align="(.*?)"/' => ' align=$1',
+            '/ border="(.*?)"/' => ' border=$1',
+            '/ crossorigin="(.*?)"/' => ' crossorigin=$1',
+            //'/ rel="(.*?)"/' => ' rel=$1',
+            '/ type="(.*?)"/' => ' type=$1',
             '/\/>/' => '>',
         ];
 

tests/Boilerplate/index.html

@@ -48,6 +48,8 @@ <h1>Test Background Image</h1>
                 <span ng-if="!btnLoading"> ok </span> <span ng-if="btnLoading"> <i class="fa fa-circle-o-notch fa-spin"></i> </span>
             </button>
 
+            <vue-component :src="'src'" :type="'type'" :width="200"></vue-component>
+
         </form>
 
         <script src="https://github.com/renatomarinho/laravel-page-speed/test/Boilerplate/js/vendor/modernizr-3.5.0.min.js"></script>

tests/Middleware/RemoveQuotes.php renamed to tests/Middleware/RemoveQuotesTest.php

@@ -23,5 +23,6 @@ public function testRemoveQuotes()
         $this->assertContains('<meta name=viewport content="width=device-width, initial-scale=1">', $response->getContent());
         $this->assertContains('<img src=http://emblemsbf.com/img/18346.jpg width=250 style="height:300px; padding:10px" >', $response->getContent());
         $this->assertContains('<img src=/images/1000coin.png>', $response->getContent());
+        $this->assertContains('<vue-component :src="\'src\'" :type="\'type\'" :width="200"></vue-component>', $response->getContent());
     }
 }