Better Field Descriptions module needs to be converted to use short array syntax as per new coding standard

CommentFileSizeAuthor
#3 better_field_descriptions_array_search_results.png11.16 KBdhruveshdtripathi
#2 2870297-2.patch10.22 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
2870297-2.patch10.22 KB

Applying the patch, please review.

dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda commented
Status: Needs review » Reviewed & tested by the community
FileSize
better_field_descriptions_array_search_results.png11.16 KB

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

Good work!

yogeshmpawar’s picture

yogeshmpawar
He/Him/His
Primary language English
Location 🇮🇳 Pune(MH)
CreditAttribution: yogeshmpawar at QED42 commented
Status: Reviewed & tested by the community » Patch (to be ported)

Above patch working good for me, so moving this patch into dev branch.

yogeshmpawar’s picture

yogeshmpawar
He/Him/His
Primary language English
Location 🇮🇳 Pune(MH)
CreditAttribution: yogeshmpawar at QED42 commented
Status: Patch (to be ported) » Fixed

Committed & pushed to 8.x-1.x branch. Nice work !

Status: Fixed » Closed (fixed)

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