Group Forum module needs to be converted to use short array syntax as per new coding standard

CommentFileSizeAuthor
#3 gforum_array_search_list.png52.22 KBdhruveshdtripathi
#2 2870941-2.patch4.84 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
2870941-2.patch4.84 KB

Applying the patch, please review

dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda commented
Status: Needs review » Patch (to be ported)
FileSize
gforum_array_search_list.png52.22 KB

Patch applied cleanly. All the arrays converted to short array syntax. Checked it manually.

Good work!

dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda commented
Status: Patch (to be ported) » Reviewed & tested by the community

Sorry. Wrong status selected by mistake. Changing status to RTBC.

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

arnoldbird CreditAttribution: arnoldbird commented
Status: Reviewed & tested by the community » Fixed

Thanks

arnoldbird’s picture

arnoldbird CreditAttribution: arnoldbird commented
Status: Fixed » Closed (fixed)