Issue: We are getting below warning visible on the drupal 7 site after upgrading the PHP version to 7.3.

Warning: "continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"? in drupal7/sites/all/modules/contrib/nodeaccess/nodeaccess_uuid/nodeaccess_uuid.module on line 74

CommentFileSizeAuthor
#2 reroll_diff_6-2.txt622 bytesminakshiPh
#2 fix-phpcompatibility-3133708-2.patch4.32 KBminakshiPh
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

minakshiPh created an issue. See original summary.

minakshiPh’s picture

minakshiPh CreditAttribution: minakshiPh at TATA Consultancy Services for Pfizer, Inc. commented
Status: Active » Needs review
FileSize
fix-phpcompatibility-3133708-2.patch4.32 KB
reroll_diff_6-2.txt622 bytes

Hi,

I have re-rolled the parent issue patch https://www.drupal.org/project/nodeaccess/issues/2784321#comment-11544809 to fix this warning.

Also uploaded diff of the parent issue patch https://www.drupal.org/project/nodeaccess/issues/2784321#comment-11544809 with this patch.

Please review.

Thanks,
minakshiPh

alison’s picture

alison
she/her
CreditAttribution: alison as a volunteer commented

@minakshiPh Hi and thank you!

I can't review this yet (or the other issue your submitted re: UUID module), but I will try to do so ASAP this week, just didn't want to leave you with continued radio silence.

priyanka.tawde’s picture

priyanka.tawde CreditAttribution: priyanka.tawde as a volunteer and commented
Status: Needs review » Reviewed & tested by the community

Hi,

I have checked and issued is fixed after applying the above patch.

  • ElusiveMind committed ddbc332 on 8.x-1.x 
    Issue #3133708 by minakshiPh: Warning: "continue" targeting switch is...
ElusiveMind’s picture

ElusiveMind CreditAttribution: ElusiveMind at ITCON Services Inc. commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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