Skip to content

Commit 93dc40c

Browse files
committedJul 12, 2024
pkp/pkp-lib#7916 revert back symfony html sanitizer and bring back html purifier
1 parent 1bcf1a2 commit 93dc40c

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed
 

‎classes/ArticleBody.php

+7-6
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,6 @@
1818
use PKP\config\Config;
1919
use PKP\core\PKPString;
2020
use PKP\search\SearchFileParser;
21-
use Symfony\Component\HtmlSanitizer\HtmlSanitizer;
22-
use Symfony\Component\HtmlSanitizer\HtmlSanitizerConfig;
2321

2422
class ArticleBody extends \DOMDocument
2523
{
@@ -49,12 +47,15 @@ public function create(Submission $submission):\DOMNode
4947
$filepath = $fileService->get($galleyFile->getData('fileId'))->path;
5048
$mimeType = $fileService->fs->mimeType($filepath);
5149
if (in_array($mimeType, ['text/html'])) {
52-
static $sanitizer = null;
53-
if (!$sanitizer) {
54-
$sanitizer = new \PKP\core\PKPHtmlSanitizer('p');
50+
static $purifier;
51+
if (!$purifier) {
52+
$config = \HTMLPurifier_Config::createDefault();
53+
$config->set('HTML.Allowed', 'p');
54+
$config->set('Cache.SerializerPath', 'cache');
55+
$purifier = new \HTMLPurifier($config);
5556
}
5657
// Remove non-paragraph content
57-
$text = $sanitizer->sanitize(file_get_contents(Config::getVar('files', 'files_dir') . '/' . $filepath));
58+
$text = $purifier->purify(file_get_contents(Config::getVar('files', 'files_dir') . '/' . $filepath));
5859

5960
// Remove empty paragraphs
6061
} else {

0 commit comments

Comments
 (0)