Fix Html::parseImage when image has no extension

CHANGELOG.md

@@ -21,6 +21,7 @@ This project adheres to [Semantic Versioning](http:https://semver.org/).
 - PclZip : strtr using empty string by @spl1nes in #2432
 - Fixed PHP 8.2 deprecated about Allow access to an undefined property by @DAdq26 in #2440
 - Template Processor : Fixed choose dimention for Float Value by @gdevilbat in #2449
+- HTML Parser : Fix image parsing from url without extension by @JokubasR in #2459
 
 ### Miscellaneous
 

src/PhpWord/Shared/Html.php

@@ -965,7 +965,10 @@ protected static function parseImage($node, $element)
  $tmpDir = Settings::getTempDir() . '/';
  $match = [];
  preg_match('/.+\.(\w+)$/', $src, $match);
- $src = $tmpDir . uniqid() . '.' . $match[1];
+ $src = $tmpDir . uniqid();
+ if (isset($match[1])) {
+ $src .= '.' . $match[1];
+ }
 
  $ifp = fopen($src, 'wb');
 

tests/PhpWordTests/AbstractWebServerEmbeddedTest.php

@@ -53,6 +53,15 @@ protected static function getRemoteImageUrl()
  return 'http:https://php.net/images/logos/new-php-logo.png';
  }
 
+ protected static function getRemoteImageUrlWithoutExtension(): string
+ {
+ if (self::$httpServer) {
+ return self::getBaseUrl() . '/images/new-php-logo';
+ }
+
+ return 'http:https://placekitten.com/200/300';
+ }
+
  protected static function getRemoteGifImageUrl()
  {
  if (self::$httpServer) {

tests/PhpWordTests/Shared/HtmlTest.php

@@ -816,6 +816,24 @@ public function testParseRemoteImage(): void
  self::assertTrue($doc->elementExists($baseXpath . '/w:pict/v:shape'));
  }
 
+ /**
+ * Test parsing of remote img without extension.
+ */
+ public function testParseRemoteImageWithoutExtension(): void
+ {
+ $src = self::getRemoteImageUrlWithoutExtension();
+
+ $phpWord = new PhpWord();
+ $section = $phpWord->addSection();
+ $html = '<p><img src="' . $src . '" width="150" height="200" style="float: right;"/><img src="' . $src . '" style="float: left;"/></p>';
+ Html::addHtml($section, $html);
+
+ $doc = TestHelperDOCX::getDocument($phpWord, 'Word2007');
+
+ $baseXpath = '/w:document/w:body/w:p/w:r';
+ self::assertTrue($doc->elementExists($baseXpath . '/w:pict/v:shape'));
+ }
+
  /**
  * Test parsing embedded image.
  */