From 443750d6dbf381264b74654ddc64252d5f04d791 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Mon, 4 Oct 2021 09:14:04 -0700
Subject: [PATCH 01/19] f strings for activistreams tasks tests

---
 bookwyrm/tests/activitystreams/test_tasks.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/bookwyrm/tests/activitystreams/test_tasks.py b/bookwyrm/tests/activitystreams/test_tasks.py
index f4c85e1b..1002e4a8 100644
--- a/bookwyrm/tests/activitystreams/test_tasks.py
+++ b/bookwyrm/tests/activitystreams/test_tasks.py
@@ -141,7 +141,7 @@ class Activitystreams(TestCase):
         call_args = mock.call_args
         self.assertEqual(call_args[0][0], status)
         self.assertEqual(
-            call_args[1]["stores"], ["{:d}-home".format(self.another_user.id)]
+            call_args[1]["stores"], [f"{self.another_user.id}-home"]
         )
 
     @patch("bookwyrm.activitystreams.LocalStream.remove_object_from_related_stores")
@@ -164,10 +164,10 @@ class Activitystreams(TestCase):
         call_args = mock.call_args
         self.assertEqual(call_args[0][0], status)
         self.assertTrue(
-            "{:d}-home".format(self.another_user.id) in call_args[1]["stores"]
+            f"{self.another_user.id}-home" in call_args[1]["stores"]
         )
         self.assertTrue(
-            "{:d}-home".format(self.local_user.id) in call_args[1]["stores"]
+            f"{self.local_user.id}-home" in call_args[1]["stores"]
         )
 
     @patch("bookwyrm.activitystreams.LocalStream.remove_object_from_related_stores")
@@ -189,5 +189,5 @@ class Activitystreams(TestCase):
         call_args = mock.call_args
         self.assertEqual(call_args[0][0], status)
         self.assertEqual(
-            call_args[1]["stores"], ["{:d}-home".format(self.local_user.id)]
+            call_args[1]["stores"], [f"{self.local_user.id}-home"]
         )

From 321949f2fac201ebf66b0cd480e151f63f377769 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Mon, 4 Oct 2021 09:47:33 -0700
Subject: [PATCH 02/19] Lightly updates tests

---
 bookwyrm/activitystreams.py                  | 2 ++
 bookwyrm/redis_store.py                      | 2 +-
 bookwyrm/tests/activitystreams/test_tasks.py | 3 ++-
 3 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/bookwyrm/activitystreams.py b/bookwyrm/activitystreams.py
index 1feb495b..f843641f 100644
--- a/bookwyrm/activitystreams.py
+++ b/bookwyrm/activitystreams.py
@@ -480,12 +480,14 @@ def handle_boost_task(boost_id):
     instance = models.Status.objects.get(id=boost_id)
     boosted = instance.boost.boosted_status
 
+    # previous boosts of this status
     old_versions = models.Boost.objects.filter(
         boosted_status__id=boosted.id,
         created_date__lt=instance.created_date,
     )
 
     for stream in streams.values():
+        # people who should see the boost (not people who see the original status)
         audience = stream.get_stores_for_object(instance)
         stream.remove_object_from_related_stores(boosted, stores=audience)
         for status in old_versions:
diff --git a/bookwyrm/redis_store.py b/bookwyrm/redis_store.py
index 521e73b2..78f373a2 100644
--- a/bookwyrm/redis_store.py
+++ b/bookwyrm/redis_store.py
@@ -35,7 +35,7 @@ class RedisStore(ABC):
 
     def remove_object_from_related_stores(self, obj, stores=None):
         """remove an object from all stores"""
-        stores = stores or self.get_stores_for_object(obj)
+        stores = self.get_stores_for_object(obj) if stores is None else stores
         pipeline = r.pipeline()
         for store in stores:
             pipeline.zrem(store, -1, obj.id)
diff --git a/bookwyrm/tests/activitystreams/test_tasks.py b/bookwyrm/tests/activitystreams/test_tasks.py
index 1002e4a8..88704783 100644
--- a/bookwyrm/tests/activitystreams/test_tasks.py
+++ b/bookwyrm/tests/activitystreams/test_tasks.py
@@ -125,7 +125,7 @@ class Activitystreams(TestCase):
     @patch("bookwyrm.activitystreams.BooksStream.remove_object_from_related_stores")
     @patch("bookwyrm.models.activitypub_mixin.broadcast_task.delay")
     def test_boost_to_another_timeline(self, *_):
-        """add a boost and deduplicate the boosted status on the timeline"""
+        """boost from a non-follower doesn't remove original status from feed"""
         status = models.Status.objects.create(user=self.local_user, content="hi")
         with patch("bookwyrm.activitystreams.handle_boost_task.delay"):
             boost = models.Boost.objects.create(
@@ -138,6 +138,7 @@ class Activitystreams(TestCase):
             activitystreams.handle_boost_task(boost.id)
 
         self.assertTrue(mock.called)
+        self.assertEqual(mock.call_count, 1)
         call_args = mock.call_args
         self.assertEqual(call_args[0][0], status)
         self.assertEqual(

From ac70a810af9cf5136baf63525d34b8a607e8b447 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Mon, 4 Oct 2021 09:58:05 -0700
Subject: [PATCH 03/19] Boosts by remote users

---
 bookwyrm/tests/activitystreams/test_tasks.py | 48 +++++++++++++++-----
 1 file changed, 36 insertions(+), 12 deletions(-)

diff --git a/bookwyrm/tests/activitystreams/test_tasks.py b/bookwyrm/tests/activitystreams/test_tasks.py
index 88704783..80b0b771 100644
--- a/bookwyrm/tests/activitystreams/test_tasks.py
+++ b/bookwyrm/tests/activitystreams/test_tasks.py
@@ -22,6 +22,16 @@ class Activitystreams(TestCase):
                 local=True,
                 localname="nutria",
             )
+        with patch("bookwyrm.models.user.set_remote_server.delay"):
+            self.remote_user = models.User.objects.create_user(
+                "rat",
+                "rat@rat.com",
+                "ratword",
+                local=False,
+                remote_id="https://example.com/users/rat",
+                inbox="https://example.com/users/rat/inbox",
+                outbox="https://example.com/users/rat/outbox",
+            )
         work = models.Work.objects.create(title="test work")
         self.book = models.Edition.objects.create(title="test book", parent_work=work)
         with patch("bookwyrm.models.activitypub_mixin.broadcast_task.delay"):
@@ -141,9 +151,29 @@ class Activitystreams(TestCase):
         self.assertEqual(mock.call_count, 1)
         call_args = mock.call_args
         self.assertEqual(call_args[0][0], status)
-        self.assertEqual(
-            call_args[1]["stores"], [f"{self.another_user.id}-home"]
-        )
+        self.assertEqual(call_args[1]["stores"], [f"{self.another_user.id}-home"])
+
+    @patch("bookwyrm.activitystreams.LocalStream.remove_object_from_related_stores")
+    @patch("bookwyrm.activitystreams.BooksStream.remove_object_from_related_stores")
+    @patch("bookwyrm.models.activitypub_mixin.broadcast_task.delay")
+    def test_boost_to_another_timeline_remote(self, *_):
+        """boost from a remote non-follower doesn't remove original status from feed"""
+        status = models.Status.objects.create(user=self.local_user, content="hi")
+        with patch("bookwyrm.activitystreams.handle_boost_task.delay"):
+            boost = models.Boost.objects.create(
+                boosted_status=status,
+                user=self.remote_user,
+            )
+        with patch(
+            "bookwyrm.activitystreams.HomeStream.remove_object_from_related_stores"
+        ) as mock:
+            activitystreams.handle_boost_task(boost.id)
+
+        self.assertTrue(mock.called)
+        self.assertEqual(mock.call_count, 1)
+        call_args = mock.call_args
+        self.assertEqual(call_args[0][0], status)
+        self.assertEqual(call_args[1]["stores"], [])
 
     @patch("bookwyrm.activitystreams.LocalStream.remove_object_from_related_stores")
     @patch("bookwyrm.activitystreams.BooksStream.remove_object_from_related_stores")
@@ -164,12 +194,8 @@ class Activitystreams(TestCase):
         self.assertTrue(mock.called)
         call_args = mock.call_args
         self.assertEqual(call_args[0][0], status)
-        self.assertTrue(
-            f"{self.another_user.id}-home" in call_args[1]["stores"]
-        )
-        self.assertTrue(
-            f"{self.local_user.id}-home" in call_args[1]["stores"]
-        )
+        self.assertTrue(f"{self.another_user.id}-home" in call_args[1]["stores"])
+        self.assertTrue(f"{self.local_user.id}-home" in call_args[1]["stores"])
 
     @patch("bookwyrm.activitystreams.LocalStream.remove_object_from_related_stores")
     @patch("bookwyrm.activitystreams.BooksStream.remove_object_from_related_stores")
@@ -189,6 +215,4 @@ class Activitystreams(TestCase):
         self.assertTrue(mock.called)
         call_args = mock.call_args
         self.assertEqual(call_args[0][0], status)
-        self.assertEqual(
-            call_args[1]["stores"], [f"{self.local_user.id}-home"]
-        )
+        self.assertEqual(call_args[1]["stores"], [f"{self.local_user.id}-home"])

From 2a88753e17cf094e6c54f09b39fbc1ef0e1dee3d Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Mon, 4 Oct 2021 10:40:39 -0700
Subject: [PATCH 04/19] Chart class

---
 bookwyrm/views/admin/dashboard.py | 82 +++++++++++++++++++------------
 1 file changed, 51 insertions(+), 31 deletions(-)

diff --git a/bookwyrm/views/admin/dashboard.py b/bookwyrm/views/admin/dashboard.py
index e02b9143..66dd9c90 100644
--- a/bookwyrm/views/admin/dashboard.py
+++ b/bookwyrm/views/admin/dashboard.py
@@ -25,13 +25,6 @@ class Dashboard(View):
         """list of users"""
         interval = int(request.GET.get("days", 1))
         now = timezone.now()
-
-        user_queryset = models.User.objects.filter(local=True)
-        user_stats = {"labels": [], "total": [], "active": []}
-
-        status_queryset = models.Status.objects.filter(user__local=True, deleted=False)
-        status_stats = {"labels": [], "total": []}
-
         start = request.GET.get("start")
         if start:
             start = timezone.make_aware(parse(start))
@@ -42,31 +35,33 @@ class Dashboard(View):
         end = timezone.make_aware(parse(end)) if end else now
         start = start.replace(hour=0, minute=0, second=0)
 
-        interval_start = start
-        interval_end = interval_start + timedelta(days=interval)
-        while interval_start <= end:
-            print(interval_start, interval_end)
-            interval_queryset = user_queryset.filter(
-                Q(is_active=True) | Q(deactivation_date__gt=interval_end),
-                created_date__lte=interval_end,
-            )
-            user_stats["total"].append(interval_queryset.filter().count())
-            user_stats["active"].append(
-                interval_queryset.filter(
-                    last_active_date__gt=interval_end - timedelta(days=31),
+        user_queryset = models.User.objects.filter(local=True)
+        user_chart = Chart(
+            queryset=user_queryset,
+            queries={
+                "total": lambda q, s, e: q.filter(
+                    Q(is_active=True) | Q(deactivation_date__gt=e),
+                    created_date__lte=e,
+                ).count(),
+                "active": lambda q, s, e: q.filter(
+                    Q(is_active=True) | Q(deactivation_date__gt=e),
+                    created_date__lte=e,
+                ).filter(
+                    last_active_date__gt=e - timedelta(days=31),
                 ).count()
-            )
-            user_stats["labels"].append(interval_start.strftime("%b %d"))
+            }
+        )
 
-            status_stats["total"].append(
-                status_queryset.filter(
-                    created_date__gt=interval_start,
-                    created_date__lte=interval_end,
+        status_queryset = models.Status.objects.filter(user__local=True, deleted=False)
+        status_chart = Chart(
+            queryset=status_queryset,
+            queries = {
+                "total": lambda q, s, e: q.filter(
+                    created_date__gt=s,
+                    created_date__lte=e,
                 ).count()
-            )
-            status_stats["labels"].append(interval_start.strftime("%b %d"))
-            interval_start = interval_end
-            interval_end += timedelta(days=interval)
+            }
+        )
 
         data = {
             "start": start.strftime("%Y-%m-%d"),
@@ -82,7 +77,32 @@ class Dashboard(View):
             "invite_requests": models.InviteRequest.objects.filter(
                 ignored=False, invite_sent=False
             ).count(),
-            "user_stats": user_stats,
-            "status_stats": status_stats,
+            "user_stats": user_chart.get_chart(start, end, interval),
+            "status_stats": status_chart.get_chart(start, end, interval),
         }
         return TemplateResponse(request, "settings/dashboard/dashboard.html", data)
+
+
+class Chart:
+    """ Data for a chart """
+
+    def __init__(self, queryset, queries: dict):
+        self.queryset = queryset
+        self.queries = queries
+
+    def get_chart(self, start, end, interval):
+        """ load the data for the chart given a time scale and interval """
+        interval_start = start
+        interval_end = interval_start + timedelta(days=interval)
+
+        chart = {k: [] for k in self.queries.keys()}
+        chart["labels"] = []
+        while interval_start <= end:
+            for (name, query) in self.queries.items():
+                chart[name].append(query(self.queryset, interval_start, interval_end))
+            chart["labels"].append(interval_start.strftime("%b %d"))
+
+            interval_start = interval_end
+            interval_end += timedelta(days=interval)
+
+        return chart

From fa241b08e7f5502a4f988ba6d5614abd90db1e13 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Mon, 4 Oct 2021 10:42:02 -0700
Subject: [PATCH 05/19] Rename charts snippets and add register chart file

---
 .../settings/dashboard/dashboard.html         | 15 ++++++++----
 .../dashboard/registration_chart.html         | 24 +++++++++++++++++++
 ...rd_status_chart.html => status_chart.html} |  0
 ...hboard_user_chart.html => user_chart.html} |  0
 bookwyrm/views/admin/dashboard.py             | 11 +++++++++
 5 files changed, 46 insertions(+), 4 deletions(-)
 create mode 100644 bookwyrm/templates/settings/dashboard/registration_chart.html
 rename bookwyrm/templates/settings/dashboard/{dashboard_status_chart.html => status_chart.html} (100%)
 rename bookwyrm/templates/settings/dashboard/{dashboard_user_chart.html => user_chart.html} (100%)

diff --git a/bookwyrm/templates/settings/dashboard/dashboard.html b/bookwyrm/templates/settings/dashboard/dashboard.html
index fbf3ff72..863df40d 100644
--- a/bookwyrm/templates/settings/dashboard/dashboard.html
+++ b/bookwyrm/templates/settings/dashboard/dashboard.html
@@ -95,9 +95,9 @@
         </div>
     </form>
 
-    <div class="columns">
+    <div class="columns is-multiline">
         <div class="column">
-            <h3>{% trans "User signup activity" %}</h3>
+            <h3>{% trans "Total users" %}</h3>
             <div class="box">
                 <canvas id="user_stats"></canvas>
             </div>
@@ -108,6 +108,12 @@
                 <canvas id="status_stats"></canvas>
             </div>
         </div>
+        <div class="column">
+            <h3>{% trans "User signup activity" %}</h3>
+            <div class="box">
+                <canvas id="user_stats"></canvas>
+            </div>
+        </div>
     </div>
 </div>
 
@@ -115,6 +121,7 @@
 
 {% block scripts %}
 <script src="https://cdn.jsdelivr.net/npm/chart.js@3.5.1/dist/chart.min.js"></script>
-{% include 'settings/dashboard/dashboard_user_chart.html' %}
-{% include 'settings/dashboard/dashboard_status_chart.html' %}
+{% include 'settings/dashboard/user_chart.html' %}
+{% include 'settings/dashboard/status_chart.html' %}
+{% include 'settings/dashboard/registration_chart.html' %}
 {% endblock %}
diff --git a/bookwyrm/templates/settings/dashboard/registration_chart.html b/bookwyrm/templates/settings/dashboard/registration_chart.html
new file mode 100644
index 00000000..eec909ed
--- /dev/null
+++ b/bookwyrm/templates/settings/dashboard/registration_chart.html
@@ -0,0 +1,24 @@
+{% load i18n %}
+<script>
+const labels = [{% for label in registration_stats.labels %}"{{ label }}",{% endfor %}];
+const data = {
+    labels: labels,
+    datasets: [{
+        label: '{% trans "Active this month" %}',
+        backgroundColor: 'rgb(75, 192, 192)',
+        borderColor: 'rgb(75, 192, 192)',
+        data: {{ register_stats.total }},
+    }]
+};
+
+const config = {
+    type: 'line',
+    data: data,
+    options: {}
+};
+
+var userStats = new Chart(
+    document.getElementById('user_stats'),
+    config
+);
+</script>
diff --git a/bookwyrm/templates/settings/dashboard/dashboard_status_chart.html b/bookwyrm/templates/settings/dashboard/status_chart.html
similarity index 100%
rename from bookwyrm/templates/settings/dashboard/dashboard_status_chart.html
rename to bookwyrm/templates/settings/dashboard/status_chart.html
diff --git a/bookwyrm/templates/settings/dashboard/dashboard_user_chart.html b/bookwyrm/templates/settings/dashboard/user_chart.html
similarity index 100%
rename from bookwyrm/templates/settings/dashboard/dashboard_user_chart.html
rename to bookwyrm/templates/settings/dashboard/user_chart.html
diff --git a/bookwyrm/views/admin/dashboard.py b/bookwyrm/views/admin/dashboard.py
index 66dd9c90..27edc343 100644
--- a/bookwyrm/views/admin/dashboard.py
+++ b/bookwyrm/views/admin/dashboard.py
@@ -63,6 +63,16 @@ class Dashboard(View):
             }
         )
 
+        register_chart = Chart(
+            queryset=user_queryset,
+            queries = {
+                "total": lambda q, s, e: q.filter(
+                    created_date__gt=s,
+                    created_date__lte=e,
+                ).count()
+            }
+        )
+
         data = {
             "start": start.strftime("%Y-%m-%d"),
             "end": end.strftime("%Y-%m-%d"),
@@ -79,6 +89,7 @@ class Dashboard(View):
             ).count(),
             "user_stats": user_chart.get_chart(start, end, interval),
             "status_stats": status_chart.get_chart(start, end, interval),
+            "register_stats": register_chart.get_chart(start, end, interval),
         }
         return TemplateResponse(request, "settings/dashboard/dashboard.html", data)
 

From 37cd7e684cb8b4f9d0ba4256043a15932b9315a7 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Mon, 4 Oct 2021 10:53:22 -0700
Subject: [PATCH 06/19] Updates chart markup

---
 .../settings/dashboard/dashboard.html         |  8 ++---
 .../dashboard/registration_chart.html         | 35 ++++++++-----------
 .../settings/dashboard/status_chart.html      | 31 +++++++---------
 3 files changed, 32 insertions(+), 42 deletions(-)

diff --git a/bookwyrm/templates/settings/dashboard/dashboard.html b/bookwyrm/templates/settings/dashboard/dashboard.html
index 863df40d..cd0ddc84 100644
--- a/bookwyrm/templates/settings/dashboard/dashboard.html
+++ b/bookwyrm/templates/settings/dashboard/dashboard.html
@@ -96,22 +96,22 @@
     </form>
 
     <div class="columns is-multiline">
-        <div class="column">
+        <div class="column is-half">
             <h3>{% trans "Total users" %}</h3>
             <div class="box">
                 <canvas id="user_stats"></canvas>
             </div>
         </div>
-        <div class="column">
+        <div class="column is-half">
             <h3>{% trans "Status activity" %}</h3>
             <div class="box">
                 <canvas id="status_stats"></canvas>
             </div>
         </div>
-        <div class="column">
+        <div class="column is-half">
             <h3>{% trans "User signup activity" %}</h3>
             <div class="box">
-                <canvas id="user_stats"></canvas>
+                <canvas id="register_stats"></canvas>
             </div>
         </div>
     </div>
diff --git a/bookwyrm/templates/settings/dashboard/registration_chart.html b/bookwyrm/templates/settings/dashboard/registration_chart.html
index eec909ed..e98a0f18 100644
--- a/bookwyrm/templates/settings/dashboard/registration_chart.html
+++ b/bookwyrm/templates/settings/dashboard/registration_chart.html
@@ -1,24 +1,19 @@
 {% load i18n %}
 <script>
-const labels = [{% for label in registration_stats.labels %}"{{ label }}",{% endfor %}];
-const data = {
-    labels: labels,
-    datasets: [{
-        label: '{% trans "Active this month" %}',
-        backgroundColor: 'rgb(75, 192, 192)',
-        borderColor: 'rgb(75, 192, 192)',
-        data: {{ register_stats.total }},
-    }]
-};
-
-const config = {
-    type: 'line',
-    data: data,
-    options: {}
-};
-
-var userStats = new Chart(
-    document.getElementById('user_stats'),
-    config
+var registerStats = new Chart(
+    document.getElementById('register_stats'),
+    {
+        type: 'bar',
+        data: {
+            labels: [{% for label in register_stats.labels %}"{{ label }}",{% endfor %}],
+            datasets: [{
+                label: '{% trans "Registrations" %}',
+                backgroundColor: 'rgb(75, 192, 192)',
+                borderColor: 'rgb(75, 192, 192)',
+                data: {{ register_stats.total }},
+            }]
+        },
+        options: {}
+    }
 );
 </script>
diff --git a/bookwyrm/templates/settings/dashboard/status_chart.html b/bookwyrm/templates/settings/dashboard/status_chart.html
index bbacf3f4..5e1045fc 100644
--- a/bookwyrm/templates/settings/dashboard/status_chart.html
+++ b/bookwyrm/templates/settings/dashboard/status_chart.html
@@ -1,26 +1,21 @@
 {% load i18n %}
 <script>
-const status_labels = [{% for label in status_stats.labels %}"{{ label }}",{% endfor %}];
-const status_data = {
-    labels: status_labels,
-    datasets: [{
-        label: '{% trans "Statuses posted" %}',
-        backgroundColor: 'rgb(255, 99, 132)',
-        borderColor: 'rgb(255, 99, 132)',
-        data: {{ status_stats.total }},
-    }]
-};
-// === include 'setup' then 'config' above ===
-
-const status_config = {
-    type: 'bar',
-    data: status_data,
-    options: {}
-};
 
 var statusStats = new Chart(
     document.getElementById('status_stats'),
-    status_config
+    {
+        type: 'bar',
+        data: {
+        labels: [{% for label in status_stats.labels %}"{{ label }}",{% endfor %}],
+        datasets: [{
+                label: '{% trans "Statuses posted" %}',
+                backgroundColor: 'rgb(255, 99, 132)',
+                borderColor: 'rgb(255, 99, 132)',
+                data: {{ status_stats.total }},
+            }]
+        },
+        options: {}
+    },
 );
 </script>
 

From 551b49b903cfdfe8e63de58de788b67b320aaf11 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Mon, 4 Oct 2021 11:14:50 -0700
Subject: [PATCH 07/19] Adds works chart and updates colors

---
 .../settings/dashboard/dashboard.html         | 11 ++++-
 .../dashboard/registration_chart.html         |  4 +-
 .../settings/dashboard/status_chart.html      |  4 +-
 .../settings/dashboard/user_chart.html        | 40 +++++++++----------
 .../settings/dashboard/works_chart.html       | 21 ++++++++++
 bookwyrm/views/admin/dashboard.py             | 11 +++++
 6 files changed, 63 insertions(+), 28 deletions(-)
 create mode 100644 bookwyrm/templates/settings/dashboard/works_chart.html

diff --git a/bookwyrm/templates/settings/dashboard/dashboard.html b/bookwyrm/templates/settings/dashboard/dashboard.html
index cd0ddc84..445bf299 100644
--- a/bookwyrm/templates/settings/dashboard/dashboard.html
+++ b/bookwyrm/templates/settings/dashboard/dashboard.html
@@ -102,6 +102,12 @@
                 <canvas id="user_stats"></canvas>
             </div>
         </div>
+        <div class="column is-half">
+            <h3>{% trans "User signup activity" %}</h3>
+            <div class="box">
+                <canvas id="register_stats"></canvas>
+            </div>
+        </div>
         <div class="column is-half">
             <h3>{% trans "Status activity" %}</h3>
             <div class="box">
@@ -109,9 +115,9 @@
             </div>
         </div>
         <div class="column is-half">
-            <h3>{% trans "User signup activity" %}</h3>
+            <h3>{% trans "Works created" %}</h3>
             <div class="box">
-                <canvas id="register_stats"></canvas>
+                <canvas id="works_stats"></canvas>
             </div>
         </div>
     </div>
@@ -124,4 +130,5 @@
 {% include 'settings/dashboard/user_chart.html' %}
 {% include 'settings/dashboard/status_chart.html' %}
 {% include 'settings/dashboard/registration_chart.html' %}
+{% include 'settings/dashboard/works_chart.html' %}
 {% endblock %}
diff --git a/bookwyrm/templates/settings/dashboard/registration_chart.html b/bookwyrm/templates/settings/dashboard/registration_chart.html
index e98a0f18..3b258fec 100644
--- a/bookwyrm/templates/settings/dashboard/registration_chart.html
+++ b/bookwyrm/templates/settings/dashboard/registration_chart.html
@@ -8,8 +8,8 @@ var registerStats = new Chart(
             labels: [{% for label in register_stats.labels %}"{{ label }}",{% endfor %}],
             datasets: [{
                 label: '{% trans "Registrations" %}',
-                backgroundColor: 'rgb(75, 192, 192)',
-                borderColor: 'rgb(75, 192, 192)',
+                backgroundColor: 'hsl(171, 100%, 41%)',
+                borderColor: 'hsl(171, 100%, 41%)',
                 data: {{ register_stats.total }},
             }]
         },
diff --git a/bookwyrm/templates/settings/dashboard/status_chart.html b/bookwyrm/templates/settings/dashboard/status_chart.html
index 5e1045fc..a59036a5 100644
--- a/bookwyrm/templates/settings/dashboard/status_chart.html
+++ b/bookwyrm/templates/settings/dashboard/status_chart.html
@@ -9,8 +9,8 @@ var statusStats = new Chart(
         labels: [{% for label in status_stats.labels %}"{{ label }}",{% endfor %}],
         datasets: [{
                 label: '{% trans "Statuses posted" %}',
-                backgroundColor: 'rgb(255, 99, 132)',
-                borderColor: 'rgb(255, 99, 132)',
+                backgroundColor: 'hsl(141, 71%, 48%)',
+                borderColor: 'hsl(141, 71%, 48%)',
                 data: {{ status_stats.total }},
             }]
         },
diff --git a/bookwyrm/templates/settings/dashboard/user_chart.html b/bookwyrm/templates/settings/dashboard/user_chart.html
index 33be28f7..a8d356bb 100644
--- a/bookwyrm/templates/settings/dashboard/user_chart.html
+++ b/bookwyrm/templates/settings/dashboard/user_chart.html
@@ -1,29 +1,25 @@
 {% load i18n %}
 <script>
-const labels = [{% for label in user_stats.labels %}"{{ label }}",{% endfor %}];
-const data = {
-    labels: labels,
-    datasets: [{
-        label: '{% trans "Total" %}',
-        backgroundColor: 'rgb(255, 99, 132)',
-        borderColor: 'rgb(255, 99, 132)',
-        data: {{ user_stats.total }},
-    }, {
-        label: '{% trans "Active this month" %}',
-        backgroundColor: 'rgb(75, 192, 192)',
-        borderColor: 'rgb(75, 192, 192)',
-        data: {{ user_stats.active }},
-    }]
-};
-
-const config = {
-    type: 'line',
-    data: data,
-    options: {}
-};
 
 var userStats = new Chart(
     document.getElementById('user_stats'),
-    config
+    {
+        type: 'line',
+        data: {
+            labels: [{% for label in user_stats.labels %}"{{ label }}",{% endfor %}],
+            datasets: [{
+                label: '{% trans "Total" %}',
+                backgroundColor: 'hsl(217, 71%, 53%)',
+                borderColor: 'hsl(217, 71%, 53%)',
+                data: {{ user_stats.total }},
+            }, {
+                label: '{% trans "Active this month" %}',
+                backgroundColor: 'hsl(171, 100%, 41%)',
+                borderColor: 'hsl(171, 100%, 41%)',
+                data: {{ user_stats.active }},
+            }]
+        },
+        options: {}
+    }
 );
 </script>
diff --git a/bookwyrm/templates/settings/dashboard/works_chart.html b/bookwyrm/templates/settings/dashboard/works_chart.html
new file mode 100644
index 00000000..c65014e9
--- /dev/null
+++ b/bookwyrm/templates/settings/dashboard/works_chart.html
@@ -0,0 +1,21 @@
+{% load i18n %}
+<script>
+
+var worksStats = new Chart(
+    document.getElementById('works_stats'),
+    {
+        type: 'bar',
+        data: {
+            labels: [{% for label in works_stats.labels %}"{{ label }}",{% endfor %}],
+            datasets: [{
+                label: '{% trans "Works" %}',
+                backgroundColor: 'hsl(204, 86%, 53%)',
+                borderColor: 'hsl(204, 86%, 53%)',
+                data: {{ works_stats.total }},
+            }]
+        },
+        options: {}
+    }
+);
+</script>
+
diff --git a/bookwyrm/views/admin/dashboard.py b/bookwyrm/views/admin/dashboard.py
index 27edc343..b1aedecf 100644
--- a/bookwyrm/views/admin/dashboard.py
+++ b/bookwyrm/views/admin/dashboard.py
@@ -73,6 +73,16 @@ class Dashboard(View):
             }
         )
 
+        works_chart = Chart(
+            queryset=models.Work.objects,
+            queries = {
+                "total": lambda q, s, e: q.filter(
+                    created_date__gt=s,
+                    created_date__lte=e,
+                ).count()
+            }
+        )
+
         data = {
             "start": start.strftime("%Y-%m-%d"),
             "end": end.strftime("%Y-%m-%d"),
@@ -90,6 +100,7 @@ class Dashboard(View):
             "user_stats": user_chart.get_chart(start, end, interval),
             "status_stats": status_chart.get_chart(start, end, interval),
             "register_stats": register_chart.get_chart(start, end, interval),
+            "works_stats": works_chart.get_chart(start, end, interval),
         }
         return TemplateResponse(request, "settings/dashboard/dashboard.html", data)
 

From 2335945a501949c81f6a40c85216f0428a7889c3 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Mon, 4 Oct 2021 11:16:02 -0700
Subject: [PATCH 08/19] Python formatting

---
 bookwyrm/views/admin/dashboard.py | 24 +++++++++++++-----------
 1 file changed, 13 insertions(+), 11 deletions(-)

diff --git a/bookwyrm/views/admin/dashboard.py b/bookwyrm/views/admin/dashboard.py
index b1aedecf..2766eeeb 100644
--- a/bookwyrm/views/admin/dashboard.py
+++ b/bookwyrm/views/admin/dashboard.py
@@ -46,41 +46,43 @@ class Dashboard(View):
                 "active": lambda q, s, e: q.filter(
                     Q(is_active=True) | Q(deactivation_date__gt=e),
                     created_date__lte=e,
-                ).filter(
+                )
+                .filter(
                     last_active_date__gt=e - timedelta(days=31),
-                ).count()
-            }
+                )
+                .count(),
+            },
         )
 
         status_queryset = models.Status.objects.filter(user__local=True, deleted=False)
         status_chart = Chart(
             queryset=status_queryset,
-            queries = {
+            queries={
                 "total": lambda q, s, e: q.filter(
                     created_date__gt=s,
                     created_date__lte=e,
                 ).count()
-            }
+            },
         )
 
         register_chart = Chart(
             queryset=user_queryset,
-            queries = {
+            queries={
                 "total": lambda q, s, e: q.filter(
                     created_date__gt=s,
                     created_date__lte=e,
                 ).count()
-            }
+            },
         )
 
         works_chart = Chart(
             queryset=models.Work.objects,
-            queries = {
+            queries={
                 "total": lambda q, s, e: q.filter(
                     created_date__gt=s,
                     created_date__lte=e,
                 ).count()
-            }
+            },
         )
 
         data = {
@@ -106,14 +108,14 @@ class Dashboard(View):
 
 
 class Chart:
-    """ Data for a chart """
+    """Data for a chart"""
 
     def __init__(self, queryset, queries: dict):
         self.queryset = queryset
         self.queries = queries
 
     def get_chart(self, start, end, interval):
-        """ load the data for the chart given a time scale and interval """
+        """load the data for the chart given a time scale and interval"""
         interval_start = start
         interval_end = interval_start + timedelta(days=interval)
 

From 636eca97cb121be6a454df9f1734b748c1565040 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Mon, 4 Oct 2021 11:22:14 -0700
Subject: [PATCH 09/19] Better mobile display

---
 bookwyrm/templates/settings/dashboard/dashboard.html | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/bookwyrm/templates/settings/dashboard/dashboard.html b/bookwyrm/templates/settings/dashboard/dashboard.html
index 445bf299..f320028b 100644
--- a/bookwyrm/templates/settings/dashboard/dashboard.html
+++ b/bookwyrm/templates/settings/dashboard/dashboard.html
@@ -9,26 +9,26 @@
 
 {% block panel %}
 
-<div class="columns block has-text-centered">
-    <div class="column is-3">
+<div class="columns block has-text-centered is-mobile is-multiline">
+    <div class="column is-3-desktop is-6-mobile">
         <div class="notification">
             <h3>{% trans "Total users" %}</h3>
             <p class="title is-5">{{ users|intcomma }}</p>
         </div>
     </div>
-    <div class="column is-3">
+    <div class="column is-3-desktop is-6-mobile">
         <div class="notification">
             <h3>{% trans "Active this month" %}</h3>
             <p class="title is-5">{{ active_users|intcomma }}</p>
         </div>
     </div>
-    <div class="column is-3">
+    <div class="column is-3-desktop is-6-mobile">
         <div class="notification">
             <h3>{% trans "Statuses" %}</h3>
             <p class="title is-5">{{ statuses|intcomma }}</p>
         </div>
     </div>
-    <div class="column is-3">
+    <div class="column is-3-desktop is-6-mobile">
         <div class="notification">
             <h3>{% trans "Works" %}</h3>
             <p class="title is-5">{{ works|intcomma }}</p>
@@ -64,7 +64,7 @@
 <div class="block content">
     <h2>{% trans "Instance Activity" %}</h2>
 
-    <form method="get" action="{% url 'settings-dashboard' %}" class="notification has-background-white-bis">
+    <form method="get" action="{% url 'settings-dashboard' %}" class="notification has-background-white-bis scroll-x">
         <div class="is-flex is-align-items-flex-end">
             <div class="ml-1 mr-1">
                 <label class="label" for="id_start">

From f99a61b6bd4f333f5cee242d856e90143760347a Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Mon, 4 Oct 2021 11:40:50 -0700
Subject: [PATCH 10/19] Remove follow suggestion on request

---
 bookwyrm/suggested_users.py | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/bookwyrm/suggested_users.py b/bookwyrm/suggested_users.py
index e8f23632..aa53fcb2 100644
--- a/bookwyrm/suggested_users.py
+++ b/bookwyrm/suggested_users.py
@@ -148,6 +148,17 @@ def update_suggestions_on_follow(sender, instance, created, *args, **kwargs):
     rerank_user_task.delay(instance.user_object.id, update_only=False)
 
 
+@receiver(signals.post_save, sender=models.UserFollowRequest)
+# pylint: disable=unused-argument
+def update_suggestions_on_follow_request(sender, instance, created, *args, **kwargs):
+    """remove a follow from the recs and update the ranks"""
+    if not created or not instance.user_object.discoverable:
+        return
+
+    if instance.user_subject.local:
+        remove_suggestion_task.delay(instance.user_subject.id, instance.user_object.id)
+
+
 @receiver(signals.post_save, sender=models.UserBlocks)
 # pylint: disable=unused-argument
 def update_suggestions_on_block(sender, instance, *args, **kwargs):

From 094931d6d1fa4777a6157e2a4d750b69b20a9a1e Mon Sep 17 00:00:00 2001
From: Levi Bard <levi@unity3d.com>
Date: Wed, 6 Oct 2021 11:37:27 +0200
Subject: [PATCH 11/19] Display half-stars in generated statuses for
 federation. Fixes #1505

---
 bookwyrm/templates/snippets/generated_status/rating.html        | 2 +-
 .../templates/snippets/generated_status/review_pure_name.html   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/bookwyrm/templates/snippets/generated_status/rating.html b/bookwyrm/templates/snippets/generated_status/rating.html
index 2e9fb601..4238be0e 100644
--- a/bookwyrm/templates/snippets/generated_status/rating.html
+++ b/bookwyrm/templates/snippets/generated_status/rating.html
@@ -1,6 +1,6 @@
 {% load i18n %}{% load humanize %}{% load utilities %}
 
-{% blocktrans trimmed with title=book|book_title path=book.remote_id display_rating=rating|floatformat:"0" count counter=rating|add:0 %}
+{% blocktrans trimmed with title=book|book_title path=book.remote_id display_rating=rating|floatformat:"-1" count counter=rating|add:0 %}
 rated <em><a href="{{ path }}">{{ title }}</a></em>: {{ display_rating }} star
 {% plural %}
 rated <em><a href="{{ path }}">{{ title }}</a></em>: {{ display_rating }} stars
diff --git a/bookwyrm/templates/snippets/generated_status/review_pure_name.html b/bookwyrm/templates/snippets/generated_status/review_pure_name.html
index 25960191..e54a818e 100644
--- a/bookwyrm/templates/snippets/generated_status/review_pure_name.html
+++ b/bookwyrm/templates/snippets/generated_status/review_pure_name.html
@@ -1,7 +1,7 @@
 {% load i18n %}
 {% if rating %}
 
-{% blocktrans with book_title=book.title|safe display_rating=rating|floatformat:"0" review_title=name|safe count counter=rating %}Review of "{{ book_title }}" ({{ display_rating }} star): {{ review_title }}{% plural %}Review of "{{ book_title }}" ({{ display_rating }} stars): {{ review_title }}{% endblocktrans %}
+{% blocktrans with book_title=book.title|safe display_rating=rating|floatformat:"-1" review_title=name|safe count counter=rating %}Review of "{{ book_title }}" ({{ display_rating }} star): {{ review_title }}{% plural %}Review of "{{ book_title }}" ({{ display_rating }} stars): {{ review_title }}{% endblocktrans %}
 
 {% else %}
 

From 4c539bc7eb326718bbf41900972eb9b3ed2cb2a1 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Tue, 5 Oct 2021 08:00:13 -0700
Subject: [PATCH 12/19] Fixes image paths in endpoints

---
 bookwyrm/settings.py        |  6 ++++--
 bookwyrm/views/wellknown.py | 17 +++++++++++------
 2 files changed, 15 insertions(+), 8 deletions(-)

diff --git a/bookwyrm/settings.py b/bookwyrm/settings.py
index 39469a7a..b2f90487 100644
--- a/bookwyrm/settings.py
+++ b/bookwyrm/settings.py
@@ -210,12 +210,13 @@ if USE_S3:
     AWS_S3_OBJECT_PARAMETERS = {"CacheControl": "max-age=86400"}
     # S3 Static settings
     STATIC_LOCATION = "static"
-    STATIC_URL = f"https://{AWS_S3_CUSTOM_DOMAIN}/{STATIC_LOCATION}/"
+    STATIC_URL = f"{PROTOCOL}://{AWS_S3_CUSTOM_DOMAIN}/{STATIC_LOCATION}/"
     STATICFILES_STORAGE = "bookwyrm.storage_backends.StaticStorage"
     # S3 Media settings
     MEDIA_LOCATION = "images"
-    MEDIA_URL = f"https://{AWS_S3_CUSTOM_DOMAIN}/{MEDIA_LOCATION}/"
+    MEDIA_URL = f"{PROTOCOL}://{AWS_S3_CUSTOM_DOMAIN}/{MEDIA_LOCATION}/"
     MEDIA_FULL_URL = MEDIA_URL
+    STATIC_FULL_URL = STATIC_URL
     DEFAULT_FILE_STORAGE = "bookwyrm.storage_backends.ImagesStorage"
     # I don't know if it's used, but the site crashes without it
     STATIC_ROOT = os.path.join(BASE_DIR, env("STATIC_ROOT", "static"))
@@ -225,4 +226,5 @@ else:
     STATIC_ROOT = os.path.join(BASE_DIR, env("STATIC_ROOT", "static"))
     MEDIA_URL = "/images/"
     MEDIA_FULL_URL = f"{PROTOCOL}://{DOMAIN}{MEDIA_URL}"
+    STATIC_FULL_URL = f"{PROTOCOL}://{DOMAIN}{STATIC_URL}"
     MEDIA_ROOT = os.path.join(BASE_DIR, env("MEDIA_ROOT", "images"))
diff --git a/bookwyrm/views/wellknown.py b/bookwyrm/views/wellknown.py
index 0f860441..de4f7e62 100644
--- a/bookwyrm/views/wellknown.py
+++ b/bookwyrm/views/wellknown.py
@@ -9,7 +9,7 @@ from django.utils import timezone
 from django.views.decorators.http import require_GET
 
 from bookwyrm import models
-from bookwyrm.settings import DOMAIN, VERSION, MEDIA_FULL_URL
+from bookwyrm.settings import DOMAIN, VERSION, MEDIA_FULL_URL, STATIC_FULL_URL
 
 
 @require_GET
@@ -93,8 +93,7 @@ def instance_info(_):
     status_count = models.Status.objects.filter(user__local=True, deleted=False).count()
 
     site = models.SiteSettings.get()
-    logo_path = site.logo or "images/logo.png"
-    logo = f"{MEDIA_FULL_URL}{logo_path}"
+    logo = get_image_url(site.logo, "logo.png")
     return JsonResponse(
         {
             "uri": DOMAIN,
@@ -134,8 +133,14 @@ def host_meta(request):
 def opensearch(request):
     """Open Search xml spec"""
     site = models.SiteSettings.get()
-    logo_path = site.favicon or "images/favicon.png"
-    logo = f"{MEDIA_FULL_URL}{logo_path}"
+    image = get_image_url(site.favicon, "favicon.png")
     return TemplateResponse(
-        request, "opensearch.xml", {"image": logo, "DOMAIN": DOMAIN}
+        request, "opensearch.xml", {"image": image, "DOMAIN": DOMAIN}
     )
+
+
+def get_image_url(obj, fallback):
+    """helper for loading the full path to an image"""
+    if obj:
+        return f"{MEDIA_FULL_URL}{obj}"
+    return f"{STATIC_FULL_URL}images/{fallback}"

From b2671e78ef63064ca059a1bbde13f11e1b93430b Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Wed, 6 Oct 2021 09:48:11 -0700
Subject: [PATCH 13/19] Moves privacy filter to model class

---
 bookwyrm/models/base_model.py | 73 ++++++++++++++++++++++++++++++++++-
 bookwyrm/views/helpers.py     | 52 -------------------------
 2 files changed, 72 insertions(+), 53 deletions(-)

diff --git a/bookwyrm/models/base_model.py b/bookwyrm/models/base_model.py
index ca32521a..cde181bd 100644
--- a/bookwyrm/models/base_model.py
+++ b/bookwyrm/models/base_model.py
@@ -2,8 +2,9 @@
 import base64
 from Crypto import Random
 
-from django.core.exceptions import PermissionDenied
+from django.core.exceptions import PermissionDenied, FieldError
 from django.db import models
+from django.db.models import Q
 from django.dispatch import receiver
 from django.http import Http404
 from django.utils.translation import gettext_lazy as _
@@ -105,6 +106,76 @@ class BookWyrmModel(models.Model):
 
         raise PermissionDenied()
 
+    @classmethod
+    def privacy_filter(
+        cls, viewer, privacy_levels=None, following_only=False, **filters
+    ):
+        """filter objects that have "user" and "privacy" fields"""
+        queryset = cls.objects
+        if hasattr(queryset, "select_subclasses"):
+            queryset = queryset.select_subclasses()
+
+        privacy_levels = privacy_levels or ["public", "unlisted", "followers", "direct"]
+        # if there'd a deleted field, exclude deleted items
+        try:
+            queryset = queryset.filter(deleted=False)
+        except FieldError:
+            pass
+
+        # exclude blocks from both directions
+        if not viewer.is_anonymous:
+            queryset = queryset.exclude(
+                Q(user__blocked_by=viewer) | Q(user__blocks=viewer)
+            )
+
+        # you can't see followers only or direct messages if you're not logged in
+        if viewer.is_anonymous:
+            privacy_levels = [
+                p for p in privacy_levels if not p in ["followers", "direct"]
+            ]
+
+        # filter to only privided privacy levels
+        queryset = queryset.filter(privacy__in=privacy_levels)
+
+        # only include statuses the user follows
+        if following_only:
+            queryset = queryset.exclude(
+                ~Q(  # remove everythign except
+                    Q(user__followers=viewer)
+                    | Q(user=viewer)  # user following
+                    | Q(mention_users=viewer)  # is self  # mentions user
+                ),
+            )
+        # exclude followers-only statuses the user doesn't follow
+        elif "followers" in privacy_levels:
+            queryset = cls.followers_filter(queryset, viewer)
+
+        # exclude direct messages not intended for the user
+        if "direct" in privacy_levels:
+            queryset = cls.direct_filter(queryset, viewer)
+
+        return queryset
+
+    @classmethod
+    def followers_filter(cls, queryset, viewer):
+        """Override-able filter for "followers" privacy level"""
+        return queryset.exclude(
+            ~Q(  # user isn't following and it isn't their own status
+                Q(user__followers=viewer) | Q(user=viewer)
+            ),
+            privacy="followers",  # and the status is followers only
+        )
+
+    @classmethod
+    def direct_filter(cls, queryset, viewer):
+        """Override-able filter for "direct" privacy level"""
+        try:
+            return queryset.exclude(
+                ~Q(Q(user=viewer) | Q(mention_users=viewer)), privacy="direct"
+            )
+        except FieldError:
+            return queryset.exclude(~Q(user=viewer), privacy="direct")
+
 
 @receiver(models.signals.post_save)
 # pylint: disable=unused-argument
diff --git a/bookwyrm/views/helpers.py b/bookwyrm/views/helpers.py
index 7e469f7f..f31b41ff 100644
--- a/bookwyrm/views/helpers.py
+++ b/bookwyrm/views/helpers.py
@@ -6,8 +6,6 @@ import dateutil.tz
 from dateutil.parser import ParserError
 
 from requests import HTTPError
-from django.core.exceptions import FieldError
-from django.db.models import Q
 from django.http import Http404
 
 from bookwyrm import activitypub, models
@@ -50,56 +48,6 @@ def is_bookwyrm_request(request):
     return True
 
 
-def privacy_filter(viewer, queryset, privacy_levels=None, following_only=False):
-    """filter objects that have "user" and "privacy" fields"""
-    privacy_levels = privacy_levels or ["public", "unlisted", "followers", "direct"]
-    # if there'd a deleted field, exclude deleted items
-    try:
-        queryset = queryset.filter(deleted=False)
-    except FieldError:
-        pass
-
-    # exclude blocks from both directions
-    if not viewer.is_anonymous:
-        queryset = queryset.exclude(Q(user__blocked_by=viewer) | Q(user__blocks=viewer))
-
-    # you can't see followers only or direct messages if you're not logged in
-    if viewer.is_anonymous:
-        privacy_levels = [p for p in privacy_levels if not p in ["followers", "direct"]]
-
-    # filter to only privided privacy levels
-    queryset = queryset.filter(privacy__in=privacy_levels)
-
-    # only include statuses the user follows
-    if following_only:
-        queryset = queryset.exclude(
-            ~Q(  # remove everythign except
-                Q(user__followers=viewer)
-                | Q(user=viewer)  # user following
-                | Q(mention_users=viewer)  # is self  # mentions user
-            ),
-        )
-    # exclude followers-only statuses the user doesn't follow
-    elif "followers" in privacy_levels:
-        queryset = queryset.exclude(
-            ~Q(  # user isn't following and it isn't their own status
-                Q(user__followers=viewer) | Q(user=viewer)
-            ),
-            privacy="followers",  # and the status is followers only
-        )
-
-    # exclude direct messages not intended for the user
-    if "direct" in privacy_levels:
-        try:
-            queryset = queryset.exclude(
-                ~Q(Q(user=viewer) | Q(mention_users=viewer)), privacy="direct"
-            )
-        except FieldError:
-            queryset = queryset.exclude(~Q(user=viewer), privacy="direct")
-
-    return queryset
-
-
 def handle_remote_webfinger(query):
     """webfingerin' other servers"""
     user = None

From 97cc129478f2d59b9d405c49b703951cd6092dca Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Wed, 6 Oct 2021 10:37:09 -0700
Subject: [PATCH 14/19] Updates calls to privacy_filter

---
 bookwyrm/activitystreams.py            | 47 +++++++++++++-------------
 bookwyrm/models/base_model.py          |  4 +--
 bookwyrm/templatetags/bookwyrm_tags.py |  6 ++--
 bookwyrm/views/books/books.py          | 15 ++++----
 bookwyrm/views/feed.py                 | 30 ++++++++--------
 bookwyrm/views/list.py                 | 14 +++-----
 bookwyrm/views/rss_feed.py             |  8 ++---
 bookwyrm/views/search.py               |  5 ++-
 bookwyrm/views/shelf.py                | 12 +++----
 bookwyrm/views/user.py                 |  5 ++-
 10 files changed, 68 insertions(+), 78 deletions(-)

diff --git a/bookwyrm/activitystreams.py b/bookwyrm/activitystreams.py
index f843641f..997f3329 100644
--- a/bookwyrm/activitystreams.py
+++ b/bookwyrm/activitystreams.py
@@ -8,7 +8,6 @@ from django.utils import timezone
 from bookwyrm import models
 from bookwyrm.redis_store import RedisStore, r
 from bookwyrm.tasks import app
-from bookwyrm.views.helpers import privacy_filter
 
 
 class ActivityStream(RedisStore):
@@ -43,7 +42,7 @@ class ActivityStream(RedisStore):
     def add_user_statuses(self, viewer, user):
         """add a user's statuses to another user's feed"""
         # only add the statuses that the viewer should be able to see (ie, not dms)
-        statuses = privacy_filter(viewer, user.status_set.all())
+        statuses = models.Status.privacy_filter(viewer).filter(user=user)
         self.bulk_add_objects_to_store(statuses, self.stream_id(viewer))
 
     def remove_user_statuses(self, viewer, user):
@@ -113,9 +112,8 @@ class ActivityStream(RedisStore):
 
     def get_statuses_for_user(self, user):  # pylint: disable=no-self-use
         """given a user, what statuses should they see on this stream"""
-        return privacy_filter(
+        return models.Status.privacy_filter(
             user,
-            models.Status.objects.select_subclasses(),
             privacy_levels=["public", "unlisted", "followers"],
         )
 
@@ -139,9 +137,8 @@ class HomeStream(ActivityStream):
         ).distinct()
 
     def get_statuses_for_user(self, user):
-        return privacy_filter(
+        return models.Status.privacy_filter(
             user,
-            models.Status.objects.select_subclasses(),
             privacy_levels=["public", "unlisted", "followers"],
             following_only=True,
         )
@@ -160,11 +157,10 @@ class LocalStream(ActivityStream):
 
     def get_statuses_for_user(self, user):
         # all public statuses by a local user
-        return privacy_filter(
+        return models.Status.privacy_filter(
             user,
-            models.Status.objects.select_subclasses().filter(user__local=True),
             privacy_levels=["public"],
-        )
+        ).filter(user__local=True)
 
 
 class BooksStream(ActivityStream):
@@ -197,50 +193,53 @@ class BooksStream(ActivityStream):
         books = user.shelfbook_set.values_list(
             "book__parent_work__id", flat=True
         ).distinct()
-        return privacy_filter(
-            user,
-            models.Status.objects.select_subclasses()
+        return (
+            models.Status.privacy_filter(
+                user,
+                privacy_levels=["public"],
+            )
             .filter(
                 Q(comment__book__parent_work__id__in=books)
                 | Q(quotation__book__parent_work__id__in=books)
                 | Q(review__book__parent_work__id__in=books)
                 | Q(mention_books__parent_work__id__in=books)
             )
-            .distinct(),
-            privacy_levels=["public"],
+            .distinct()
         )
 
     def add_book_statuses(self, user, book):
         """add statuses about a book to a user's feed"""
         work = book.parent_work
-        statuses = privacy_filter(
-            user,
-            models.Status.objects.select_subclasses()
+        statuses = (
+            models.Status.privacy_filter(
+                user,
+                privacy_levels=["public"],
+            )
             .filter(
                 Q(comment__book__parent_work=work)
                 | Q(quotation__book__parent_work=work)
                 | Q(review__book__parent_work=work)
                 | Q(mention_books__parent_work=work)
             )
-            .distinct(),
-            privacy_levels=["public"],
+            .distinct()
         )
         self.bulk_add_objects_to_store(statuses, self.stream_id(user))
 
     def remove_book_statuses(self, user, book):
         """add statuses about a book to a user's feed"""
         work = book.parent_work
-        statuses = privacy_filter(
-            user,
-            models.Status.objects.select_subclasses()
+        statuses = (
+            models.Status.privacy_filter(
+                user,
+                privacy_levels=["public"],
+            )
             .filter(
                 Q(comment__book__parent_work=work)
                 | Q(quotation__book__parent_work=work)
                 | Q(review__book__parent_work=work)
                 | Q(mention_books__parent_work=work)
             )
-            .distinct(),
-            privacy_levels=["public"],
+            .distinct()
         )
         self.bulk_remove_objects_from_store(statuses, self.stream_id(user))
 
diff --git a/bookwyrm/models/base_model.py b/bookwyrm/models/base_model.py
index cde181bd..1e147b5e 100644
--- a/bookwyrm/models/base_model.py
+++ b/bookwyrm/models/base_model.py
@@ -107,9 +107,7 @@ class BookWyrmModel(models.Model):
         raise PermissionDenied()
 
     @classmethod
-    def privacy_filter(
-        cls, viewer, privacy_levels=None, following_only=False, **filters
-    ):
+    def privacy_filter(cls, viewer, privacy_levels=None, following_only=False):
         """filter objects that have "user" and "privacy" fields"""
         queryset = cls.objects
         if hasattr(queryset, "select_subclasses"):
diff --git a/bookwyrm/templatetags/bookwyrm_tags.py b/bookwyrm/templatetags/bookwyrm_tags.py
index 2e03c13b..bccd8c75 100644
--- a/bookwyrm/templatetags/bookwyrm_tags.py
+++ b/bookwyrm/templatetags/bookwyrm_tags.py
@@ -2,7 +2,7 @@
 from django import template
 from django.db.models import Avg
 
-from bookwyrm import models, views
+from bookwyrm import models
 
 
 register = template.Library()
@@ -11,8 +11,8 @@ register = template.Library()
 @register.filter(name="rating")
 def get_rating(book, user):
     """get the overall rating of a book"""
-    queryset = views.helpers.privacy_filter(
-        user, models.Review.objects.filter(book__parent_work__editions=book)
+    queryset = models.Review.privacy_filter(user).filter(
+        book__parent_work__editions=book
     )
     return queryset.aggregate(Avg("rating"))["rating__avg"]
 
diff --git a/bookwyrm/views/books/books.py b/bookwyrm/views/books/books.py
index 298ba5a3..e495da3e 100644
--- a/bookwyrm/views/books/books.py
+++ b/bookwyrm/views/books/books.py
@@ -16,7 +16,7 @@ from bookwyrm.activitypub import ActivitypubResponse
 from bookwyrm.connectors import connector_manager
 from bookwyrm.connectors.abstract_connector import get_image
 from bookwyrm.settings import PAGE_LENGTH
-from bookwyrm.views.helpers import is_api_request, privacy_filter
+from bookwyrm.views.helpers import is_api_request
 
 
 # pylint: disable=no-self-use
@@ -48,8 +48,8 @@ class Book(View):
             raise Http404()
 
         # all reviews for all editions of the book
-        reviews = privacy_filter(
-            request.user, models.Review.objects.filter(book__parent_work__editions=book)
+        reviews = models.Review.privacy_filter(request.user).filter(
+            book__parent_work__editions=book
         )
 
         # the reviews to show
@@ -66,12 +66,9 @@ class Book(View):
         queryset = queryset.select_related("user").order_by("-published_date")
         paginated = Paginator(queryset, PAGE_LENGTH)
 
-        lists = privacy_filter(
-            request.user,
-            models.List.objects.filter(
-                listitem__approved=True,
-                listitem__book__in=book.parent_work.editions.all(),
-            ),
+        lists = models.List.privacy_filter(request.user,).filter(
+            listitem__approved=True,
+            listitem__book__in=book.parent_work.editions.all(),
         )
         data = {
             "book": book,
diff --git a/bookwyrm/views/feed.py b/bookwyrm/views/feed.py
index 7f1bc22c..8eff848c 100644
--- a/bookwyrm/views/feed.py
+++ b/bookwyrm/views/feed.py
@@ -13,7 +13,7 @@ from bookwyrm import activitystreams, forms, models
 from bookwyrm.activitypub import ActivitypubResponse
 from bookwyrm.settings import PAGE_LENGTH, STREAMS
 from bookwyrm.suggested_users import suggested_users
-from .helpers import get_user_from_username, privacy_filter
+from .helpers import get_user_from_username
 from .helpers import is_api_request, is_bookwyrm_request
 
 
@@ -56,11 +56,15 @@ class DirectMessage(View):
     def get(self, request, username=None):
         """like a feed but for dms only"""
         # remove fancy subclasses of status, keep just good ol' notes
-        queryset = models.Status.objects.filter(
-            review__isnull=True,
-            comment__isnull=True,
-            quotation__isnull=True,
-            generatednote__isnull=True,
+        activities = (
+            models.Status.privacy_filter(request.user, privacy_levels=["direct"])
+            .filter(
+                review__isnull=True,
+                comment__isnull=True,
+                quotation__isnull=True,
+                generatednote__isnull=True,
+            )
+            .order_by("-published_date")
         )
 
         user = None
@@ -70,11 +74,7 @@ class DirectMessage(View):
             except Http404:
                 pass
         if user:
-            queryset = queryset.filter(Q(user=user) | Q(mention_users=user))
-
-        activities = privacy_filter(
-            request.user, queryset, privacy_levels=["direct"]
-        ).order_by("-published_date")
+            activities = activities.filter(Q(user=user) | Q(mention_users=user))
 
         paginated = Paginator(activities, PAGE_LENGTH)
         data = {
@@ -109,9 +109,11 @@ class Status(View):
                 status.to_activity(pure=not is_bookwyrm_request(request))
             )
 
-        visible_thread = privacy_filter(
-            request.user, models.Status.objects.filter(thread_id=status.thread_id)
-        ).values_list("id", flat=True)
+        visible_thread = (
+            models.Status.privacy_filter(request.user)
+            .filter(thread_id=status.thread_id)
+            .values_list("id", flat=True)
+        )
         visible_thread = list(visible_thread)
 
         ancestors = models.Status.objects.select_subclasses().raw(
diff --git a/bookwyrm/views/list.py b/bookwyrm/views/list.py
index a9d77597..b01a0e0e 100644
--- a/bookwyrm/views/list.py
+++ b/bookwyrm/views/list.py
@@ -18,7 +18,7 @@ from django.views.decorators.http import require_POST
 from bookwyrm import book_search, forms, models
 from bookwyrm.activitypub import ActivitypubResponse
 from bookwyrm.settings import PAGE_LENGTH
-from .helpers import is_api_request, privacy_filter
+from .helpers import is_api_request
 from .helpers import get_user_from_username
 
 
@@ -30,9 +30,10 @@ class Lists(View):
         """display a book list"""
         # hide lists with no approved books
         lists = (
-            models.List.objects.annotate(
-                item_count=Count("listitem", filter=Q(listitem__approved=True))
+            models.List.privacy_filter(
+                request.user, privacy_levels=["public", "followers"]
             )
+            .annotate(item_count=Count("listitem", filter=Q(listitem__approved=True)))
             .filter(item_count__gt=0)
             .select_related("user")
             .prefetch_related("listitem_set")
@@ -40,10 +41,6 @@ class Lists(View):
             .distinct()
         )
 
-        lists = privacy_filter(
-            request.user, lists, privacy_levels=["public", "followers"]
-        )
-
         paginated = Paginator(lists, 12)
         data = {
             "lists": paginated.get_page(request.GET.get("page")),
@@ -89,8 +86,7 @@ class UserLists(View):
     def get(self, request, username):
         """display a book list"""
         user = get_user_from_username(request.user, username)
-        lists = models.List.objects.filter(user=user)
-        lists = privacy_filter(request.user, lists)
+        lists = models.List.privacy_filter(request.user).filter(user=user)
         paginated = Paginator(lists, 12)
 
         data = {
diff --git a/bookwyrm/views/rss_feed.py b/bookwyrm/views/rss_feed.py
index 5faa1624..b924095c 100644
--- a/bookwyrm/views/rss_feed.py
+++ b/bookwyrm/views/rss_feed.py
@@ -4,7 +4,8 @@ from django.contrib.syndication.views import Feed
 from django.template.loader import get_template
 from django.utils.translation import gettext_lazy as _
 
-from .helpers import get_user_from_username, privacy_filter
+from bookwyrm import models
+from .helpers import get_user_from_username
 
 # pylint: disable=no-self-use, unused-argument
 class RssFeed(Feed):
@@ -35,11 +36,10 @@ class RssFeed(Feed):
 
     def items(self, obj):
         """the user's activity feed"""
-        return privacy_filter(
+        return models.Status.privacy_filter(
             obj,
-            obj.status_set.select_subclasses(),
             privacy_levels=["public", "unlisted"],
-        )
+        ).filter(user=obj)
 
     def item_link(self, item):
         """link to the status"""
diff --git a/bookwyrm/views/search.py b/bookwyrm/views/search.py
index d131b399..7a56ae72 100644
--- a/bookwyrm/views/search.py
+++ b/bookwyrm/views/search.py
@@ -13,7 +13,7 @@ from bookwyrm.connectors import connector_manager
 from bookwyrm.book_search import search, format_search_result
 from bookwyrm.settings import PAGE_LENGTH
 from bookwyrm.utils import regex
-from .helpers import is_api_request, privacy_filter
+from .helpers import is_api_request
 from .helpers import handle_remote_webfinger
 
 
@@ -108,9 +108,8 @@ def user_search(query, viewer, *_):
 def list_search(query, viewer, *_):
     """any relevent lists?"""
     return (
-        privacy_filter(
+        models.List.privacy_filter(
             viewer,
-            models.List.objects,
             privacy_levels=["public", "followers"],
         )
         .annotate(
diff --git a/bookwyrm/views/shelf.py b/bookwyrm/views/shelf.py
index 35f660b5..37f320dc 100644
--- a/bookwyrm/views/shelf.py
+++ b/bookwyrm/views/shelf.py
@@ -17,7 +17,6 @@ from bookwyrm import forms, models
 from bookwyrm.activitypub import ActivitypubResponse
 from bookwyrm.settings import PAGE_LENGTH
 from .helpers import is_api_request, get_user_from_username
-from .helpers import privacy_filter
 
 
 # pylint: disable=no-self-use
@@ -33,7 +32,7 @@ class Shelf(View):
         if is_self:
             shelves = user.shelf_set.all()
         else:
-            shelves = privacy_filter(request.user, user.shelf_set).all()
+            shelves = models.Shelf.privacy_filter(request.user).filter(user=user).all()
 
         # get the shelf and make sure the logged in user should be able to see it
         if shelf_identifier:
@@ -58,16 +57,17 @@ class Shelf(View):
         if is_api_request(request):
             return ActivitypubResponse(shelf.to_activity(**request.GET))
 
-        reviews = models.Review.objects.filter(
+        reviews = models.Review.objects
+        if not is_self:
+            reviews = models.Review.privacy_filter(request.user)
+
+        reviews = reviews.filter(
             user=user,
             rating__isnull=False,
             book__id=OuterRef("id"),
             deleted=False,
         ).order_by("-published_date")
 
-        if not is_self:
-            reviews = privacy_filter(request.user, reviews)
-
         books = books.annotate(
             rating=Subquery(reviews.values("rating")[:1]),
             shelved_date=F("shelfbook__shelved_date"),
diff --git a/bookwyrm/views/user.py b/bookwyrm/views/user.py
index fb553228..0d8d385b 100644
--- a/bookwyrm/views/user.py
+++ b/bookwyrm/views/user.py
@@ -12,7 +12,6 @@ from bookwyrm import models
 from bookwyrm.activitypub import ActivitypubResponse
 from bookwyrm.settings import PAGE_LENGTH
 from .helpers import get_user_from_username, is_api_request
-from .helpers import privacy_filter
 
 
 # pylint: disable=no-self-use
@@ -56,10 +55,10 @@ class User(View):
 
         # user's posts
         activities = (
-            privacy_filter(
+            models.Status.privacy_filter(
                 request.user,
-                user.status_set.select_subclasses(),
             )
+            .filter(user=user)
             .select_related(
                 "user",
                 "reply_parent",

From 68b6b1fbae10ca8d2acb756831255d8c14a20405 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Wed, 6 Oct 2021 11:08:54 -0700
Subject: [PATCH 15/19] User override in child class instead of if/else for
 privacy_filter

---
 bookwyrm/models/base_model.py | 7 +------
 bookwyrm/models/status.py     | 7 +++++++
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/bookwyrm/models/base_model.py b/bookwyrm/models/base_model.py
index 1e147b5e..fd70fc17 100644
--- a/bookwyrm/models/base_model.py
+++ b/bookwyrm/models/base_model.py
@@ -167,12 +167,7 @@ class BookWyrmModel(models.Model):
     @classmethod
     def direct_filter(cls, queryset, viewer):
         """Override-able filter for "direct" privacy level"""
-        try:
-            return queryset.exclude(
-                ~Q(Q(user=viewer) | Q(mention_users=viewer)), privacy="direct"
-            )
-        except FieldError:
-            return queryset.exclude(~Q(user=viewer), privacy="direct")
+        return queryset.exclude(~Q(user=viewer), privacy="direct")
 
 
 @receiver(models.signals.post_save)
diff --git a/bookwyrm/models/status.py b/bookwyrm/models/status.py
index 58488123..3e5f638f 100644
--- a/bookwyrm/models/status.py
+++ b/bookwyrm/models/status.py
@@ -207,6 +207,13 @@ class Status(OrderedCollectionPageMixin, BookWyrmModel):
         if isinstance(self, (GeneratedNote, ReviewRating)):
             raise PermissionDenied()
 
+    @classmethod
+    def direct_filter(cls, queryset, viewer):
+        """Override-able filter for "direct" privacy level"""
+        return queryset.exclude(
+            ~Q(Q(user=viewer) | Q(mention_users=viewer)), privacy="direct"
+        )
+
 
 class GeneratedNote(Status):
     """these are app-generated messages about user activity"""

From 2307cb2227740f2900e172e9b148c2ba564f507f Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Wed, 6 Oct 2021 11:12:03 -0700
Subject: [PATCH 16/19] Filter followers only in place

It's only used in one spot
---
 bookwyrm/activitystreams.py   |  7 ++++++-
 bookwyrm/models/base_model.py | 14 ++------------
 2 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/bookwyrm/activitystreams.py b/bookwyrm/activitystreams.py
index 997f3329..4896e07d 100644
--- a/bookwyrm/activitystreams.py
+++ b/bookwyrm/activitystreams.py
@@ -140,7 +140,12 @@ class HomeStream(ActivityStream):
         return models.Status.privacy_filter(
             user,
             privacy_levels=["public", "unlisted", "followers"],
-            following_only=True,
+        ).exclude(
+            ~Q(  # remove everything except
+                Q(user__followers=user)  # user following
+                | Q(user=user)  # is self
+                | Q(mention_users=user)  # mentions user
+            ),
         )
 
 
diff --git a/bookwyrm/models/base_model.py b/bookwyrm/models/base_model.py
index fd70fc17..6145a1d4 100644
--- a/bookwyrm/models/base_model.py
+++ b/bookwyrm/models/base_model.py
@@ -107,7 +107,7 @@ class BookWyrmModel(models.Model):
         raise PermissionDenied()
 
     @classmethod
-    def privacy_filter(cls, viewer, privacy_levels=None, following_only=False):
+    def privacy_filter(cls, viewer, privacy_levels=None):
         """filter objects that have "user" and "privacy" fields"""
         queryset = cls.objects
         if hasattr(queryset, "select_subclasses"):
@@ -135,17 +135,7 @@ class BookWyrmModel(models.Model):
         # filter to only privided privacy levels
         queryset = queryset.filter(privacy__in=privacy_levels)
 
-        # only include statuses the user follows
-        if following_only:
-            queryset = queryset.exclude(
-                ~Q(  # remove everythign except
-                    Q(user__followers=viewer)
-                    | Q(user=viewer)  # user following
-                    | Q(mention_users=viewer)  # is self  # mentions user
-                ),
-            )
-        # exclude followers-only statuses the user doesn't follow
-        elif "followers" in privacy_levels:
+        if "followers" in privacy_levels:
             queryset = cls.followers_filter(queryset, viewer)
 
         # exclude direct messages not intended for the user

From 34d6c8886174c909c87b66ba07faecb318c47a35 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Wed, 6 Oct 2021 11:15:17 -0700
Subject: [PATCH 17/19] Move privacy levels filter to clearer location

---
 bookwyrm/models/base_model.py | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/bookwyrm/models/base_model.py b/bookwyrm/models/base_model.py
index 6145a1d4..06445a11 100644
--- a/bookwyrm/models/base_model.py
+++ b/bookwyrm/models/base_model.py
@@ -114,7 +114,13 @@ class BookWyrmModel(models.Model):
             queryset = queryset.select_subclasses()
 
         privacy_levels = privacy_levels or ["public", "unlisted", "followers", "direct"]
-        # if there'd a deleted field, exclude deleted items
+        # you can't see followers only or direct messages if you're not logged in
+        if viewer.is_anonymous:
+            privacy_levels = [
+                p for p in privacy_levels if not p in ["followers", "direct"]
+            ]
+
+        # if there's a deleted field, exclude deleted items
         try:
             queryset = queryset.filter(deleted=False)
         except FieldError:
@@ -126,12 +132,6 @@ class BookWyrmModel(models.Model):
                 Q(user__blocked_by=viewer) | Q(user__blocks=viewer)
             )
 
-        # you can't see followers only or direct messages if you're not logged in
-        if viewer.is_anonymous:
-            privacy_levels = [
-                p for p in privacy_levels if not p in ["followers", "direct"]
-            ]
-
         # filter to only privided privacy levels
         queryset = queryset.filter(privacy__in=privacy_levels)
 

From 53e576256313bdd45f4a7e43ecb3b810a075446e Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Wed, 6 Oct 2021 11:19:13 -0700
Subject: [PATCH 18/19] Status-specific filters in status model

---
 bookwyrm/models/base_model.py | 8 +-------
 bookwyrm/models/status.py     | 8 +++++++-
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/bookwyrm/models/base_model.py b/bookwyrm/models/base_model.py
index 06445a11..d32ec67b 100644
--- a/bookwyrm/models/base_model.py
+++ b/bookwyrm/models/base_model.py
@@ -2,7 +2,7 @@
 import base64
 from Crypto import Random
 
-from django.core.exceptions import PermissionDenied, FieldError
+from django.core.exceptions import PermissionDenied
 from django.db import models
 from django.db.models import Q
 from django.dispatch import receiver
@@ -120,12 +120,6 @@ class BookWyrmModel(models.Model):
                 p for p in privacy_levels if not p in ["followers", "direct"]
             ]
 
-        # if there's a deleted field, exclude deleted items
-        try:
-            queryset = queryset.filter(deleted=False)
-        except FieldError:
-            pass
-
         # exclude blocks from both directions
         if not viewer.is_anonymous:
             queryset = queryset.exclude(
diff --git a/bookwyrm/models/status.py b/bookwyrm/models/status.py
index 3e5f638f..1325aa88 100644
--- a/bookwyrm/models/status.py
+++ b/bookwyrm/models/status.py
@@ -6,6 +6,7 @@ from django.apps import apps
 from django.core.exceptions import PermissionDenied
 from django.core.validators import MaxValueValidator, MinValueValidator
 from django.db import models
+from django.db.models import Q
 from django.dispatch import receiver
 from django.template.loader import get_template
 from django.utils import timezone
@@ -207,9 +208,14 @@ class Status(OrderedCollectionPageMixin, BookWyrmModel):
         if isinstance(self, (GeneratedNote, ReviewRating)):
             raise PermissionDenied()
 
+    @classmethod
+    def privacy_filter(cls, viewer, privacy_levels=None):
+        queryset = super().privacy_filter(viewer, privacy_levels=privacy_levels)
+        return queryset.filter(deleted=False)
+
     @classmethod
     def direct_filter(cls, queryset, viewer):
-        """Override-able filter for "direct" privacy level"""
+        """Overridden filter for "direct" privacy level"""
         return queryset.exclude(
             ~Q(Q(user=viewer) | Q(mention_users=viewer)), privacy="direct"
         )

From 8d00ef4318a5f7500cf90cd0ae5bd30efa45fab4 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Wed, 6 Oct 2021 11:23:36 -0700
Subject: [PATCH 19/19] Style cleanup

---
 bookwyrm/models/base_model.py | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/bookwyrm/models/base_model.py b/bookwyrm/models/base_model.py
index d32ec67b..b781a506 100644
--- a/bookwyrm/models/base_model.py
+++ b/bookwyrm/models/base_model.py
@@ -119,14 +119,13 @@ class BookWyrmModel(models.Model):
             privacy_levels = [
                 p for p in privacy_levels if not p in ["followers", "direct"]
             ]
-
-        # exclude blocks from both directions
-        if not viewer.is_anonymous:
+        else:
+            # exclude blocks from both directions
             queryset = queryset.exclude(
                 Q(user__blocked_by=viewer) | Q(user__blocks=viewer)
             )
 
-        # filter to only privided privacy levels
+        # filter to only provided privacy levels
         queryset = queryset.filter(privacy__in=privacy_levels)
 
         if "followers" in privacy_levels: