Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

How do I update an already existing row when using ModelForms?

I have a question on how to update an existing row in my database when one of the fields is my primary key. I am using ModelForm and Django-Piston - my main goal here is to have RESTful Post send to my webservice. I am able to have initial Posts be sent correctly (i.e. that Primary key value doesn't exist yet). The problem is when I want to update a value where the Primary key already exists - when I issue a f.is_valid() it fails because "this UniqueIdentifier already exists". How can I do form validation using ModelForms to update an existing row?

My models.py:

from django.db import models
class DeviceModel(models.Model):
    uniqueIdentifier = models.CharField(primary_key=True, max_length=100)
    deviceToken = models.CharField(max_length=100)

forms.py

from django import forms
from models import DeviceModel
class DeviceModelForm(forms.ModelForm):
    class Meta:
        model = DeviceModel

handlers.py

class DeviceHandler(BaseHandler):
allowed_methods = ('POST', 'GET', 'DELETE',)
def create(self, request):
    f = DeviceModelForm(request.POST)
    if f.is_valid():
        new_object = f.save()
        return new_object
    return rc.BAD_REQUEST

urls.py

from django.conf.urls.defaults import *
from piston.resource import Resource
from api.handlers import DeviceHandler

device_handler = Resource(DeviceHandler)

urlpatterns = patterns('',
    (r'^api/$', device_handler, {'emitter_format': 'json'}),
)
like image 458
letsgofast Avatar asked Nov 23 '09 00:11

letsgofast


3 Answers

The django docs give a simple example of how to create "a form to change an existing [[entity]]":

>>> article = Article.objects.get(pk=1)
>>> form = ArticleForm(instance=article)

If as it seems you want to use the same flow both for inserting new objects and changing existing ones, you'll have to instantiate the form separately depending on whether looking for the primary key succeeds (existing object) or fails (new object)!-)

like image 132
Alex Martelli Avatar answered Oct 03 '22 20:10

Alex Martelli


To update an existing row (or object in ORM-speak), you have to tell the ModelForm what instance to use when instantiating it:

f = DeviceModelForm(request.POST, instance=myobject)

I'm not sure where you get myobject from using piston, though, but your question seems to imply that you solved that particular problem already.

like image 21
Benjamin Wohlwend Avatar answered Oct 04 '22 20:10

Benjamin Wohlwend


Here is a more complete solution not using any Class based views, bringing together the other answers and comments on this page.

I have it working as a reply to a jquery ajax.

def save_product(request):                                                                       
    if request.method == "POST":                                                                 
        # first get the model pk we are looking for
        postpk = request.POST.get('pk', None)

        # get the model from the db                                                
        model, created = Product.objects.get_or_create(pk = postpk)                          

        # create the from based on the model, but with the 
        # request data overriding the model data                                                 
        form = ProductForm(request.POST, instance = model)

        # save if valid                                       
        if form.is_valid():                                                                      
            form.save()                                                                          
            return HttpResponse("saved")                                         
        else:                             
            # will go to the the ajax error: data.responseText                                                       
            return HttpResponseNotFound("%s" % (form.errors))                                    
    else:                                                                                        
        return HttpResponseNotFound('eh? this was not a Post?')   
like image 39
SpiRail Avatar answered Oct 04 '22 20:10

SpiRail