Eliminare un'attività

Fa parte della sezione API JSON del percorso Django di Coddy. Lezione 10 di 19.

Un'attività completata dovrebbe sparire, non solo essere contrassegnata: DELETE /api/tasks/1/ rimuove l'attività 1 e poi non c'è più nulla da inviare.

Per questo HTTP prevede 204 No Content: la richiesta è andata a buon fine e la risposta non ha un corpo. Una JsonResponse scrive sempre un corpo, anche {}. Un semplice HttpResponse con solo uno stato non invia altro:

return HttpResponse(status=204)

Mantieni per primo il controllo 404, così l'eliminazione di un'attività inesistente risponde comunque con 404.

Un URL, tre metodi: GET legge l'attività, PATCH la modifica, DELETE la rimuove. Un URL per risorsa, con il metodo come verbo: è ciò che si chiama API REST.

Sfida

Facile

Consenti ai programmi di eliminare un'attività con DELETE /api/tasks/<id>/:

Consenti "DELETE" su task_detail.
Elimina l'attività e rispondi con 204 e un corpo vuoto.
Se l'attività non esiste, rispondi comunque con 404.

Premi Run, poi nella scheda Shell: from django.test import Client; print(Client().delete("/api/tasks/1/").status_code) stampa 204.

Provalo tu

import json

from django.http import JsonResponse
from django.shortcuts import render
from django.views.decorators.csrf import csrf_exempt
from django.views.decorators.http import require_GET, require_http_methods

from .models import Task


def task_to_dict(task):
    return {
        "id": task.id,
        "title": task.title,
        "done": task.done,
        "priority": task.priority,
    }


def home(request):
    return render(request, "tasks/home.html", {"tasks": Task.objects.all()})


@require_GET
def stats(request):
    data = {
        "total": Task.objects.count(),
        "done": Task.objects.filter(done=True).count(),
    }
    return JsonResponse(data)


@csrf_exempt
@require_http_methods(["GET", "POST"])
def task_list(request):
    if request.method == "POST":
        try:
            data = json.loads(request.body)
        except json.JSONDecodeError:
            return JsonResponse({"error": "The body is not valid JSON."}, status=400)
        errors = {}
        title = data.get("title", "").strip()
        if not title:
            errors["title"] = "This field is required."
        if errors:
            return JsonResponse({"errors": errors}, status=400)
        task = Task.objects.create(
            title=title,
            priority=data.get("priority", "normal"),
        )
        return JsonResponse(task_to_dict(task), status=201)
    tasks = Task.objects.all()
    done = request.GET.get("done")
    if done == "true":
        tasks = tasks.filter(done=True)
    elif done == "false":
        tasks = tasks.filter(done=False)
    return JsonResponse([task_to_dict(task) for task in tasks], safe=False)


@csrf_exempt
@require_http_methods(["GET", "PATCH"])
def task_detail(request, pk):
    task = Task.objects.filter(pk=pk).first()
    if task is None:
        return JsonResponse({"error": "Task not found"}, status=404)
    if request.method == "PATCH":
        data = json.loads(request.body)
        if "title" in data:
            task.title = data["title"]
        if "done" in data:
            task.done = data["done"]
        task.save()
    return JsonResponse(task_to_dict(task))