La nota de otra persona

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

La lista oculta las notas de otras personas, pero las páginas de notas no. Las notas están numeradas como /1/, /2/, /3/, así que cualquiera puede probar con el siguiente número y leerla. No basta con ocultar los enlaces de una lista: cada página tiene que comprobarlo.

get_object_or_404() acepta los mismos filtros que filter(), tantos como quieras, y encuentra una fila solo cuando todos coinciden. La página de una tienda muestra únicamente los pedidos que siguen abiertos:

order = get_object_or_404(Order, pk=pk, status="open")

Un pedido cerrado responde allí con 404 Página no encontrada, igual que un número de pedido que no existe. En el caso de una fila privada, eso es intencional: un error 403 Forbidden le revelaría a un desconocido que la fila existe.

Order.objects.get() generaría en cambio DoesNotExist, y el visitante recibiría un error 500 del servidor: un fallo.

Desafío

Fácil

En note_detail, muestra una nota solo a su propietario. Si la nota es de otra persona o el número no existe, la página responde con 404.

Inicia sesión como bob (contraseña coddy-pass-123) y abre /1/, una de las notas de ada: página 404 no encontrada.

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)
    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})