Closed (fixed)
Project:
Extensible BBCode
Version:
4.0.x-dev
Component:
Core
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
25 Jan 2023 at 14:03 UTC
Updated:
11 Feb 2023 at 23:59 UTC
Jump to comment: Most recent
Comments
Comment #2
cburschkaThis issue appears to have already existed in 9.4.8.
Comment #3
cburschkaCorrection, the issue has always existed when returning Filter::tips() as a string instead of Markup. This was introduced by #3207134: Add strict types to xbbcode, which in a twist of dramatic irony does mention "methods which can return strings or MarkupInterface" as a potential obstacle, but fails to recognize that Filter::tips() is such a method.
Due to PHP silently casting the return value to string in methods that are hinted as string (instead of triggering a type error), this was not previously discovered.
Comment #4
cburschkaPushing a test-only patch. This should fail.
Comment #6
cburschkaPushing the fix. This should pass.
Comment #8
cburschkaThis probably needs to be backported to 4.0.x, because the strict return types were in there as well.
Comment #10
cburschka