Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

params clash bug #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

params clash bug #13

wants to merge 1 commit into from

Conversation

milos-u
Copy link

@milos-u milos-u commented Dec 13, 2012

see test case..

@thiagofa
Copy link

It's a very critical bug. Why this pull request was not accepted yet?
I noticed this bug in production, fortunately, for a non critical module of my website, but it could be worst, for example my website could mix data of different users.

Before I see this pull request, I forked the code and fixed it in my own way (I'm not an expert in Python yet), as you can see at thiagofa@a5c32bb.

Wilfred added a commit to Wilfred/johnny-cache that referenced this pull request Jan 21, 2014
I used v1.5 in this commit: 4cf3db2
but since Django 1.5 support has been merged upstream, this fork now
only includes jmoiron#13 .
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants