Merge branch 'hotline#136213_probleme_d_affichage_portail' into 'master'

hotline#136213: fix iframe closing tag

See merge request !4058

VERSIONS_HOTLINE/136213

+ - ticket #136213 : Affichage : correction de la fermeture de balise iframe
\ No newline at end of file

library/Class/Cookies/Base.php

@@ -92,7 +92,7 @@ height="0" width="0" style="display:none;visibility:hidden"></iframe></noscript>
 
 
   public function getPlayer($name, $url) {
-    return sprintf('<iframe src="%s" crossorigin=1 allowfullscreen=1 allow= "accelerometer; autoplay; encrypted-media; gyroscope; picture-in-picture" class ="embed-responsive-item"/>', $url);
+    return sprintf('<iframe src="%s" crossorigin=1 allowfullscreen=1 allow="accelerometer; autoplay; encrypted-media; gyroscope; picture-in-picture" class="embed-responsive-item"> </iframe>', $url);
   }
 
 

tests/scenarios/AuthorPage/AuthorPageTest.php

@@ -698,6 +698,12 @@ class AuteurPageNoticeAjaxRenderYoutubeChannelTest extends AuthorPageTestCase {
   public function playlist441TitleShouldBePresent() {
     $this->assertXPathContentContains('//h3', 'David Guetta Playlist 441');
   }
+
+
+  /** @test */
+  public function iframesShouldBeExplicitlyClosed() {
+    $this->assertXPathCount('//iframe[child::text()=" "]', 3);
+  }
 }
 
 
@@ -738,7 +744,6 @@ class AuteurPageNoticeAjaxRenderYoutubeChannelTarteAuCitronTest extends AuthorPa
   /** @test */
   public function numberOfIframesShouldBeThree() {
     $this->assertXPathCount('//div[@class="youtube_player"]', 3);
-
   }