We have to convert the module to use short array as per new coding standard

CommentFileSizeAuthor
#3 legacy_redirect_array_search_results.png51.96 KBdhruveshdtripathi
#2 2876479-2.patch2.16 KBPavan B S
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

Pavan B S CreditAttribution: Pavan B S at Valuebound commented
Assigned: Pavan B S » Unassigned
Status: Active » Needs review
FileSize
2876479-2.patch2.16 KB

Applying the patch, please review

dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda commented
Status: Needs review » Reviewed & tested by the community
FileSize
legacy_redirect_array_search_results.png51.96 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 5 array declarations, after applying patch there are 0.

Good work!

  • kscheirer committed c968501 on 8.x-1.x authored by Pavan B S 
    Issue #2876479 by Pavan B S, dhruveshdtripathi: Convert module to use...
kscheirer’s picture

kscheirer
Primary language English
Location Vallejo
CreditAttribution: kscheirer commented
Status: Reviewed & tested by the community » Fixed

Thanks for the contribution! This patch has been committed.

Status: Fixed » Closed (fixed)

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