use short array syntax as per new coding standard.

CommentFileSizeAuthor
#3 role_based_theme_switcher_array_search_results.png44.94 KBdhruveshdtripathi
#2 Clean_up_the_codebase-2875678-2.patch3.23 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-2875678-2.patch3.23 KB
dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda commented
Status: Needs review » Reviewed & tested by the community
FileSize
role_based_theme_switcher_array_search_results.png44.94 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 there were 17 array declarations, after applying patch there are 4. Those 4 arrays are in_array.

Good work!

harsh.behl’s picture

harsh.behl CreditAttribution: harsh.behl at TO THE NEW commented
Status: Reviewed & tested by the community » Fixed

Hi Munavijayalakshmi ,

I have committed your code in latest dev branch.

Thanks for contribution.

Status: Fixed » Closed (fixed)

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