Opened 8 years ago

Closed 7 years ago

Last modified 5 years ago

#9545 closed (worksforme)

FileField: height_field and width_field are not working

Reported by: julian24 Owned by: nobody
Component: File uploads/storage Version: 1.0
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:


With this model height_field and width_field aren't working:

class Wallpaper(models.Model):
        title = models.CharField(_('Title'), max_length=80)
        wallpaper = models.ImageField(_('Wallpaper'),
          upload_to = 'wallpapers/',
          height_field = 'wallpaper_height',
          width_field = 'wallpaper_width')
        wallpaper_height = models.IntegerField(_('Wallpaper height'), blank=True, null=True)
        wallpaper_width = models.IntegerField(_('Wallpaper width'), blank=True, null=True)
        def __unicode__(self):
                return unicode(self.title)

But if I add the following save method, it works fine:

        def save(self):
                if self.wallpaper.field.width_field:
                        setattr(self.wallpaper.instance, self.wallpaper.field.width_field, self.wallpaper.width)
                if self.wallpaper.field.height_field:
                        setattr(self.wallpaper.instance, self.wallpaper.field.height_field, self.wallpaper.height)
                super(Wallpaper, self).save()

Change History (5)

comment:1 Changed 8 years ago by kmtracey

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

What does "aren't working" mean? Is this possibly the same as #8208?

comment:2 Changed 8 years ago by julian24

It means that the fields are empty.

comment:3 Changed 7 years ago by jacob

  • milestone set to 1.1
  • Triage Stage changed from Unreviewed to Accepted

comment:4 Changed 7 years ago by Alex

  • Resolution set to worksforme
  • Status changed from new to closed

There are now several tests for this, I'm going to mark as worksforme, if you can show case where it doesn't work please reopen with a testcase.

comment:5 Changed 5 years ago by jacob

  • milestone 1.1 deleted

Milestone 1.1 deleted

Note: See TracTickets for help on using tickets.
Back to Top