Skip to:

Opened 20 months ago

Closed 20 months ago

Last modified 20 months 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


In [includes/admin/parser.php]( 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.


Change History (6)

#1 @johnjamesjacoby
20 months ago

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

Kunena needed to fix this also.

#2 @johnjamesjacoby
20 months ago

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

#3 @johnjamesjacoby
20 months 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
20 months ago

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

Last edited 20 months ago by rebasaurus (previous) (diff)

#5 @johnjamesjacoby
20 months 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.