pkp/pkp-lib#7916 revert back symfony html sanitizer and bring back ht…

…ml purifier
pkp · Jul 16, 2024 · c7099bc · c7099bc
1 parent 64175a3
commit c7099bc
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/classes/ArticleBody.php b/classes/ArticleBody.php
@@ -18,8 +18,6 @@
 use PKP\config\Config;
 use PKP\core\PKPString;
 use PKP\search\SearchFileParser;
-use Symfony\Component\HtmlSanitizer\HtmlSanitizer;
-use Symfony\Component\HtmlSanitizer\HtmlSanitizerConfig;
 
 class ArticleBody extends \DOMDocument
 {
@@ -49,12 +47,15 @@ public function create(Submission $submission):\DOMNode
             $filepath = $fileService->get($galleyFile->getData('fileId'))->path;
             $mimeType = $fileService->fs->mimeType($filepath);
             if (in_array($mimeType, ['text/html'])) {
-                static $sanitizer = null;
-                if (!$sanitizer) {
-                    $sanitizer = new \PKP\core\PKPHtmlSanitizer('p');
+                static $purifier;
+                if (!$purifier) {
+                    $config = \HTMLPurifier_Config::createDefault();
+                    $config->set('HTML.Allowed', 'p');
+                    $config->set('Cache.SerializerPath', 'cache');
+                    $purifier = new \HTMLPurifier($config);
                 }
                 // Remove non-paragraph content
-                $text = $sanitizer->sanitize(file_get_contents(Config::getVar('files', 'files_dir') . '/' . $filepath));
+                $text = $purifier->purify(file_get_contents(Config::getVar('files', 'files_dir') . '/' . $filepath));
 
                 // Remove empty paragraphs
             } else {