Skip to content

add htmlspecialchars to escape & characters to solve issue 374#375

Open
Nemo64 wants to merge 1 commit intogabrielbull:masterfrom
hauptsacheNet:fix/escaping
Open

add htmlspecialchars to escape & characters to solve issue 374#375
Nemo64 wants to merge 1 commit intogabrielbull:masterfrom
hauptsacheNet:fix/escaping

Conversation

@Nemo64
Copy link
Copy Markdown

@Nemo64 Nemo64 commented Jun 22, 2023

This is not really a nice fix.
It's just a search and replace for all createElement calls that adds htmlspecialchars everywhere

These were the patterns executed:
createElement\(([^,()]+\,\s*)(\$([^,;()]+|\([^;()]*\))+)\)
createElement($1($2) !== null ? htmlspecialchars($2) : null)

This is not really a nice fix.
It's just a search and replace for all createElement calls
that adds htmlspecialchars everywhere

These were the patterns executed:
`createElement\(([^,()]+\,\s*)(\$([^,;()]+|\([^;()]*\))+)\)`
`createElement($1($2) !== null ? htmlspecialchars($2) : null)`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant