govCMS Dissemination Limiting Marker (DLM) module needs to be converted to use short array syntax as per new coding standard.

CommentFileSizeAuthor
#3 govcms_dlm_array_search_results.png37.11 KBdhruveshdtripathi
#2 short_array_syntax-2874833-2.patch865 bytesMunavijayalakshmi

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-2874833-2.patch865 bytes
dhruveshdtripathi’s picture

dhruveshdtripathi commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new govcms_dlm_array_search_results.png37.11 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 3 array declarations, after applying patch there are 0.

Good work!

pkil’s picture

pkil commented
Status: Reviewed & tested by the community » Fixed

Thanks Munavijayalakshmi and dhruveshdtripathi I've merged this in and created a 8.x-1.1 tag.

Status: Fixed » Closed (fixed)

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