Problem/Motivation

The module have some arrays definitions that need to be updated.

Proposed resolution

Use short array syntax as per new coding standard. Read more at PHP 5.4 short array syntax coding standards for Drupal 8.

See this tool: https://github.com/adrian-cid/php-short-array-syntax-converter

Remaining tasks

None.

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#3 path_alias_xt_array_search.png9.29 KBdhruveshdtripathi
#2 short_array_syntax-2868922-2.patch3.82 KBMunavijayalakshmi

Comments

Munavijayalakshmi created an issue. See original summary.

Munavijayalakshmi’s picture

Munavijayalakshmi commented
Assigned: Munavijayalakshmi » Unassigned
Status: Active » Needs review
StatusFileSize
new short_array_syntax-2868922-2.patch3.82 KB
dhruveshdtripathi’s picture

dhruveshdtripathi commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new path_alias_xt_array_search.png9.29 KB

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

Good work!

adriancid credited adriancid.

adriancid’s picture

adriancid
Primary language Spanish
Location Qc, Canada 🇨🇦
commented
Title: Convert module to use short array syntax (new coding standard). » Update the arrays definitions to the short syntax
Issue summary: View changes

Adding the Issue Template

  • adriancid committed 3039408 on 8.x-1.x 
    Issue #2868922 by Munavijayalakshmi, dhruveshdtripathi, adriancid:...
adriancid’s picture

adriancid
Primary language Spanish
Location Qc, Canada 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Thanks for the patch and the review, please consider for the next time the use of the Issue Summary Template.

Status: Fixed » Closed (fixed)

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