Convalida tramite DeleteView prima dell'eliminazione dell'istanza

Question

Qual è l'approccio migliore per la gestione dell'eliminazione di un oggetto con qualche convalida prima che l'oggetto venga eliminato? Ad esempio, nella mia configurazione sono disponibili due modelli: Game e Team (che sono ovviamente correlati). Gli utenti dovrebbero essere in grado di eliminare solo squadre che NON sono legate a nessun gioco.

Ho creato un modulo (senza campi) per l'eliminazione di una squadra ...

class TeamDeleteForm(ModelForm): 
    class Meta: 
     model = Team 
     fields = [] 

    def clean(self): 
     # Check to see if this team is tied to any existing games 
     if self.instance.gameteams_set.exists(): 
      raise ValidationError("This team is tied to 1 or more games") 
     return super().clean() 

ma poi ho capito che la base di classe di visualizzazione DeleteView non ha alcun tipo di metodo form_valid(). Dovrei estendere il FormView generico invece di DeleteView o c'è un approccio migliore che mi manca?

Answer 1

Penso che l'approccio migliore sostituirà il metodo di eliminazione del modello.Ad esempio:

class Team(models.Model): 
    ... 
    def delete(self, *args, **kwargs): 
     if Game.objects.filter(team__pk= self.pk).exists(): 
      raise Exception('This team is related to a game.') 
     super().delete(*args, **kwargs) 

Answer 2

Per il tuo caso particolare, dovrei semplicemente ignorare l'attributo queryset della vista per escludere Team s con Game s associato.

class TeamDeleteView(DeleteView): 
    queryset = Team.objects.distinct().exclude(games__isnull=False) 

C'è un Django ticket opened to make the DeleteView behave like other form views ma fino the proposed patch si fonde e rilasciati (Non farà in 1.8) dovrete ignorare completamente il metodo della visualizzazione come la seguente delete:

class TeamDeleteView(DeleteView): 
    model = Team 

    def delete(request, *args, **kwargs): 
     self.object = self.get_object() 
     if self.object.gameteams_set.exists(): 
      # Return the appropriate response 
     success_url = self.get_success_url() 
     self.object.delete() 
     return HttpResponseRedirect(success_url) 

Modifica:

Dalla soluzione accettata sembra che si stia tentando di impedire la cancellazione a livello di modello. Tale applicazione dovrebbe essere eseguita utilizzando un gestore PROTECTon_delete.

from django.db import models 

class Team(models.Model): 
    pass 

class Game(models.Model): 
    team = models.ForeignKey(Team, on_delete=models.PROTECT) 

Avrai ancora a che fare con la sollevato ProtectedError nella vista:

from django.db import models 
from django.http.response import HttpResponseForbidden 

class TeamDeleteView(DeleteView): 
    model = Team 

    def delete(request, *args, **kwargs): 
     try: 
      return super(TeamDeleteView, self).delete(
       request, *args, **kwargs 
      ) 
     except models.ProtectedError as e: 
      # Return the appropriate response 
      return HttpResponseForbidden(
       "This team is tied to 1 or more games" 
      ) 

si potrebbe anche utilizzare la proprietà protected_objects di e per visualizzare un messaggio di errore più significativo, proprio come l'amministratore fa .

Answer 3

Ho usato sia DeleteView che FormView per questo scenario. Entrambi hanno i loro pro e contro.

DeleteView è bello perché basato su SingleObjectMixin e consente di accedere facilmente all'oggetto che si desidera eliminare. Un buon modo per farlo è sollevare un'eccezione in get_object. Questo rende così possibile sollevare un'eccezione sia su get che su post.

def get_object(self, qs): 
    obj = super(FooView, self).get_object(qs) 
    if obj.can_delete(): 
    return obj 
    raise PermissionDenied 

FormView è bello perché è possibile sfruttare form_invalid ei metodi di pulizia, ma poi si devono ancora fare il lavoro per ottenere l'oggetto, messa a punto una sorta di forma (non necessaria in deleteview).

È davvero una questione di come si vuole affrontarlo. Alcune altre domande sono: sollevate un'eccezione su GET o volete mostrare una bella pagina per far sapere all'utente che non possono cancellare l'oggetto. Questo può essere fatto in entrambi i tipi di vista.

Aggiorna la tua domanda se hai più punti da approfondire e aggiornerò la mia risposta.