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

django 1.6 admin saving #54

Open
vishen opened this issue Apr 11, 2014 · 3 comments
Open

django 1.6 admin saving #54

vishen opened this issue Apr 11, 2014 · 3 comments

Comments

@vishen
Copy link

vishen commented Apr 11, 2014

I know there is / was still a problem with django>=1.6 and the current atomic transactions django uses.

Django uses the @transaction.atomic around all of the admin views (https://github.com/django/django/blob/stable/1.6.x/django/contrib/admin/options.py#L1099). This causes a problem, because when johnny tries to invalidate a table, it checks if the current transaction is managed, and handles invalidation differently depending on if it is managed or not.

I was having the problem where I would add something in the admin panel, and I wasn't seeing the new object in the change view.

Sorry if this is a little sparse, writing this quickly so I don't lose my train of thought.

@Jaza
Copy link

Jaza commented Apr 30, 2014

I can confirm that this is a problem on latest stable Django (1.6.4). I deleted a comment via the admin, and the comment still displays in the admin listing, and in my article detail view in site's front-end. Commend was definitely deleted - clicking on it in the admin listing, to go to its admin detail page, results in 404.

Clearing all caches (i.e. restarting memcached) makes the deleted item disappear from admin / front-end. But that, of course, is not a solution.

Will look into this if I can, although might be a bit out of my depth debugging it in johnny-cache.

@ebrelsford
Copy link

Also seeing this, exactly as @Jaza is describing it.

gdub added a commit to gdub/johnny-cache that referenced this issue Jun 10, 2014
@Tatsh
Copy link

Tatsh commented Jul 27, 2014

Would like to see this in. I cannot create a new item in the admin back-end at all when caching is enabled.

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

No branches or pull requests

4 participants