The module needs to be converted to use short array syntax as per new coding standard.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bandanasharma created an issue. See original summary.

bandanasharma’s picture

FileSize
22.8 KB
22.8 KB

I have created the patch for this. Please review the patch.

bandanasharma’s picture

Status: Active » Needs review
bandanasharma’s picture

dhruveshdtripathi’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
129.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 79 array declarations, after applying patch there are 0.

Good work!

renatog’s picture

Assigned: Unassigned » renatog
Issue tags: +ciandt-contrib

Thanks guys.

Looks good for me.

  • RenatoG committed 927c4ab on 8.x-1.x authored by bandanasharma
    Issue #2876902 by bandanasharma, dhruveshdtripathi, RenatoG: Convert...
renatog’s picture

Status: Reviewed & tested by the community » Fixed

Commited in dev branch.

Thanks people.

Regards.

Status: Fixed » Closed (fixed)

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