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

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Munavijayalakshmi created an issue. See original summary.

Munavijayalakshmi’s picture

Assigned: Munavijayalakshmi » Unassigned
Status: Active » Needs review
FileSize
7.5 KB
dhruveshdtripathi’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
126.32 KB

Patch applied cleanly. All the arrays converted to short array syntax. Checked it manually. As you can see in the attached png file, earlier there were 19 array declarations, after applying patch there are 6. Those 6 arrays are in_arrays or is_arrays.

Good work!

tedbow’s picture

Status: Reviewed & tested by the community » Fixed

Committed!

@Munavijayalakshmi thanks for the patch!

@dhruveshdtripathi thanks for reviewing. Nice idea with the grep screenshot!

Status: Fixed » Closed (fixed)

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