WordPress.org

Make WordPress Core

Opened 5 weeks ago

Closed 5 weeks ago

Last modified 5 weeks ago

#48339 closed defect (bug) (invalid)

Remove empty DO statement

Reported by: dkarfa Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Editor Keywords:
Focuses: Cc:
PR Number:

Description

Found empty DO statement for WP Block Parser

Attachments (1)

48339.patch (449 bytes) - added by dkarfa 5 weeks ago.

Download all attachments as: .zip

Change History (4)

@dkarfa
5 weeks ago

#1 @ocean90
5 weeks ago

  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed

WP Block Parser is part of the @wordpress/block-serialization-default-parser package. Please submit any changes at https://github.com/WordPress/gutenberg first. Thank you!

From a quick look, your patch will break processing the tokens.

#2 @SergeyBiryukov
5 weeks ago

  • Component changed from General to Editor

#3 @SergeyBiryukov
5 weeks ago

Although the loop itself is empty, repeatedly calling $this->proceed() is important here and cannot be removed.

Note: See TracTickets for help on using tickets.