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
FileSize
1.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

FileSize
1.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.