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

CommentFileSizeAuthor
#3 nimbus_array_search.png91.54 KBdhruveshdtripathi
#2 2868524-2.patch2.39 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
2868524-2.patch2.39 KB

Applying the patch, please review.

dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda for OpenSense Labs commented
Status: Needs review » Reviewed & tested by the community
FileSize
nimbus_array_search.png91.54 KB

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

Good work!

  • marvin_B8 committed 7e595e1 on 8.x-1.x authored by Pavan B S 
    Issue #2868524 by Pavan B S, dhruveshdtripathi: Convert module to use...
marvin_B8’s picture

marvin_B8 CreditAttribution: marvin_B8 as a volunteer and at comm-press commented
Status: Reviewed & tested by the community » Fixed

Thank you for your work.

Status: Fixed » Closed (fixed)

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