Сохранение старого файла при редактировании объекта и не загрузка нового в форму Django

Question

У меня есть класс Player, который использует ForeignKey для объекта Photo для присоединения фотографии. Моя проблема в том, что когда я редактирую игрока, у которого есть изображение, и я не загружаю новый, старый файл снимается с объекта Player. Должно быть, что-то не так с моей логикой.

class Player( models.Model ):
    ...
    mugshot = models.ForeignKey( 'content.Photo', null=True, blank=True )
    ...

Мой класс PlayerForm выглядит так:

class PlayerForm( forms.ModelForm ):
    ...
    mugshot = forms.FileField( widget=forms.FileInput(), required=False )
    ...

    def save(self, commit=True, request=None, user=None):
        player = super( PlayerForm, self ).save( commit=False )

        if commit:
            player.save()

        return player

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

def edit_player( request, team_season_id, player_id ):
    team_season = get_object_or_404( TeamSeasonConference, pk=team_season_id )
    player = get_object_or_404( Player, pk=player_id )

    if request.method == 'POST' and 'cancel' not in request.POST:
        player_form = PlayerForm( request.POST, instance=player )

        if player_form.is_valid():
            updated_player = player_form.save(True)

            if request.FILES: # if sent a file
                file_data = request.FILES['mugshot']
                picture = Photo.objects.create( # create a photo object with that file
                name = player_form.cleaned_data['first_name'] + ' ' + player_form.cleaned_data['last_name'], # and give it the name of the player you're adding
                cutline = '',
                file = file_data,
                source = '',
            )
        else:
            picture = player.mugshot; # keep the old one

        updated_player.mugshot = picture
        updated_player.save()

Answer 1

Небольшая доработка должна решить проблему:

def edit_player( request, team_season_id, player_id ):
team_season = get_object_or_404( TeamSeasonConference, pk=team_season_id )
player = get_object_or_404( Player, pk=player_id )

if request.method == 'POST' and 'cancel' not in request.POST:
    player_form = PlayerForm( request.POST, instance=player )

    if player_form.is_valid():
        updated_player = player_form.save(True)

        if request.FILES: # if sent a file
            file_data = request.FILES['mugshot']
            picture = Photo.objects.create( # create a photo object with that file
                name = player_form.cleaned_data['first_name'] + ' ' + player_form.cleaned_data['last_name'], # and give it the name of the player you're adding
                cutline = '',
                file = file_data,
                source = '',
            )

            # You only need to save again if there is a picture
            updated_player.mugshot = picture
            updated_player.save()

Кроме того, ваша форма должна исключить существующую фотографию:

class PlayerForm( forms.ModelForm ):

    class Meta:
        model = Player
        exclude = ('mugshot',)

    def save(self, ...):
        #as it was before