Closed (fixed)
Project:
Extensible BBCode
Version:
5.0.x-dev
Component:
Coding standards
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
6 Nov 2023 at 12:12 UTC
Updated:
15 May 2024 at 16:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #5
sandipta commentedComment #7
gisleI've created a MR. It is identical to the patch by sandipta.It was created in parallell. Moving to "Needs review".
Comment #8
raveen_thakur51 commented@gisle, I have reviewed the MR!, Yes it was a straight forward fix. Thank You
Comment #9
raveen_thakur51 commentedComment #12
cburschkaThank you!
I'll check and port this change to other supported branches wherever needed. As far as I can tell, the double pipe syntax has been recommended since at least 2017, so it should be safe to do this on all versions.
Comment #14
cburschkaCherry-picked and pushed to 4.0.x (with the appropriate version numbers).
3.0.x was D8 only (and predates core_version_requirement), so no change is needed.
6.0.x is currently marked as D10 only (I'll add D11 at some point after D11 is in beta), so for now it doesn't use a logical OR and doesn't need to be changed.
That's all :)