Use Only mb_convert_encoding in StringHelper sanitizeUTF8

src/PhpSpreadsheet/Shared/StringHelper.php

@@ -334,7 +334,7 @@ public static function controlCharacterPHP2OOXML($textValue)
     public static function sanitizeUTF8(string $textValue): string
     {
         $textValue = str_replace(["\xef\xbf\xbe", "\xef\xbf\xbf"], "\xef\xbf\xbd", $textValue);
-        if (class_exists(UConverter::class)) {
+        if (class_exists(UConverter::class, false)) {
             $returnValue = UConverter::transcode($textValue, 'UTF-8', 'UTF-8');
             if ($returnValue !== false) {
                 return $returnValue;
@@ -349,8 +349,11 @@ public static function sanitizeUTF8(string $textValue): string
             }
         }
 
+        $subst = mb_substitute_character(); // default is question mark
+        mb_substitute_character(65533); // Unicode substitution character
         // Phpstan does not think this can return false.
         $returnValue = mb_convert_encoding($textValue, 'UTF-8', 'UTF-8');
+        mb_substitute_character($subst);
 
         return $returnValue;
         // @codeCoverageIgnoreEnd

tests/PhpSpreadsheetTests/Shared/StringHelperInvalidCharTest.php

@@ -2,6 +2,7 @@
 
 namespace PhpOffice\PhpSpreadsheetTests\Shared;
 
+use Exception as Except;
 use PhpOffice\PhpSpreadsheet\Shared\StringHelper;
 use PhpOffice\PhpSpreadsheet\Spreadsheet;
 use PHPUnit\Framework\TestCase;
@@ -41,4 +42,38 @@ public function testInvalidChar(): void
             );
         }
     }
+
+    public function fakespl(string $name): void
+    {
+        if (strlen($name) > 0) {
+            throw new Except("$name not found");
+        }
+    }
+
+    public function testClassNotFound(): void
+    {
+        // see issue 2982.
+        // This test will work all the time, but it is valuable
+        //     only if php-intl not available
+        //     and a user-supplied autoloader can issue exception.
+        /** @var callable */
+        $fakespl = [$this, 'fakespl'];
+        spl_autoload_register($fakespl);
+
+        try {
+            self::assertFalse(class_exists('abc123xyz'));
+            self::fail('Expected exception here');
+        } catch (Except $e) {
+            // Nothing to do here
+        }
+
+        $spreadsheet = new Spreadsheet();
+        $sheet = $spreadsheet->getActiveSheet();
+        $sheet->setCellValue('A1', 'Hello World !');
+
+        $spreadsheet->disconnectWorksheets();
+
+        spl_autoload_unregister($fakespl);
+        self::assertFalse(class_exists('abc123wxyz'));
+    }
 }