We should use short array syntax as per new coding standard for the module

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pavan B S created an issue. See original summary.

Pavan B S’s picture

Assigned: Pavan B S » Unassigned
Status: Active » Needs review
FileSize
1.86 KB

Applying the patch, please review

dhruveshdtripathi’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
43.93 KB

Patch applied cleanly. All the arrays converted to short array syntax. Checked it manually. As you can see in the attached png file, earlier here were 3 array declarations, after applying patch there are 0.

Good work!

Liam Morland’s picture

Status: Reviewed & tested by the community » Fixed

Thanks!

Status: Fixed » Closed (fixed)

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