Auto Login URl module needs to be converted to use short array syntax as per new coding standard.

CommentFileSizeAuthor
#9 array_cs-2867377-9.patch8.83 KBmatrixlord
#6 2867377-4.patch7.29 KBPavan B S
#4 2867377-3.patch7.72 KBPavan B S
#2 2867377-2.patch7.71 KBPavan B S

Comments

Pavan B S created an issue. See original summary.

Pavan B S’s picture

Pavan B S commented
Assigned: Pavan B S » Unassigned
Status: Active » Needs review
StatusFileSize
new 2867377-2.patch7.71 KB

Applying the patch, please review.

Status: Needs review » Needs work

The last submitted patch, 2: 2867377-2.patch, failed testing.

Pavan B S’s picture

Pavan B S commented
Status: Needs work » Needs review
StatusFileSize
new 2867377-3.patch7.72 KB

Status: Needs review » Needs work

The last submitted patch, 4: 2867377-3.patch, failed testing.

Pavan B S’s picture

Pavan B S commented
Status: Needs work » Needs review
StatusFileSize
new 2867377-4.patch7.29 KB

Status: Needs review » Needs work

The last submitted patch, 6: 2867377-4.patch, failed testing.

matrixlord’s picture

matrixlord commented

@Pavan B S I 've just made some changes for another issue #2867363: Error when enabling module that is why probably the patch cannot apply. Give me a moment to check the patch.

matrixlord’s picture

matrixlord commented
StatusFileSize
new array_cs-2867377-9.patch8.83 KB

Added a couple more arrays and fixed the issue with patch not finding file.

matrixlord’s picture

matrixlord commented
Status: Needs work » Needs review
Pavan B S’s picture

Pavan B S commented

@matrixlord thanks for helping on that

  • matrixlord committed e933386 on 8.x-1.x 
    Issue #2867377 by Pavan B S, matrixlord: Convert module to use short...
matrixlord’s picture

matrixlord commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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