-
Notifications
You must be signed in to change notification settings - Fork 1.1k
ProfilingDebugPanel causes Admin model to save twice #209
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
Comments
I feel like this has to be something we're doing wrong, and at some point I may have fixed it in the old fork of DjDt. There's some pending changes to the profiling panel that I need to go over, but I think it's possible that it gets executed multiple times. |
Same problem here. I have a view in my that adds data, and when I enable the ProfilingDebugPanel I see that the data is doubled. Using debug-toolbar 8.5 |
I can confirm this. I have revisioned data where my POST request contains the revision of the object being updated, after it gets updated, the object has a new revision, and because view is called twice, the second time there is a mismatch between the revision in the POST request and which view expect to find in the database and the revision of the object in the database. So my view raises a conflict error (assuming somebody else changed the object). Once i disable profiling panel things work as expected again. |
There is a patch in ticket #204 for this, it seems? |
I have same Issue. Views that POST data to the server - do so twice - so I get duplicates everytime I create and object with the toolbar loaded. User 0.9.4 and Django 1.3.1 |
This bug seems to be fixed. However there is no release containing it. Are there any plans on releasing a new one some time soon? |
See #312. |
After scratching my head figuring out why my all my admin models gets saved twice, I nailed it down to ProfilingDebugPanel being the cause. Is it an unavoidable side effect of ProfilingDebugPanel?
The text was updated successfully, but these errors were encountered: