PHP_Codesniffer running the PHPCompatibility standard produces the following error:

FILE: workflow/workflow.tokens.inc
------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
------------------------------------------------------------------------------------
 187 | ERROR | Indirect access to variables, properties and methods will be
     |       | evaluated strictly in left-to-right order since PHP 7.0. Use curly
     |       | braces to remove ambiguity.
------------------------------------------------------------------------------------

I'm not sure if this is creating any side-effects but I'm going for clean codesniffs so posting a patch here in case others find it useful.

CommentFileSizeAuthor
#2 php7_error_indirect-2918484-2.patch673 bytesjamesrward

Comments

jamesrward created an issue. See original summary.

jamesrward’s picture

jamesrward commented
Status: Active » Needs review
StatusFileSize
new php7_error_indirect-2918484-2.patch673 bytes

Here's a patch that passes the codesniff.

  • johnv committed 63f7bd2 on 7.x-2.x authored by jamesrward 
    Issue #2918484 by jamesrward: PHP7 Error: Indirect access to variable
johnv’s picture

johnv
Primary language Dutch
commented
Version: 7.x-2.x-dev » 7.x-2.10
Priority: Normal » Minor
Status: Needs review » Fixed

Thanks, committed as is.

  • johnv committed a5c902c on 7.x-2.x authored by jamesrward 
    Issue #2918484 by jamesrward: PHP7 Error: Indirect access to variable

Status: Fixed » Closed (fixed)

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