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

CommentFileSizeAuthor
#4 2873633-4.patch18.06 KBbhanuprakashnani
#2 2873633-2.patch1.98 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
2873633-2.patch1.98 KB

Applying the patch,please review.

Status: Needs review » Needs work

The last submitted patch, 2: 2873633-2.patch, failed testing.

bhanuprakashnani’s picture

bhanuprakashnani CreditAttribution: bhanuprakashnani at Google Summer of Code commented
Assigned: Unassigned » bhanuprakashnani
Status: Needs work » Needs review
FileSize
2873633-4.patch18.06 KB

Made a new patch changing the syntaxes. Please review and mention if any more changes are to be made. Thank you.

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
CreditAttribution: claudiu.cristea as a volunteer and at Webikon for European Commission and European Union Institutions, Agencies and Bodies commented
Status: Needs review » Closed (duplicate)

This is already fulfilled by #3069918: Fix coding standards and block non-standard patches.