Context Navigation

← Previous Ticket
Next Ticket →

#44944 closed defect (bug) (duplicate)

Possibly incorrect use of "continue" in wp-includeds/pomo/plural-forms.php

Reported by:	tobifjellner	Owned by:	SergeyBiryukov
Priority:	normal	Milestone:
Component:	I18N	Version:
Severity:	normal	Keywords:	php73
Cc:		Focuses:

Description

It seems as if wp-includeds/pomo/plural-forms.php uses a "stray" continue in line 210. This causes a warning to appear on some sites:

Warning: "continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"? in [REDACTED]/wp-includes/pomo/plural-forms.php on line 210

Initial report (in Swedish): @rdcsg in https://sv.wordpress.org/support/topic/konstigt-felmeddelande-dyker-upp-pa-sajten-sen-dagar-tillbaka/

Attachments (1)

44944.patch (1.0 KB ) - added by ayeshrajans 8 years ago.: This is due to PHP 7.3 deprecating this approach. Attaching a patch for this one and one more use in phpunit utils.php.

Download all attachments as: .zip

Change History (4)

@ayeshrajans
8 years ago

Attachment 44944.patch added

This is due to PHP 7.3 deprecating this approach. Attaching a patch for this one and one more use in phpunit utils.php.

#1 @SergeyBiryukov
8 years ago

Milestone Awaiting Review → 4.9.9
Owner set to SergeyBiryukov
Status new → reviewing

#2 @jorbin
8 years ago

Milestone 4.9.9
Resolution → duplicate
Status reviewing → closed

This is fixed in [43653] and was previously opened as #44543.

#3 @desrosj
8 years ago

Keywords php73 added

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core