[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