Solo el propietario puede editar

Parte de la sección Usuarios y cuentas del Journey de Django de Coddy. Lección 11 de 16.

La página de una nota ahora es segura, pero sus páginas de edición y eliminación no lo son: /3/edit/ y /3/delete/ siguen abriendo cualquier nota. Peor aún, ni siquiera hace falta abrirlas. Un formulario es una solicitud POST, y cualquiera puede enviar una solicitud POST a /3/delete/ sin ver nunca la página.

Así que la regla se aplica a cada vista, no a cada página a la que enlazas: cada vista que carga una fila por su número decide allí quién puede cargarla. Fíjate dónde está la búsqueda en una vista que gestiona tanto GET como POST:

def ticket_close(request, pk):
    ticket = ...  # la búsqueda: se ejecuta para cada solicitud
    if request.method == "POST":
        ticket.closed = True
        ticket.save()
        return redirect("tickets:list")
    return render(request, "tickets/close.html", {"ticket": ticket})

La primera línea se ejecuta antes de la rama if request.method == "POST". Una búsqueda ahí que rechace a los desconocidos les responde con un error 404 tanto para GET como para POST, y no se cambia nada.

Desafío

Fácil

En notes/views.py, haz que note_edit y note_delete rechacen las notas que no sean del usuario. La nota de otra persona devuelve 404 allí, tanto para GET como para POST, y permanece sin cambios. Tus propias notas se siguen editando y eliminando.

Pulsa Run, inicia sesión como bob (contraseña coddy-pass-123) y abre /1/edit/.

Pruébalo tú mismo

from django.contrib.auth import login
from django.contrib.auth.decorators import login_not_required
from django.contrib.auth.forms import UserCreationForm
from django.shortcuts import get_object_or_404, redirect, render

from .forms import NoteForm
from .models import Note


def note_list(request):
    notes = Note.objects.filter(owner=request.user)
    return render(request, "notes/note_list.html", {"notes": notes})


def note_detail(request, pk):
    note = get_object_or_404(Note, pk=pk, owner=request.user)
    return render(request, "notes/note_detail.html", {"note": note})


def note_create(request):
    if request.method == "POST":
        form = NoteForm(request.POST)
        if form.is_valid():
            note = form.save(commit=False)
            note.owner = request.user
            note.save()
            return redirect("notes:detail", pk=note.pk)
    else:
        form = NoteForm()
    return render(request, "notes/note_form.html", {"form": form})


def note_edit(request, pk):
    note = get_object_or_404(Note, pk=pk)
    if request.method == "POST":
        form = NoteForm(request.POST, instance=note)
        if form.is_valid():
            form.save()
            return redirect("notes:detail", pk=note.pk)
    else:
        form = NoteForm(instance=note)
    return render(request, "notes/note_form.html", {"form": form, "note": note})


def note_delete(request, pk):
    note = get_object_or_404(Note, pk=pk)
    if request.method == "POST":
        note.delete()
        return redirect("notes:list")
    return render(request, "notes/note_confirm_delete.html", {"note": note})


@login_not_required
def signup(request):
    if request.method == "POST":
        form = UserCreationForm(request.POST)
        if form.is_valid():
            user = form.save()
            login(request, user)
            return redirect("notes:list")
    else:
        form = UserCreationForm()
    return render(request, "notes/signup.html", {"form": form})