Re: Confused about model save/update

I may se the formating incorrectly (on a phone) but it looks like your save() method does nothing if self.pk already exists (i.e. model is not new)

HTH

Jirka

-----Original Message-----
From: Lachlan Musicman <datakid@gmail.com>
Sender: django-users@googlegroups.com
Date: Tue, 27 Nov 2012 11:07:42
To: <django-users@googlegroups.com>
Reply-To: django-users@googlegroups.com
Subject: Re: Confused about model save/update

On Tue, Nov 27, 2012 at 11:00 AM, <jirka.vejrazka@gmail.com> wrote:
> Hi there,
>
> a long shot since you have not provided your save() method. Are you calling super() there?

Jirka, yes I am, but it's only been modified to set the slug:

def save(self, *args, **kwargs):
if not self.pk:
super(Applicant, self).save(*args, **kwargs) # Call the
first save() method to get pk
self.slug = slugify(str(self))
super(Applicant, self).save(*args, **kwargs) # Call the
"real" save() method.


Cheers
L.

>
> Cheers
>
> Jirka
>
> -----Original Message-----
> From: Lachlan Musicman <datakid@gmail.com>
> Sender: django-users@googlegroups.com
> Date: Tue, 27 Nov 2012 10:14:03
> To: <django-users@googlegroups.com>
> Reply-To: django-users@googlegroups.com
> Subject: Confused about model save/update
>
> Hi
>
> Sorry about the last email - fat fingered it.
>
> For some reason I can't get the model.save() method to actually save
> the data I want it to. I have tried using both save() and
> save(force_update=True) without any joy.
>
> I have the following model:
>
> class Applicant(model.Models):
> ...
> successful = NullBooleanField()
> ...
>
> def mark_successful(self):
> self.successful = 0
> self.last_change_by = request.user
> #self.save(force_update=True)
> self.save()
>
> def mark_successful(self):
> self.successful = 1
> self.last_change_by = request.user
> self.save(force_update=True)
> self.save()
>
>
> I call these functions from the admin interface, via admin.py:
>
> class ApplicantAdmin(admin.ModelAdmin):
> ...
> actions = ['mark_successful', 'mark_unsuccessful']
> ...
>
> def mark_unsuccessful(self, request, queryset):
> '''Marks a group of applicants as unsuccessful'''
> rows_updated = 0
> for applicant in queryset:
> applicant.mark_unsuccessful(request)
> rows_updated += 1
>
> if rows_updated == 1:
> message_bit = "1 applicant was"
> else:
> message_bit = "%s applicants were" % rows_updated
> self.message_user(request, "%s marked unsuccessful." % message_bit)
>
>
>
> Can anyone see what's going wrong?
>
> cheers
> L.
>
> --
> ...we look at the present day through a rear-view mirror. This is
> something Marshall McLuhan said back in the Sixties, when the world
> was in the grip of authentic-seeming future narratives. He said, "We
> look at the present through a rear-view mirror. We march backwards
> into the future."
>
> http://www.warrenellis.com/?p=14314
>
> --
> You received this message because you are subscribed to the Google Groups "Django users" group.
> To post to this group, send email to django-users@googlegroups.com.
> To unsubscribe from this group, send email to django-users+unsubscribe@googlegroups.com.
> For more options, visit this group at http://groups.google.com/group/django-users?hl=en.
>
> --
> You received this message because you are subscribed to the Google Groups "Django users" group.
> To post to this group, send email to django-users@googlegroups.com.
> To unsubscribe from this group, send email to django-users+unsubscribe@googlegroups.com.
> For more options, visit this group at http://groups.google.com/group/django-users?hl=en.
>



--
...we look at the present day through a rear-view mirror. This is
something Marshall McLuhan said back in the Sixties, when the world
was in the grip of authentic-seeming future narratives. He said, "We
look at the present through a rear-view mirror. We march backwards
into the future."

http://www.warrenellis.com/?p=14314

--
You received this message because you are subscribed to the Google Groups "Django users" group.
To post to this group, send email to django-users@googlegroups.com.
To unsubscribe from this group, send email to django-users+unsubscribe@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/django-users?hl=en.

--
You received this message because you are subscribed to the Google Groups "Django users" group.
To post to this group, send email to django-users@googlegroups.com.
To unsubscribe from this group, send email to django-users+unsubscribe@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/django-users?hl=en.