Django загрузка картинки в папку по id записи?

Question

[Dauren S]

В админке создал поле для загрузки изображения. все нормально загружается, но я хочу чтобы изображения загружались в папку, которая генерировалась бы по id записи.
Т.е создаю новую запись в админке, создается папка с id этой записи и туда сохраняются картинки

Answer 1

[Roman K]

Ты можешь у ImageField (также, как и FileField) переопределить поле upload_to и присвоить ему функцию, принимающую два аргумента - обект модели и имя файла. У тебя получится что-то вроде этого:

import os
from django.conf import settings
def get_image_path(instance, filename):
    # Но так никогда нельзя делать!
    return os.path.join('images', str(instance.pk), filename)

А вот так будет лучше:

import os
from uuid import uuid1
def get_image_path(instance, filename):
    result = os.path.join('images', str(instance.pk), uuid1().hex)
    if '.' in filename:
        result = os.path.join(result, filename.split('.')[-1])
    return result

Comments

[Dauren S]

а вместо pk id написать?

[sim3x]

а не стоит ли тут использовать os.join.path ?

[Dauren S]

from django.contrib.auth.models import User
import os
from django.db import models
SHORT_TEXT_LEN=1000
# Create your models here.

def get_upload_to(instance, filename):
return u'images/%s/%s' % (instance.id, filename)

class Category(models.Model):
title=models.CharField(max_length=200)

def __str__(self):
return self.title

class Article(models.Model):

title=models.CharField(max_length=200)
description=models.TextField()
fulltext=models.TextField()
created=models.DateTimeField()
image=models.ImageField(upload_to=get_upload_to)
user=models.ForeignKey(User)
category=models.ForeignKey(Category)

def __str__(self):
return self.title

def get_short_text(self):
if len(self.description)>SHORT_TEXT_LEN:
return self.description[:SHORT_TEXT_LEN]
else:
return self.description

[Dauren S]

правильно?потому что так не работает..

[Roman K]

sim3x: Да, верно. Лучше использовать абсолютные пути. Хотя, обычно рабочей директорией выбирается именно папка проекта.

[Roman K]

dauren101: Тебе лучше прочитать, что такое pk =)

[Dauren S]

Roman Kitaev: моя код модели верен?

[Dauren S]

Roman Kitaev: да я знаю что такое pk, просто спрашиваю id вставлять? вместо pk

[Roman K]

dauren101: На вид - да.

[Dauren S]

Roman Kitaev: при таком коде файлы записываются в папку None

[Roman K]

dauren101: Если бы знал - не спрашивал бы. Дважды. None потому что у тебя новый объект (не сохранённый) и у него пока нет id.

[Dauren S]

Roman Kitaev: Ты сказал что код на вид верен. Тогда что я упустил?Что еще нужно дописать или исправить

[Roman K]

dauren101: Может быть, за тебя всё приложение написать? Думай головой.

[Dauren S]

Roman Kitaev: передать параметр self в метод?

[sim3x]

Roman Kitaev: не
абсолютные пути зло
я про вот такой вариант

def get_image_path(instance, filename):
    new_file_name = uuid1().hex
    if '.' in filename:
        new_file_name = '{}.{}'.format(uuid1().hex, filename.split('.')[-1])
    return os.path.join('images', instance.pk, new_file_name)

[Roman K]

sim3x: В пределах линукса, конечно, это так и отрендерится. Но да, если кто-то вздумает разрабатывать на поделии майкрософта - так будет лучше.
Кстати, скажи, как с тобой связаться? Назрел вопрос один.

[Dauren S]

Roman Kitaev: не получается что то

[Roman K]

dauren101: None потому что у тебя новый объект (не сохранённый) и у него пока нет id.

[Dauren S]

Roman Kitaev: А как сохранить?

[Roman K]

dauren101: Иди-ка, читай документацию.

[Alexander]

загружать картинку ПОСЛЕ сохранения объекта, когда уже появится этот самый ID

[Mr_Floppy]

stackoverflow.com/a/15776267/1266031