S3 File System module needs to be converted to use short array syntax as per new coding standard.

CommentFileSizeAuthor
#3 short_array_syntax-2871329-3.png86.41 KBamit.drupal
#2 short_array_syntax-2871329-2.patch3.44 KBMunavijayalakshmi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Munavijayalakshmi created an issue. See original summary.

Munavijayalakshmi’s picture

Munavijayalakshmi CreditAttribution: Munavijayalakshmi at Valuebound commented
Assigned: Munavijayalakshmi » Unassigned
Status: Active » Needs review
FileSize
short_array_syntax-2871329-2.patch3.44 KB
amit.drupal’s picture

amit.drupal CreditAttribution: amit.drupal as a volunteer and at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commented
FileSize
short_array_syntax-2871329-3.png86.41 KB

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

amit.drupal’s picture

amit.drupal CreditAttribution: amit.drupal as a volunteer and at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commented
Status: Needs review » Reviewed & tested by the community
jansete’s picture

jansete CreditAttribution: jansete commented
Issue summary: View changes

Good job guys! I will add it soon.

jansete’s picture

jansete CreditAttribution: jansete commented
Issue summary: View changes
jansete’s picture

jansete CreditAttribution: jansete commented
Status: Reviewed & tested by the community » Needs work

  • jansete committed ae624cc on 8.x-3.x 
    Issue #2871329 by Munavijayalakshmi, amit.drupal, jansete: Convert...
jansete’s picture

jansete CreditAttribution: jansete commented
Parent issue: » #2863447: Improve coding standards
jansete’s picture

jansete CreditAttribution: jansete commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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