[wp-trac] [WordPress Trac] #47014: Tag balancing corrupts Custom Elements
WordPress Trac
noreply at wordpress.org
Wed Jun 5 14:51:35 UTC 2019
#47014: Tag balancing corrupts Custom Elements
--------------------------------------+------------------------
Reporter: westonruter | Owner: flixos90
Type: defect (bug) | Status: reviewing
Priority: normal | Milestone: 5.3
Component: Formatting | Version: 2.0.4
Severity: normal | Resolution:
Keywords: has-patch has-unit-tests | Focuses:
--------------------------------------+------------------------
Comment (by birgire):
Good point @dmsnell regarding the extra maintenance burden of having the
regex pattern duplicated.
Having an easy way to remove the comments from the regex pattern, e.g.
from the PHP shell example you provided, sounds good to me, if one needs
to test it further with regex tools.
For the tests in [attachment:"47014-4.diff"] I would suggest :
- adding {{{@ticket 47014}}} annotation to new {{{test_}}} methods
- {{{@since 5.3.0}}} annotation to new non {{{test_}}} methods.
- adding some missing end of line dots (also in tests)
- e.g. use {{{@see}}} annotation in the comment: {{{// see http...}}}.
PS: also few missing end of line dots, both in code comments and tests
comments.
All the best
--
Ticket URL: <https://core.trac.wordpress.org/ticket/47014#comment:11>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list