- Issue created by @j-barnes
- Merge request !166Issue #3473897: Added check for text to fix deprecated warning. β (Closed) created by j-barnes
- Status changed to Needs review
3 months ago 3:20pm 12 September 2024 - π¦πΉAustria drunken monkey Vienna, Austria
drunken monkey β made their first commit to this issueβs fork.
- Merge request !167Fix failing preg_replace() leading to PHP warning β (Merged) created by drunken monkey
- π¦πΉAustria drunken monkey Vienna, Austria
Thanks for reporting this problem!
However, instead of just fixing the symptom, we should fix the root cause, i.e., thepreg_replace()
call failing. This probably happens sometimes when the matched strings get too large. (If you have an example of a text that makes this fail, it would be great to have for the tests!)I created MR !167 for replacing the
preg_replace()
with a more performant version, which should fix the problem. -
drunken monkey β
committed 199a6484 on 8.x-1.x
Issue #3473897 by drunken monkey: Fixed error in HTML Filter processor...
-
drunken monkey β
committed 199a6484 on 8.x-1.x
Automatically closed - issue fixed for 2 weeks with no activity.