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

CommentFileSizeAuthor
#6 short_array_syntax-2868594-5.patch100.59 KBbhupesh_upadhyay
#2 short_array_syntax-2868594-2.patch43.34 KBMunavijayalakshmi

Issue fork anonymous_publishing-2868594

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

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-2868594-2.patch43.34 KB
dhruveshdtripathi’s picture

dhruveshdtripathi commented
Status: Needs review » Needs work

Following warnings after applying patch:

warning: squelched 18 whitespace errors
warning: 23 lines add whitespace errors.

Many more arrays are yet to be converted into short array syntax.

samir_shukla made their first commit to this issue’s fork.

bhupesh_upadhyay’s picture

bhupesh_upadhyay commented
Assigned: Unassigned » bhupesh_upadhyay

Hello,
I'm working on this issue.

bhupesh_upadhyay’s picture

bhupesh_upadhyay commented
Assigned: bhupesh_upadhyay » Unassigned
Status: Needs work » Needs review
StatusFileSize
new short_array_syntax-2868594-5.patch100.59 KB

I have converted to short array syntax with new coding standards on 8.x-1.x, please review it.

  • gisle committed 9cbdff9 on 8.x-1.x 
    Issue #2868594 by Bhupesh_upadhyay: Use short array syntax
gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Status: Needs review » Fixed

Patch in comment #6 committed to most recent snapshot of 8.x-1.x-dev.

Status: Fixed » Closed (fixed)

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