The PHPCompatibility sniffer for PHPCS flagged an issues related to "Using 'continue' outside of a loop or switch structure"

$ phpcs --standard=PHPCompatibility --runtime-set testVersion 7.1 contrib/ultimate_cron

FILE: ...docroot/modules/contrib/ultimate_cron/src/CronPluginMultiple.php
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
45 | ERROR | Using 'continue' outside of a loop or switch structure
| | is invalid and will throw a fatal error since PHP 7.0
----------------------------------------------------------------------

CommentFileSizeAuthor
#2 continue_in_loop-2890428-2.patch409 bytesMojiferous
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

click2tman created an issue. See original summary.

Mojiferous’s picture

Mojiferous
he/him
CreditAttribution: Mojiferous at Elevated Third commented
Version: 8.x-2.0-alpha1 » 8.x-2.x-dev
Status: Active » Needs review
FileSize
continue_in_loop-2890428-2.patch409 bytes
17chances’s picture

17chances CreditAttribution: 17chances as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Looks good, +1 for RTBC

  • Berdir committed 27f2ce2 on 8.x-2.x authored by Mojiferous 
    Issue #2890428 by Mojiferous, click2tman, 17chances: Using 'continue'...
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Reviewed & tested by the community » Fixed

Thanks, I think that's dead code but committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.