CODEWITHSUNDEEP
djangodjango-modelsdjango-forms
pyprism
pyprism

Reputation: 3008

django form not updating as expected

Here is my model:

class Browser(models.Model):
    profile_name = models.CharField(max_length=400)
    browser_type = (
        ('fr', 'Firefox'),
        ('ch', 'Chrome'),
        ('op', 'Opera'),
        ('ot', 'Other'),
    )
    browser_name = models.CharField(choices=browser_type, max_length=2)
    device_name = models.CharField(max_length=400)
    created_at = models.DateTimeField(auto_now_add=True)
    updated_at = models.DateTimeField(auto_now=True)


class Bookmark(models.Model):
    browser = models.ForeignKey(Browser, on_delete=models.CASCADE, null=True, blank=True)
    title = models.TextField()
    url = models.TextField()
    iv = models.TextField()
    salt = models.TextField()
    iteration = models.IntegerField(default=1500)
    tags = TaggableManager()

I only want to update certain fields, so here is the modelform

class BookmarkFormEdit(ModelForm):
    class Meta:
        model = Browser
        exclude = ('tags', 'browser_name', 'device_name', 'profile_name')

but my problem is, values are not updating as expected . Here is the view:

def bookmark_edit(request, pk=None):
    if request.method == 'POST':
        bookmark = Bookmark.objects.get(pk=pk)
        frm = BookmarkFormEdit(request.POST, instance=bookmark)
        print(request.POST.get('iteration')) // printing correct value from front-end
        if frm.is_valid():
            x = frm.save()
            print(x.iteration) // not saving the new value !
            return JsonResponse({'status': 'created'})
        else:
            return JsonResponse({'error': frm.errors})
    return render(request, 'bookmark_edit.html', {'pk': pk})

Upvotes: 0

Views: 63

Answers (1)

Rohan
Rohan

Reputation: 53386

You are incorrectly defined model in the form. You should use Bookmark model instead of Browser.

class BookmarkFormEdit(ModelForm):
    class Meta:
        model = Bookmark

You may need to define fields to include/exclude as you want for this model.

Upvotes: 1

Related Questions