From 443dfd4f8e1295d423d467c8a4e52ad7adb9d6a6 Mon Sep 17 00:00:00 2001
From: Lacey Williams Henschel <laceynwilliams@gmail.com>
Date: Fri, 6 Jan 2023 10:11:54 -0800
Subject: [PATCH 1/4] :sparkles: Add form to select library category

---
 libraries/forms.py            | 8 ++++++++
 libraries/tests/test_forms.py | 6 ++++++
 2 files changed, 14 insertions(+)
 create mode 100644 libraries/forms.py
 create mode 100644 libraries/tests/test_forms.py

diff --git a/libraries/forms.py b/libraries/forms.py
new file mode 100644
index 00000000..fa78bb41
--- /dev/null
+++ b/libraries/forms.py
@@ -0,0 +1,8 @@
+from django.forms import ModelForm
+from .models import Library, Category
+
+
+class LibraryForm(ModelForm):
+    class Meta:
+        model = Library
+        fields = ["categories"]
diff --git a/libraries/tests/test_forms.py b/libraries/tests/test_forms.py
new file mode 100644
index 00000000..96b2759e
--- /dev/null
+++ b/libraries/tests/test_forms.py
@@ -0,0 +1,6 @@
+from ..forms import LibraryForm
+
+
+def test_library_form_success(tp, library, category):
+    form = LibraryForm(data={"categories": [category]})
+    assert form.is_valid() is True

From 0c8552a45bbb2cdea48b91fe0da32b6a2ea6e5c6 Mon Sep 17 00:00:00 2001
From: Lacey Williams Henschel <laceynwilliams@gmail.com>
Date: Fri, 6 Jan 2023 10:13:20 -0800
Subject: [PATCH 2/4] :sparkles: :umbrella: User is able to filter libraries by
 category in the UI

---
 libraries/tests/fixtures.py   |  5 +++++
 libraries/tests/test_views.py | 17 +++++++++++++++++
 libraries/views.py            | 22 ++++++++++++++++++++--
 templates/libraries/list.html | 23 +++++++++++++++--------
 4 files changed, 57 insertions(+), 10 deletions(-)

diff --git a/libraries/tests/fixtures.py b/libraries/tests/fixtures.py
index a8a85bc2..7d0f01d2 100644
--- a/libraries/tests/fixtures.py
+++ b/libraries/tests/fixtures.py
@@ -3,6 +3,11 @@ from fastcore.xtras import dict2obj
 from model_bakery import baker
 
 
+@pytest.fixture
+def category(db):
+    return baker.make("libraries.Category", name="Math", slug="math")
+
+
 @pytest.fixture
 def library(db):
     return baker.make(
diff --git a/libraries/tests/test_views.py b/libraries/tests/test_views.py
index 396f5ba1..6a25eb34 100644
--- a/libraries/tests/test_views.py
+++ b/libraries/tests/test_views.py
@@ -7,6 +7,23 @@ def test_library_list(library, tp):
     tp.response_200(res)
 
 
+def test_library_list_select_category(library, category, tp):
+    """POST /libraries/ to submit a category redirects to the libraries-by-category page"""
+    res = tp.post("libraries", data={"categories": category.pk})
+    tp.response_302(res)
+
+
+def test_libraries_by_category(tp, library, category):
+    """GET /libraries-by-category/{slug}/"""
+    baker.make("libraries.Library", name="Sample")
+    library.categories.add(category)
+    res = tp.get("libraries-by-category", category.slug)
+    tp.response_200(res)
+    assert "library_list" in res.context
+    assert len(res.context["library_list"]) == 1
+    assert library in res.context["library_list"]
+
+
 def test_library_detail(library, tp):
     """GET /libraries/{repo}/"""
     url = tp.reverse("library-detail", library.slug)
diff --git a/libraries/views.py b/libraries/views.py
index 4be02f30..d913d80c 100644
--- a/libraries/views.py
+++ b/libraries/views.py
@@ -1,7 +1,14 @@
-from django.views.generic import DetailView, ListView
+import structlog
 
+from django.shortcuts import redirect
+from django.views.generic import DetailView, ListView
+from django.views.generic.edit import FormMixin
+
+from .forms import LibraryForm
 from .models import Category, Issue, Library, PullRequest
 
+logger = structlog.get_logger()
+
 
 class CategoryMixin:
     def get_context_data(self, **kwargs):
@@ -10,15 +17,26 @@ class CategoryMixin:
         return context
 
 
-class LibraryList(CategoryMixin, ListView):
+class LibraryList(CategoryMixin, FormMixin, ListView):
     """List all of our libraries by name"""
 
+    form_class = LibraryForm
     paginate_by = 25
     queryset = (
         Library.objects.prefetch_related("authors", "categories").all().order_by("name")
     )
     template_name = "libraries/list.html"
 
+    def post(self, request):
+        """User has submitted a form and will be redirected to the right results"""
+        form = self.get_form()
+        if form.is_valid():
+            category = form.cleaned_data["categories"][0]
+            return redirect("libraries-by-category", category=category.slug)
+        else:
+            logger.info("library_list_invalid_category")
+        return super().get(request)
+
 
 class LibraryByLetter(CategoryMixin, ListView):
     """List all of our libraries that begin with a certain letter"""
diff --git a/templates/libraries/list.html b/templates/libraries/list.html
index 3d3fd3a8..166ba583 100644
--- a/templates/libraries/list.html
+++ b/templates/libraries/list.html
@@ -43,14 +43,21 @@
         </span>
         <input type="search" name="q" class="w-full md:w-1/3 text-sm text-white bg-charcoal focus:text-charcoal text-orange px-3 py-2 rounded-md" type="text" value="" placeholder="Search Library" />
       </div>
-      <div>
-        <select class="mb-3 md:mb-0 w-full md:w-auto cursor-pointer block sm:inline-block text-sm uppercase rounded-md bg-black text-orange border border-slate pl-5 pr-11 py-3 mr-3">
-          <option>Filter by category</option>
-          {% for c in categories %}
-          <option value="{{c.name}}">{{ c.name }}</option>
-          {% endfor %}
-        </select>
-      </div>
+
+      <!-- Form to select a category -->
+      <form action="/libraries/" method="post">
+        {% csrf_token %}
+        <div>
+          <label for="id_categories">Categories:</label>
+          <select name="categories" class="mb-3 md:mb-0 w-full md:w-auto cursor-pointer block sm:inline-block text-sm uppercase rounded-md bg-black text-orange border border-slate pl-5 pr-11 py-3 mr-3" id="id_categories">
+            <option>Filter by category</option>
+            {% for c in categories %}
+              <option value="{{ c.pk }}">{{ c.name }}</option>
+            {% endfor %}
+          </select>
+        </div>
+        <input type="submit" value="Submit">
+      </form>
     </div>
   </div>
 

From 5046b7e8a26c77edce7926bc8330ed489e29d8fd Mon Sep 17 00:00:00 2001
From: Lacey Williams Henschel <laceynwilliams@gmail.com>
Date: Fri, 6 Jan 2023 10:15:23 -0800
Subject: [PATCH 3/4] :pencil: Hide label

---
 templates/libraries/list.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/templates/libraries/list.html b/templates/libraries/list.html
index 166ba583..c8217d3c 100644
--- a/templates/libraries/list.html
+++ b/templates/libraries/list.html
@@ -48,7 +48,7 @@
       <form action="/libraries/" method="post">
         {% csrf_token %}
         <div>
-          <label for="id_categories">Categories:</label>
+          <label for="id_categories" hidden="true">Categories:</label>
           <select name="categories" class="mb-3 md:mb-0 w-full md:w-auto cursor-pointer block sm:inline-block text-sm uppercase rounded-md bg-black text-orange border border-slate pl-5 pr-11 py-3 mr-3" id="id_categories">
             <option>Filter by category</option>
             {% for c in categories %}

From 8f1c675ea479c96f48c03e6e40c6723fb293d8d3 Mon Sep 17 00:00:00 2001
From: Lacey Williams Henschel <laceynwilliams@gmail.com>
Date: Mon, 9 Jan 2023 14:34:51 -0800
Subject: [PATCH 4/4] :sparkles: Add category to breadcrumb, preselect category

---
 libraries/tests/test_views.py |  2 ++
 libraries/views.py            | 11 +++++++++++
 templates/libraries/list.html |  6 +++---
 3 files changed, 16 insertions(+), 3 deletions(-)

diff --git a/libraries/tests/test_views.py b/libraries/tests/test_views.py
index 6a25eb34..bcc37d13 100644
--- a/libraries/tests/test_views.py
+++ b/libraries/tests/test_views.py
@@ -22,6 +22,8 @@ def test_libraries_by_category(tp, library, category):
     assert "library_list" in res.context
     assert len(res.context["library_list"]) == 1
     assert library in res.context["library_list"]
+    assert "category" in res.context
+    assert res.context["category"] == category
 
 
 def test_library_detail(library, tp):
diff --git a/libraries/views.py b/libraries/views.py
index d913d80c..f9bdc430 100644
--- a/libraries/views.py
+++ b/libraries/views.py
@@ -60,6 +60,17 @@ class LibraryByCategory(CategoryMixin, ListView):
     paginate_by = 25
     template_name = "libraries/list.html"
 
+    def get_context_data(self, **kwargs):
+        context = super().get_context_data()
+        category_slug = self.kwargs.get("category")
+        if category_slug:
+            try:
+                category = Category.objects.get(slug=category_slug)
+                context["category"] = category
+            except Category.DoesNotExist:
+                logger.info("libraries_by_category_view_category_not_found")
+        return context
+
     def get_queryset(self):
         category = self.kwargs.get("category")
 
diff --git a/templates/libraries/list.html b/templates/libraries/list.html
index c8217d3c..60d21da8 100644
--- a/templates/libraries/list.html
+++ b/templates/libraries/list.html
@@ -5,7 +5,7 @@
 {% block content %}
   <!-- Breadcrumb used on filtered views -->
   <div class="p-3 md:p-0">
-    <a class="text-orange" href="{% url "libraries" %}">Libraries</a> > Categorized
+    <a class="text-orange" href="{% url "libraries" %}">Libraries</a> > Categorized{% if category %} > <a class="text-orange" href="{% url "libraries-by-category" category.slug %}">{{ category.name }}</a>{% endif %}
   </div>
   <!-- end breadcrumb -->
 
@@ -52,11 +52,11 @@
           <select name="categories" class="mb-3 md:mb-0 w-full md:w-auto cursor-pointer block sm:inline-block text-sm uppercase rounded-md bg-black text-orange border border-slate pl-5 pr-11 py-3 mr-3" id="id_categories">
             <option>Filter by category</option>
             {% for c in categories %}
-              <option value="{{ c.pk }}">{{ c.name }}</option>
+              <option value="{{ c.pk }}" {% if category == c %}selected="selected"{% endif %}>{{ c.name }}</option>
             {% endfor %}
           </select>
         </div>
-        <input type="submit" value="Submit">
+        <input type="submit" value="Submit" style="color: red">
       </form>
     </div>
   </div>