Content for session 9. Tidy up URLs and Views.

wsot · wsot · commit db72024d5464 · 2022-09-19T12:10:49.000+10:00
diff --git a/src/community_db/templates/community_db/person_detail_in_base.html b/src/community_db/templates/community_db/person_detail_in_base.html
@@ -0,0 +1,14 @@
+{% extends "base.html" %}
+
+{% block content %}
+    <br>
+    <a href="{% url 'fbv-person-list' %}">Back to list</a>
+    <br>
+    This is the detail of a person:
+    <ul>
+        <li>First Name: {{ object.first_name }}</li>
+        <li>Last Name: {{ object.last_name }}</li>
+        <li>Country: {{ object.country }}</li>
+        <li>Phone number: {{ object.mobile_number }}</li>
+    </ul>
+{% endblock %}
diff --git a/src/community_db/templates/community_db/person_list_in_base.html b/src/community_db/templates/community_db/person_list_in_base.html
@@ -4,8 +4,11 @@
     This is my list of folks
     <ul>
         {% for person in object_list %}
-        <li>{{ person.first_name }} {{ person.last_name }}
-            from {{ person.country }}</li>
+            <li>
+                <a href="{% url 'fbv-person-detail' person.id %}">
+                    {{ person.first_name }} {{ person.last_name }}
+                </a> from {{ person.country }}
+            </li>
         {% endfor %}
     </ul>
 {% endblock %}
diff --git a/src/community_db/views.py b/src/community_db/views.py
@@ -1,41 +1,29 @@
-from django.http import HttpResponse
 from django.shortcuts import render
-from django.template import loader
-from django.views.generic.list import ListView
+from django.views.generic import DetailView, ListView
 
-from community_db.models import Person
+from .models import Person
 
-
-def list_persons(request):
-    html = "<html><body>This is my list of folks<ul>"
-
-    for person in Person.objects.all():
-        html += f"<li>{person.first_name} {person.last_name} from {person.country}</li>"
-
-    html += "</ul></body></html>"
-    return HttpResponse(html)
-
-
-# def list_persons_with_template(request):
-#     persons = Person.objects.all()
-#     template = loader.get_template("community_db/person_list.html")
-#     context = {"object_list": persons}
-#     return HttpResponse(template.render(context, request))
+# FUNCTION BASED VIEWS
 
 
 def list_persons_with_template(request):
     persons = Person.objects.all()
-    template = loader.get_template("community_db/person_list_in_base.html")
     context = {"object_list": persons}
-    return HttpResponse(template.render(context, request))
+    return render(request, "community_db/person_list_in_base.html", context)
 
 
-# def list_persons_with_template(request):
-#     persons = Person.objects.all()
-#     context = {"object_list": persons}
-#     return render(request, "community_db/person_list.html", context)
+def detail_person_with_template(request, pk):
+    person = Person.objects.get(id=pk)
+    context = {"object": person}
+    return render(request, "community_db/person_detail_in_base.html", context)
 
 
+# CLASS BASED VIEWS
 class PersonListView(ListView):
     model = Person
     template_name = "community_db/person_list_in_base.html"
+
+
+class PersonDetailView(DetailView):
+    model = Person
+    template_name = "community_db/person_detail_in_base.html"
diff --git a/src/pacificconnect/urls.py b/src/pacificconnect/urls.py
@@ -20,7 +20,16 @@
 
 urlpatterns = [
     path("admin/", admin.site.urls),
-    path("myview", views.list_persons),
-    path("myview-with-template/", views.list_persons_with_template),
-    path("person-list/", views.PersonListView.as_view()),
+    path("fbv/people/", views.list_persons_with_template, name="fbv-person-list"),
+    path(
+        "fbv/people/<int:pk>/",
+        views.detail_person_with_template,
+        name="fbv-person-detail",
+    ),
+    path("cbv/people/", views.PersonListView.as_view(), name="cbv-person-list"),
+    path(
+        "cbv/people/<int:pk>/",
+        views.PersonDetailView.as_view(),
+        name="cbv-person-detail",
+    ),
 ]
