Fix for #2781 #5219

Merged
merged 1 commit into from Feb 29, 2012

Conversation

Projects
None yet
6 participants
Contributor

mattfawcett commented Feb 29, 2012

Fix the assert_recognizes test method so that it works when there are constraints on the querystring.

Issue #2781

@tenderlove tenderlove added a commit that referenced this pull request Feb 29, 2012

@tenderlove tenderlove Merge pull request #5219 from mattfawcett/2781-fix-querystring-route-…
…constraint-for-testing

Fix for #2781
1d21528

@tenderlove tenderlove merged commit 1d21528 into rails:master Feb 29, 2012

Contributor

bronson commented May 4, 2012

@tenderlove any chance this fix could also go on 3-2-stable? It applies cleanly and seems to deserve a part in the next 3.2.x release.

Contributor

jfirebaugh commented Aug 23, 2012

I tried briefly to backport it -- it applies cleanly but the test fails.

Contributor

maxim commented Jan 26, 2013

No hope for 3-2-stable?

Member

steveklabnik commented Jan 26, 2013

If someone wants to do the work of backporting it, feel free.

beedub referenced this pull request Feb 22, 2013

Merged

PR #5219 backported to 3-2 #9368

@steveklabnik steveklabnik added a commit that referenced this pull request Sep 10, 2013

@steveklabnik steveklabnik Merge pull request #9368 from CrowdFlower/3-2-stable
PR #5219 backported to 3-2
cff8d1d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment