Django

Code

Changeset 7188

Show
Ignore:
Timestamp:
02/29/08 15:39:13 (6 months ago)
Author:
brosner
Message:

newforms-admin: Fixed #5735. Wrapped save_add and save_change with commit_on_success to ensure data consistency if an inline fails to save. Thanks, Honza Kral.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • django/branches/newforms-admin/django/contrib/admin/options.py

    r7178 r7188  
    66from django.contrib.admin.util import get_deleted_objects 
    77from django.core.exceptions import ImproperlyConfigured, PermissionDenied 
    8 from django.db import models 
     8from django.db import models, transaction 
    99from django.http import Http404, HttpResponse, HttpResponseRedirect 
    1010from django.shortcuts import get_object_or_404, render_to_response 
    …… 
    402402                post_url = '../../../' 
    403403            return HttpResponseRedirect(post_url) 
     404    save_add = transaction.commit_on_success(save_add) 
    404405 
    405406    def save_change(self, request, model, form, formsets=None): 
    …… 
    452453            request.user.message_set.create(message=msg) 
    453454            return HttpResponseRedirect("../") 
     455    save_change = transaction.commit_on_success(save_change) 
    454456 
    455457    def render_change_form(self, request, model, context, add=False, change=False, form_url='', obj=None):