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

CommentFileSizeAuthor
#5 short_array_syntax-2869280-4.patch25.43 KBdom.
#2 short_array_syntax-2869280-2.patch20.72 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-2869280-2.patch20.72 KB
dhruveshdtripathi’s picture

dhruveshdtripathi commented
Status: Needs review » Needs work

After applying patch #2, I got these 2 errors:

warning: squelched 5 whitespace errors
warning: 10 lines add whitespace errors.

Also few arrays from . install and .module files are yet to converted in short array syntax.

  • Dom. committed 0e59707 on 8.x-1.x 
    Issue #2869280 by Dom., Munavijayalakshmi: Convert module to use short...
dom.’s picture

dom. commented
Status: Needs work » Fixed
StatusFileSize
new short_array_syntax-2869280-4.patch25.43 KB

Patch #2 do not apply anymore. New patch created and commited.

Status: Fixed » Closed (fixed)

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