Files: 
CommentFileSizeAuthor
#7 views-rename_group_multiple_exposed_input-2003508-7.patch1.63 KBJeroenT
PASSED: [[SimpleTest]]: [MySQL] 57,406 pass(es).
[ View ]
#2 core-groupMultipleExposedInput_replace-2003508.patch1.62 KBRayCascella
FAILED: [[SimpleTest]]: [MySQL] 56,008 pass(es), 1 fail(s), and 0 exception(s).
[ View ]

Comments

RayCascella’s picture

Assigned:Unassigned» RayCascella

Working on issue at DC portland.

RayCascella’s picture

Status:Active» Needs review
StatusFileSize
new1.62 KB
FAILED: [[SimpleTest]]: [MySQL] 56,008 pass(es), 1 fail(s), and 0 exception(s).
[ View ]

My first patch. Hooray! Please work =-D

RayCascella’s picture

Assigned:RayCascella» Unassigned

Status:Needs review» Needs work
Issue tags:-Novice, -VDC

The last submitted patch, core-groupMultipleExposedInput_replace-2003508.patch, failed testing.

RayCascella’s picture

Status:Needs work» Needs review

Status:Needs review» Needs work
Issue tags:+Novice, +VDC

The last submitted patch, core-groupMultipleExposedInput_replace-2003508.patch, failed testing.

JeroenT’s picture

StatusFileSize
new1.63 KB
PASSED: [[SimpleTest]]: [MySQL] 57,406 pass(es).
[ View ]

rename group_multiple_exposed_input to groupMultipleExposedInput.

JeroenT’s picture

Status:Needs work» Needs review

marking as needs review.

RayCascella’s picture

Odd, no functionality was changed, yet it passed on the 3rd go around. Delay from the testing queue backup at DrupalCon?

EDIT: Also, thank you for updating the function names to keep them on standard. Completely slipped my mind with all of the sprint excitement in Portland. =-D

heddn’s picture

oenie’s picture

Status:Needs review» Reviewed & tested by the community

Looks good to me now !

alexpott’s picture

Status:Reviewed & tested by the community» Fixed

Committed 686a2ef and pushed to 8.x. Thanks!

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