Skip to:
Content

Opened 4 weeks ago

Closed 3 weeks ago

Last modified 3 weeks ago

#3242 closed defect (fixed)

PHP 7.3 Compat: Continue Targeting Switch issues Warning

Reported by: rebasaurus Owned by: johnjamesjacoby
Milestone: 2.6 Priority: highest
Severity: major Version:
Component: General Keywords: needs-patch
Cc:

Description

In [includes/admin/parser.php](https://bbpress.trac.wordpress.org/browser/trunk/src/includes/admin/parser.php#L225) on lines 225, 237 and 241,

continue statements targeting switch control flow structures will now generate a warning. In PHP such
continue statements are equivalent to break, while they behave as continue 2 in other languages.

See http://php.net/manual/en/migration73.incompatible.php#migration73.incompatible.core.continue-targeting-switch

Change History (6)

#1 @johnjamesjacoby
3 weeks ago

  • Milestone changed from Awaiting Review to 2.6
  • Owner set to johnjamesjacoby

Kunena needed to fix this also.

#2 @johnjamesjacoby
3 weeks ago

We maintain a fork of NBBC on GitHub, so I'll get this fixed up over there.

https://github.com/bbpress/NBBC

#3 @johnjamesjacoby
3 weeks ago

  • Resolution set to fixed
  • Status changed from new to closed

In 6887:

Parser: Fix PHP 7.3 warning ("continue" in "switch" is equal to "break")

Fixes #3242.

#4 @rebasaurus
3 weeks ago

@johnjamesjacoby Thanks! Any chance this can make it to an upcoming RC/version or the current RC?

Last edited 3 weeks ago by rebasaurus (previous) (diff)

#5 @johnjamesjacoby
3 weeks ago

@rebasaurus It definitely will make it into the next RC. Thank you for bringing this up here. I really appreciate it.

Note: See TracTickets for help on using tickets.