Merge remote-tracking branch '32919/cleanup/remove-extra-space-in-link-script-tag' into community_prs_april

Author: Indraniks

lib/internal/Magento/Framework/View/Page/Config/Renderer.php

@@ -375,12 +375,12 @@ protected function getAssetTemplate($contentType, $attributes)
     {
         switch ($contentType) {
             case 'js':
-                $groupTemplate = '<script ' . $attributes . ' src="%s"></script>' . "\n";
+                $groupTemplate = preg_replace('/\s+/', ' ', '<script ' . $attributes . ' src="%s"></script>') . "\n";
                 break;
 
             case 'css':
             default:
-                $groupTemplate = '<link ' . $attributes . ' href="%s" />' . "\n";
+                $groupTemplate = preg_replace('/\s+/', ' ', '<link ' . $attributes . ' href="%s" />') . "\n";
                 break;
         }
         return $groupTemplate;

lib/internal/Magento/Framework/View/Test/Unit/Page/Config/RendererTest.php

@@ -427,40 +427,40 @@ public function dataProviderRenderAssets(): array
             [
                 ['type' => 'css', 'attributes' => '', 'condition' => null],
                 ['type' => 'js', 'attributes' => 'attr="value"', 'condition' => null],
-                '<link  rel="stylesheet" type="text/css"  media="all" href="url" />' . "\n"
-                    . '<link  rel="stylesheet" type="text/css"  media="all" href="url" />' . "\n"
-                    . '<script  type="text/javascript"  attr="value" src="no_route_url"></script>' . "\n"
+                '<link rel="stylesheet" type="text/css" media="all" href="url" />' . "\n"
+                    . '<link rel="stylesheet" type="text/css" media="all" href="url" />' . "\n"
+                    . '<script type="text/javascript" attr="value" src="no_route_url"></script>' . "\n"
             ],
             [
                 ['type' => 'js', 'attributes' => ['attr' => 'value'], 'condition' => 'lt IE 7'],
                 ['type' => 'css', 'attributes' => 'attr="value"', 'condition' => null],
-                '<link  rel="stylesheet" type="text/css"  attr="value" href="no_route_url" />' . "\n"
+                '<link rel="stylesheet" type="text/css" attr="value" href="no_route_url" />' . "\n"
                     . '<!--[if lt IE 7]>' . "\n"
-                    . '<script  type="text/javascript"  attr="value" src="url"></script>' . "\n"
-                    . '<script  type="text/javascript"  attr="value" src="url"></script>' . "\n"
+                    . '<script type="text/javascript" attr="value" src="url"></script>' . "\n"
+                    . '<script type="text/javascript" attr="value" src="url"></script>' . "\n"
                     . '<![endif]-->' . "\n"
             ],
             [
                 ['type' => 'ico', 'attributes' => 'attr="value"', 'condition' => null],
                 ['type' => 'css', 'attributes' => '', 'condition' => null],
-                '<link  rel="stylesheet" type="text/css"  media="all" href="no_route_url" />' . "\n"
-                    . '<link  attr="value" href="url" />' . "\n"
-                    . '<link  attr="value" href="url" />' . "\n"
+                '<link rel="stylesheet" type="text/css" media="all" href="no_route_url" />' . "\n"
+                    . '<link attr="value" href="url" />' . "\n"
+                    . '<link attr="value" href="url" />' . "\n"
             ],
             [
                 ['type' => 'js', 'attributes' => '', 'condition' => null],
                 ['type' => 'ico', 'attributes' => ['attr' => 'value'], 'condition' => null],
-                '<link  attr="value" href="no_route_url" />' . "\n"
-                    . '<script  type="text/javascript"  src="url"></script>' . "\n"
-                    . '<script  type="text/javascript"  src="url"></script>' . "\n"
+                '<link attr="value" href="no_route_url" />' . "\n"
+                    . '<script type="text/javascript" src="url"></script>' . "\n"
+                    . '<script type="text/javascript" src="url"></script>' . "\n"
             ],
             [
                 ['type' => 'non', 'attributes' => ['attr' => 'value'], 'condition' => null],
                 ['type' => 'ico', 'attributes' => '', 'condition' => null],
-                '<link  href="no_route_url" />' . "\n"
-                    . '<link  attr="value" href="url" />' . "\n"
-                    . '<link  attr="value" href="url" />' . "\n"
-            ]
+                '<link href="no_route_url" />' . "\n"
+                    . '<link attr="value" href="url" />' . "\n"
+                    . '<link attr="value" href="url" />' . "\n"
+            ],
         ];
     }
 
@@ -506,7 +506,7 @@ public function testRenderAssetWithNoContentType() : void
             ->willReturn([$groupMockOne]);
 
         $this->assertEquals(
-            '<link  rel="some-rel" href="url" />' . "\n",
+            '<link rel="some-rel" href="url" />' . "\n",
             $this->renderer->renderAssets($this->renderer->getAvailableResultGroups())
         );
     }