use short array syntax as per new coding standard.

CommentFileSizeAuthor
#3 Clean_up_the_codebase-2881351-3.patch722 byteschanderbhushan
#2 Clean_up_the_codebase-2881351-2.patch2.67 KBMunavijayalakshmi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Munavijayalakshmi created an issue. See original summary.

Munavijayalakshmi’s picture

Munavijayalakshmi CreditAttribution: Munavijayalakshmi at Valuebound commented
Assigned: Munavijayalakshmi » Unassigned
Status: Active » Needs review
FileSize
Clean_up_the_codebase-2881351-2.patch2.67 KB
chanderbhushan’s picture

chanderbhushan CreditAttribution: chanderbhushan as a volunteer and at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commented
FileSize
Clean_up_the_codebase-2881351-3.patch722 bytes

adding patch

nkoporec’s picture

nkoporec
Primary language English
Location Slovenia
CreditAttribution: nkoporec as a volunteer and at Agiledrop - Your Trusted Drupal Teammates commented
Status: Needs review » Reviewed & tested by the community

Tested #2 and it's working as expected, don't know what #3 patch is for...

chanderbhushan’s picture

chanderbhushan CreditAttribution: chanderbhushan as a volunteer and at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commented

Hi one place where array() is there so I removed

src/Plugin/Derivative/MenuBlock.php

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet at The University of British Columbia commented
Status: Reviewed & tested by the community » Fixed

Thank you, I've committed this with some updates to the -dev release.

Status: Fixed » Closed (fixed)

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