[wp-trac] [WordPress Trac] #57832: Multiple classes should be in separate files (class-wp-block-parser.php)

WordPress Trac noreply at wordpress.org
Tue Jun 27 00:12:55 UTC 2023


#57832: Multiple classes should be in separate files (class-wp-block-parser.php)
----------------------------------------------+----------------------------
 Reporter:  aristath                          |       Owner:
                                              |  SergeyBiryukov
     Type:  enhancement                       |      Status:  accepted
 Priority:  normal                            |   Milestone:  6.3
Component:  Editor                            |     Version:  6.2
 Severity:  normal                            |  Resolution:
 Keywords:  has-patch commit gutenberg-merge  |     Focuses:  coding-
                                              |  standards
----------------------------------------------+----------------------------
Changes (by peterwilsoncc):

 * component:  General => Editor


Comment:

 The PR looks ready to commit, having discussed this with @isabel_brison
 I'll be committing this seperately to the other package updates as it's a
 little more complex than the usual package update.

 In order to retain the history in the new files, I'll be running the
 following commands:


 {{{
 svn cp ./src/wp-includes/class-wp-block-parser.php ./src/wp-includes
 /class-wp-block-parser-block.php
 rm ./src/wp-includes/class-wp-block-parser-block.php
 svn cp ./src/wp-includes/class-wp-block-parser.php ./src/wp-includes
 /class-wp-block-parser-frame.php
 rm ./src/wp-includes/class-wp-block-parser-frame.php
 gh pr diff 4152 --repo=WordPress/wordpress-develop | patch -p1
 }}}

 The `rm` commands are to make sure `patch` doesn't get upset creating the
 files but doesn't affect the history.

 I've moved this ticket to the Editor component as the bulk of the changes
 come from the upstream Gutenberg repo.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/57832#comment:25>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list