Problem/Motivation

Use the old array sintax

Proposed resolution

Use short array syntax

CommentFileSizeAuthor
#2 linkchecker-Use_short_array_syntax-3058160-2-D8.patch13.41 KBthalles
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

thalles created an issue. See original summary.

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
CreditAttribution: thalles at CI&T commented
Status: Needs work » Needs review
FileSize
linkchecker-Use_short_array_syntax-3058160-2-D8.patch13.41 KB

Follow the patch!

waverate’s picture

waverate CreditAttribution: waverate commented
Status: Needs review » Needs work

This looks already committed at 5fcf109.

Can you check if there are still arrays to be converted?

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
CreditAttribution: thalles at CI&T commented

Aparentemetn are other files.

waverate’s picture

waverate CreditAttribution: waverate commented

I expect /src/Controller/LinkCheckerAdminReportPage.php is going to be deleted in #2992604: Broken links report.

  • hass committed 23d1182 on 8.x-1.x authored by thalles 
    Issue #3058160 by thalles: Use short array syntax
hass’s picture

hass CreditAttribution: hass commented
Status: Needs work » Fixed

I committed this patch without the changes in LinkCheckerAdminReportPage.php as all this code seems to get deleted in #2992604: Broken links report.

Status: Fixed » Closed (fixed)

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