From 1e01e76ac25d9bb064317f6cfce794cda6a95b86 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Sat, 12 Dec 2020 18:06:48 -0800
Subject: [PATCH 1/5] removes unneeded imports

---
 bookwyrm/models/author.py       | 3 ++-
 bookwyrm/models/base_model.py   | 1 +
 bookwyrm/models/import_job.py   | 1 -
 bookwyrm/models/relationship.py | 2 +-
 bookwyrm/status.py              | 2 +-
 5 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/bookwyrm/models/author.py b/bookwyrm/models/author.py
index 79973a37..331d2dd6 100644
--- a/bookwyrm/models/author.py
+++ b/bookwyrm/models/author.py
@@ -16,7 +16,8 @@ class Author(ActivitypubMixin, BookWyrmModel):
         max_length=255, blank=True, null=True, deduplication_field=True)
     sync = models.BooleanField(default=True)
     last_sync_date = models.DateTimeField(default=timezone.now)
-    wikipedia_link = fields.CharField(max_length=255, blank=True, null=True, deduplication_field=True)
+    wikipedia_link = fields.CharField(
+        max_length=255, blank=True, null=True, deduplication_field=True)
     # idk probably other keys would be useful here?
     born = fields.DateTimeField(blank=True, null=True)
     died = fields.DateTimeField(blank=True, null=True)
diff --git a/bookwyrm/models/base_model.py b/bookwyrm/models/base_model.py
index f44797ab..dd3065c9 100644
--- a/bookwyrm/models/base_model.py
+++ b/bookwyrm/models/base_model.py
@@ -44,6 +44,7 @@ class BookWyrmModel(models.Model):
 
 
 @receiver(models.signals.post_save)
+#pylint: disable=unused-argument
 def execute_after_save(sender, instance, created, *args, **kwargs):
     ''' set the remote_id after save (when the id is available) '''
     if not created or not hasattr(instance, 'get_remote_id'):
diff --git a/bookwyrm/models/import_job.py b/bookwyrm/models/import_job.py
index fe39325f..8b09216f 100644
--- a/bookwyrm/models/import_job.py
+++ b/bookwyrm/models/import_job.py
@@ -6,7 +6,6 @@ from django.db import models
 from django.utils import timezone
 
 from bookwyrm import books_manager
-from bookwyrm.connectors import ConnectorException
 from bookwyrm.models import ReadThrough, User, Book
 from bookwyrm.utils.fields import JSONField
 from .base_model import PrivacyLevels
diff --git a/bookwyrm/models/relationship.py b/bookwyrm/models/relationship.py
index 8913b9ab..debe2ace 100644
--- a/bookwyrm/models/relationship.py
+++ b/bookwyrm/models/relationship.py
@@ -37,7 +37,7 @@ class UserRelationship(ActivitypubMixin, BookWyrmModel):
 
     activity_serializer = activitypub.Follow
 
-    def get_remote_id(self, status=None):
+    def get_remote_id(self, status=None):# pylint: disable=arguments-differ
         ''' use shelf identifier in remote_id '''
         status = status or 'follows'
         base_path = self.user_subject.remote_id
diff --git a/bookwyrm/status.py b/bookwyrm/status.py
index 83a106e5..648f2e7d 100644
--- a/bookwyrm/status.py
+++ b/bookwyrm/status.py
@@ -1,7 +1,7 @@
 ''' Handle user activity '''
 from django.utils import timezone
 
-from bookwyrm import activitypub, books_manager, models
+from bookwyrm import models
 from bookwyrm.sanitize_html import InputHtmlParser
 
 

From 2b3daa022711e6e7181cf2574cf95dc59b8cbb27 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Sat, 12 Dec 2020 18:13:00 -0800
Subject: [PATCH 2/5] disable some warnings

---
 bookwyrm/context_processors.py | 2 +-
 bookwyrm/forms.py              | 3 ++-
 bookwyrm/goodreads_import.py   | 2 +-
 bookwyrm/incoming.py           | 3 ---
 4 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/bookwyrm/context_processors.py b/bookwyrm/context_processors.py
index 72839dce..a1471ac4 100644
--- a/bookwyrm/context_processors.py
+++ b/bookwyrm/context_processors.py
@@ -1,7 +1,7 @@
 ''' customize the info available in context for rendering templates '''
 from bookwyrm import models
 
-def site_settings(request):
+def site_settings(request):# pylint: disable=unused-argument
     ''' include the custom info about the site '''
     return {
         'site': models.SiteSettings.objects.get()
diff --git a/bookwyrm/forms.py b/bookwyrm/forms.py
index 784f1038..a2c3e24b 100644
--- a/bookwyrm/forms.py
+++ b/bookwyrm/forms.py
@@ -30,7 +30,7 @@ class CustomForm(ModelForm):
                 visible.field.widget.attrs['rows'] = None
             visible.field.widget.attrs['class'] = css_classes[input_type]
 
-
+# pylint: disable=missing-class-docstring
 class LoginForm(CustomForm):
     class Meta:
         model = models.User
@@ -131,6 +131,7 @@ class ImportForm(forms.Form):
 
 class ExpiryWidget(widgets.Select):
     def value_from_datadict(self, data, files, name):
+        ''' human-readable exiration time buckets '''
         selected_string = super().value_from_datadict(data, files, name)
 
         if selected_string == 'day':
diff --git a/bookwyrm/goodreads_import.py b/bookwyrm/goodreads_import.py
index 3fd330ab..93fc1c48 100644
--- a/bookwyrm/goodreads_import.py
+++ b/bookwyrm/goodreads_import.py
@@ -53,7 +53,7 @@ def import_data(job_id):
         for item in job.items.all():
             try:
                 item.resolve()
-            except Exception as e:
+            except Exception as e:# pylint: disable=broad-except
                 logger.exception(e)
                 item.fail_reason = 'Error loading book'
                 item.save()
diff --git a/bookwyrm/incoming.py b/bookwyrm/incoming.py
index bbbebf0f..4964d393 100644
--- a/bookwyrm/incoming.py
+++ b/bookwyrm/incoming.py
@@ -17,9 +17,6 @@ from bookwyrm.signatures import Signature
 @csrf_exempt
 def inbox(request, username):
     ''' incoming activitypub events '''
-    # TODO: should do some kind of checking if the user accepts
-    # this action from the sender probably? idk
-    # but this will just throw a 404 if the user doesn't exist
     try:
         models.User.objects.get(localname=username)
     except models.User.DoesNotExist:

From 1e08eeb4c295b79e8cce1aff6e411f32352ea442 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Sat, 12 Dec 2020 18:25:04 -0800
Subject: [PATCH 3/5] Renames custom template tags file

---
 bookwyrm/templates/author.html                            | 2 +-
 bookwyrm/templates/book.html                              | 2 +-
 bookwyrm/templates/editions.html                          | 2 +-
 bookwyrm/templates/feed.html                              | 2 +-
 bookwyrm/templates/followers.html                         | 2 +-
 bookwyrm/templates/following.html                         | 2 +-
 bookwyrm/templates/import_status.html                     | 2 +-
 bookwyrm/templates/layout.html                            | 2 +-
 bookwyrm/templates/notifications.html                     | 2 +-
 bookwyrm/templates/shelf.html                             | 2 +-
 bookwyrm/templates/snippets/avatar.html                   | 2 +-
 bookwyrm/templates/snippets/book_cover.html               | 2 +-
 bookwyrm/templates/snippets/book_preview.html             | 2 +-
 bookwyrm/templates/snippets/boost_button.html             | 2 +-
 bookwyrm/templates/snippets/cover_alt.html                | 2 +-
 bookwyrm/templates/snippets/create_status.html            | 2 +-
 bookwyrm/templates/snippets/fav_button.html               | 2 +-
 bookwyrm/templates/snippets/finish_reading_modal.html     | 2 +-
 bookwyrm/templates/snippets/follow_request_buttons.html   | 2 +-
 bookwyrm/templates/snippets/privacy_select.html           | 2 +-
 bookwyrm/templates/snippets/rate_action.html              | 2 +-
 bookwyrm/templates/snippets/reply_form.html               | 2 +-
 bookwyrm/templates/snippets/shelf.html                    | 2 +-
 bookwyrm/templates/snippets/shelve_button.html            | 2 +-
 bookwyrm/templates/snippets/status.html                   | 2 +-
 bookwyrm/templates/snippets/status_body.html              | 2 +-
 bookwyrm/templates/snippets/status_content.html           | 2 +-
 bookwyrm/templates/snippets/status_header.html            | 2 +-
 bookwyrm/templates/snippets/thread.html                   | 2 +-
 bookwyrm/templates/snippets/trimmed_text.html             | 2 +-
 bookwyrm/templates/snippets/user_header.html              | 2 +-
 bookwyrm/templates/snippets/username.html                 | 2 +-
 bookwyrm/templates/tag.html                               | 2 +-
 bookwyrm/templatetags/{fr_display.py => bookwyrm_tags.py} | 2 +-
 34 files changed, 34 insertions(+), 34 deletions(-)
 rename bookwyrm/templatetags/{fr_display.py => bookwyrm_tags.py} (98%)

diff --git a/bookwyrm/templates/author.html b/bookwyrm/templates/author.html
index 3e3e0018..9a7a20ab 100644
--- a/bookwyrm/templates/author.html
+++ b/bookwyrm/templates/author.html
@@ -1,5 +1,5 @@
 {% extends 'layout.html' %}
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% block content %}
 <div class="block">
     <h1 class="title">{{ author.name }}</h1>
diff --git a/bookwyrm/templates/book.html b/bookwyrm/templates/book.html
index 8b21b88c..10c2a27b 100644
--- a/bookwyrm/templates/book.html
+++ b/bookwyrm/templates/book.html
@@ -1,5 +1,5 @@
 {% extends 'layout.html' %}
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% load humanize %}
 {% block content %}
 
diff --git a/bookwyrm/templates/editions.html b/bookwyrm/templates/editions.html
index 273b2cd6..619ceafb 100644
--- a/bookwyrm/templates/editions.html
+++ b/bookwyrm/templates/editions.html
@@ -1,5 +1,5 @@
 {% extends 'layout.html' %}
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% block content %}
 <div class="block">
     <h1 class="title">Editions of <a href="/book/{{ work.id }}">"{{ work.title }}"</a></h1>
diff --git a/bookwyrm/templates/feed.html b/bookwyrm/templates/feed.html
index 6e49943a..07ad8d0f 100644
--- a/bookwyrm/templates/feed.html
+++ b/bookwyrm/templates/feed.html
@@ -1,5 +1,5 @@
 {% extends 'layout.html' %}
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% block content %}
 
 <div class="columns">
diff --git a/bookwyrm/templates/followers.html b/bookwyrm/templates/followers.html
index 645e46a1..00cb13ca 100644
--- a/bookwyrm/templates/followers.html
+++ b/bookwyrm/templates/followers.html
@@ -1,5 +1,5 @@
 {% extends 'layout.html' %}
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% block content %}
 <div class="block">
     <h1 class="title">
diff --git a/bookwyrm/templates/following.html b/bookwyrm/templates/following.html
index 2cca9127..478ca813 100644
--- a/bookwyrm/templates/following.html
+++ b/bookwyrm/templates/following.html
@@ -1,5 +1,5 @@
 {% extends 'layout.html' %}
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% block content %}
 <div class="block">
     <h1 class="title">
diff --git a/bookwyrm/templates/import_status.html b/bookwyrm/templates/import_status.html
index f91e2cce..6bb903b0 100644
--- a/bookwyrm/templates/import_status.html
+++ b/bookwyrm/templates/import_status.html
@@ -1,5 +1,5 @@
 {% extends 'layout.html' %}
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% load humanize %}
 {% block content %}
 <div class="block">
diff --git a/bookwyrm/templates/layout.html b/bookwyrm/templates/layout.html
index b37c9cda..bcbdca2a 100644
--- a/bookwyrm/templates/layout.html
+++ b/bookwyrm/templates/layout.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 <!DOCTYPE html>
 <html lang="en">
 <head>
diff --git a/bookwyrm/templates/notifications.html b/bookwyrm/templates/notifications.html
index f9494c31..f31df76d 100644
--- a/bookwyrm/templates/notifications.html
+++ b/bookwyrm/templates/notifications.html
@@ -1,6 +1,6 @@
 {% extends 'layout.html' %}
 {% load humanize %}
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% block content %}
 <div class="block">
     <h1 class="title">Notifications</h1>
diff --git a/bookwyrm/templates/shelf.html b/bookwyrm/templates/shelf.html
index d6842d13..390b9fc6 100644
--- a/bookwyrm/templates/shelf.html
+++ b/bookwyrm/templates/shelf.html
@@ -1,5 +1,5 @@
 {% extends 'layout.html' %}
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% block content %}
 
 <div class="columns">
diff --git a/bookwyrm/templates/snippets/avatar.html b/bookwyrm/templates/snippets/avatar.html
index cb0a12ea..da4b5fd2 100644
--- a/bookwyrm/templates/snippets/avatar.html
+++ b/bookwyrm/templates/snippets/avatar.html
@@ -1,3 +1,3 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 <img class="avatar image {% if large %}is-96x96{% else %}is-32x32{% endif %}" src="{% if user.avatar %}/images/{{ user.avatar }}{% else %}/static/images/default_avi.jpg{% endif %}" alt="avatar for {{ user|username }}">
 
diff --git a/bookwyrm/templates/snippets/book_cover.html b/bookwyrm/templates/snippets/book_cover.html
index 3fd83616..ceeef426 100644
--- a/bookwyrm/templates/snippets/book_cover.html
+++ b/bookwyrm/templates/snippets/book_cover.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 <div class="cover-container is-{{ size }}">
 {% if book.cover %}
     <img class="book-cover" src="/images/{{ book.cover }}" alt="{% include 'snippets/cover_alt.html' with book=book %}">
diff --git a/bookwyrm/templates/snippets/book_preview.html b/bookwyrm/templates/snippets/book_preview.html
index c675c45f..e7eca455 100644
--- a/bookwyrm/templates/snippets/book_preview.html
+++ b/bookwyrm/templates/snippets/book_preview.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 <div class="columns">
     <div class="column is-narrow">
         <div>
diff --git a/bookwyrm/templates/snippets/boost_button.html b/bookwyrm/templates/snippets/boost_button.html
index 57765bed..bf06cef7 100644
--- a/bookwyrm/templates/snippets/boost_button.html
+++ b/bookwyrm/templates/snippets/boost_button.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% with status.id|uuid as uuid %}
 <form name="boost" action="/boost/{{ status.id }}" method="post" onsubmit="return interact(event)" class="boost-{{ status.id }}-{{ uuid }} {% if request.user|boosted:status  %}hidden{% endif %}" data-id="boost-{{ status.id }}-{{ uuid }}">
     {% csrf_token %}
diff --git a/bookwyrm/templates/snippets/cover_alt.html b/bookwyrm/templates/snippets/cover_alt.html
index 52bd52f2..0cccc2e1 100644
--- a/bookwyrm/templates/snippets/cover_alt.html
+++ b/bookwyrm/templates/snippets/cover_alt.html
@@ -1,2 +1,2 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 '{{ book.title }}' Cover ({{ book|edition_info }})
diff --git a/bookwyrm/templates/snippets/create_status.html b/bookwyrm/templates/snippets/create_status.html
index e36b1b19..ac8c0b75 100644
--- a/bookwyrm/templates/snippets/create_status.html
+++ b/bookwyrm/templates/snippets/create_status.html
@@ -1,5 +1,5 @@
 {% load humanize %}
-{% load fr_display %}
+{% load bookwyrm_tags %}
 
 <div class="tabs is-boxed">
     <ul role="tablist">
diff --git a/bookwyrm/templates/snippets/fav_button.html b/bookwyrm/templates/snippets/fav_button.html
index 58ece1e4..11e03cdb 100644
--- a/bookwyrm/templates/snippets/fav_button.html
+++ b/bookwyrm/templates/snippets/fav_button.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% with status.id|uuid as uuid %}
 <form name="favorite" action="/favorite/{{ status.id }}" method="POST" onsubmit="return interact(event)" class="fav-{{ status.id }}-{{ uuid }} {% if request.user|liked:status %}hidden{% endif %}" data-id="fav-{{ status.id }}-{{ uuid }}">
     {% csrf_token %}
diff --git a/bookwyrm/templates/snippets/finish_reading_modal.html b/bookwyrm/templates/snippets/finish_reading_modal.html
index 8f8aeeff..d04d508d 100644
--- a/bookwyrm/templates/snippets/finish_reading_modal.html
+++ b/bookwyrm/templates/snippets/finish_reading_modal.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 <div>
     <input class="toggle-control" type="checkbox" name="finish-reading-{{ uuid }}" id="finish-reading-{{ uuid }}">
     <div class="modal toggle-content hidden">
diff --git a/bookwyrm/templates/snippets/follow_request_buttons.html b/bookwyrm/templates/snippets/follow_request_buttons.html
index e1b80631..b5a64cc8 100644
--- a/bookwyrm/templates/snippets/follow_request_buttons.html
+++ b/bookwyrm/templates/snippets/follow_request_buttons.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% if request.user|follow_request_exists:user %}
 <form action="/accept-follow-request/" method="POST">
     {% csrf_token %}
diff --git a/bookwyrm/templates/snippets/privacy_select.html b/bookwyrm/templates/snippets/privacy_select.html
index c8d974c0..dbe11c43 100644
--- a/bookwyrm/templates/snippets/privacy_select.html
+++ b/bookwyrm/templates/snippets/privacy_select.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 <div class="select">
     {% with 0|uuid as uuid %}
     {% if not no_label %}
diff --git a/bookwyrm/templates/snippets/rate_action.html b/bookwyrm/templates/snippets/rate_action.html
index 39154017..b9c443ce 100644
--- a/bookwyrm/templates/snippets/rate_action.html
+++ b/bookwyrm/templates/snippets/rate_action.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 <span class="is-sr-only">Leave a rating</span>
 <div class="field is-grouped stars rate-stars">
     {% for i in '12345'|make_list %}
diff --git a/bookwyrm/templates/snippets/reply_form.html b/bookwyrm/templates/snippets/reply_form.html
index 2d8abd23..40cf6de1 100644
--- a/bookwyrm/templates/snippets/reply_form.html
+++ b/bookwyrm/templates/snippets/reply_form.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% with activity.id|uuid as uuid %}
 <form class="is-flex-grow-1" name="reply" action="/reply" method="post" onsubmit="return reply(event)">
     <div class="columns">
diff --git a/bookwyrm/templates/snippets/shelf.html b/bookwyrm/templates/snippets/shelf.html
index 4e41cd30..0d05e661 100644
--- a/bookwyrm/templates/snippets/shelf.html
+++ b/bookwyrm/templates/snippets/shelf.html
@@ -1,5 +1,5 @@
 {% load humanize %}
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% if shelf.books.all|length > 0 %}
 <table class="table is-striped is-fullwidth">
 
diff --git a/bookwyrm/templates/snippets/shelve_button.html b/bookwyrm/templates/snippets/shelve_button.html
index 857f0f9c..d452169e 100644
--- a/bookwyrm/templates/snippets/shelve_button.html
+++ b/bookwyrm/templates/snippets/shelve_button.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% if request.user.is_authenticated %}
 
 {% with book.id|uuid as uuid %}
diff --git a/bookwyrm/templates/snippets/status.html b/bookwyrm/templates/snippets/status.html
index a7f6a392..13d5ee7c 100644
--- a/bookwyrm/templates/snippets/status.html
+++ b/bookwyrm/templates/snippets/status.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% if not status.deleted %}
     {% if status.status_type == 'Boost' %}
         {% include 'snippets/avatar.html' with user=status.user %}
diff --git a/bookwyrm/templates/snippets/status_body.html b/bookwyrm/templates/snippets/status_body.html
index 4ef763fb..e2022247 100644
--- a/bookwyrm/templates/snippets/status_body.html
+++ b/bookwyrm/templates/snippets/status_body.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% load humanize %}
 
 {% if not status.deleted %}
diff --git a/bookwyrm/templates/snippets/status_content.html b/bookwyrm/templates/snippets/status_content.html
index e71588ad..2ef06421 100644
--- a/bookwyrm/templates/snippets/status_content.html
+++ b/bookwyrm/templates/snippets/status_content.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 <div class="block">
     {% if status.status_type == 'Review' %}
     <h3>
diff --git a/bookwyrm/templates/snippets/status_header.html b/bookwyrm/templates/snippets/status_header.html
index 1e4179ad..890b4ab8 100644
--- a/bookwyrm/templates/snippets/status_header.html
+++ b/bookwyrm/templates/snippets/status_header.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% include 'snippets/avatar.html' with user=status.user %}
 {% include 'snippets/username.html' with user=status.user %}
 
diff --git a/bookwyrm/templates/snippets/thread.html b/bookwyrm/templates/snippets/thread.html
index ae42c51a..589702ce 100644
--- a/bookwyrm/templates/snippets/thread.html
+++ b/bookwyrm/templates/snippets/thread.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 <div class="block">
 
 {% with depth=depth|add:1 %}
diff --git a/bookwyrm/templates/snippets/trimmed_text.html b/bookwyrm/templates/snippets/trimmed_text.html
index 9ed8fa87..43fcb38e 100644
--- a/bookwyrm/templates/snippets/trimmed_text.html
+++ b/bookwyrm/templates/snippets/trimmed_text.html
@@ -1,4 +1,4 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% with 0|uuid as uuid %}
 {% if full %}
 
diff --git a/bookwyrm/templates/snippets/user_header.html b/bookwyrm/templates/snippets/user_header.html
index d9ca71b2..f4f9733f 100644
--- a/bookwyrm/templates/snippets/user_header.html
+++ b/bookwyrm/templates/snippets/user_header.html
@@ -1,5 +1,5 @@
 {% load humanize %}
-{% load fr_display %}
+{% load bookwyrm_tags %}
 <div class="block">
     <div class="columns">
         <div class="column is-narrow">
diff --git a/bookwyrm/templates/snippets/username.html b/bookwyrm/templates/snippets/username.html
index ec621f60..fbd6ac85 100644
--- a/bookwyrm/templates/snippets/username.html
+++ b/bookwyrm/templates/snippets/username.html
@@ -1,2 +1,2 @@
-{% load fr_display %}
+{% load bookwyrm_tags %}
 <a href="/user/{{ user | username }}" class="user">{% if user.name %}{{ user.name }}{% else %}{{ user | username }}{% endif %}</a>{% if possessive %}'s{% endif %}{% if show_full and user.name or show_full and user.localname %} ({{ user.username }}){% endif %}
diff --git a/bookwyrm/templates/tag.html b/bookwyrm/templates/tag.html
index c6641385..7eea0c21 100644
--- a/bookwyrm/templates/tag.html
+++ b/bookwyrm/templates/tag.html
@@ -1,5 +1,5 @@
 {% extends 'layout.html' %}
-{% load fr_display %}
+{% load bookwyrm_tags %}
 {% block content %}
 
 <div class="block">
diff --git a/bookwyrm/templatetags/fr_display.py b/bookwyrm/templatetags/bookwyrm_tags.py
similarity index 98%
rename from bookwyrm/templatetags/fr_display.py
rename to bookwyrm/templatetags/bookwyrm_tags.py
index 690b4c64..8c77921c 100644
--- a/bookwyrm/templatetags/fr_display.py
+++ b/bookwyrm/templatetags/bookwyrm_tags.py
@@ -116,6 +116,7 @@ def get_book_description(book):
     ''' use the work's text if the book doesn't have it '''
     return book.description or book.parent_work.description
 
+
 @register.filter(name='text_overflow')
 def text_overflow(text):
     ''' dont' let book descriptions run for ages '''
@@ -160,7 +161,6 @@ def time_since(date):
 @register.simple_tag(takes_context=True)
 def active_shelf(context, book):
     ''' check what shelf a user has a book on, if any '''
-    #TODO: books can be on multiple shelves, handle that better
     shelf = models.ShelfBook.objects.filter(
         shelf__user=context['request'].user,
         book=book

From b61a7cccff8ddd9a2b4450d051344f72500b98be Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Sat, 12 Dec 2020 19:34:33 -0800
Subject: [PATCH 4/5] Use trimmedwords_html instead of a custom filter

---
 bookwyrm/templates/snippets/trimmed_text.html |   2 +-
 bookwyrm/templatetags/bookwyrm_tags.py        |  17 +-
 bookwyrm/tests/test_templatetags.py           | 209 ++++++++++++++++++
 3 files changed, 211 insertions(+), 17 deletions(-)
 create mode 100644 bookwyrm/tests/test_templatetags.py

diff --git a/bookwyrm/templates/snippets/trimmed_text.html b/bookwyrm/templates/snippets/trimmed_text.html
index 43fcb38e..b3207a92 100644
--- a/bookwyrm/templates/snippets/trimmed_text.html
+++ b/bookwyrm/templates/snippets/trimmed_text.html
@@ -2,7 +2,7 @@
 {% with 0|uuid as uuid %}
 {% if full %}
 
-{% with full|text_overflow as trimmed %}
+{% with full|truncatewords_html:60 as trimmed %}
 {% if trimmed != full %}
 <div>
     <input type="radio" name="show-hide-{{ uuid }}" id="show-{{ uuid }}" class="toggle-control" checked>
diff --git a/bookwyrm/templatetags/bookwyrm_tags.py b/bookwyrm/templatetags/bookwyrm_tags.py
index 8c77921c..deb90427 100644
--- a/bookwyrm/templatetags/bookwyrm_tags.py
+++ b/bookwyrm/templatetags/bookwyrm_tags.py
@@ -13,7 +13,7 @@ register = template.Library()
 
 @register.filter(name='dict_key')
 def dict_key(d, k):
-    '''Returns the given key from a dictionary.'''
+    ''' Returns the given key from a dictionary. '''
     return d.get(k) or 0
 
 
@@ -117,21 +117,6 @@ def get_book_description(book):
     return book.description or book.parent_work.description
 
 
-@register.filter(name='text_overflow')
-def text_overflow(text):
-    ''' dont' let book descriptions run for ages '''
-    if not text:
-        return ''
-    char_max = 400
-    if text and len(text) < char_max:
-        return text
-
-    trimmed = text[:char_max]
-    # go back to the last space
-    trimmed = ' '.join(trimmed.split(' ')[:-1])
-    return trimmed + '...'
-
-
 @register.filter(name='uuid')
 def get_uuid(identifier):
     ''' for avoiding clashing ids when there are many forms '''
diff --git a/bookwyrm/tests/test_templatetags.py b/bookwyrm/tests/test_templatetags.py
new file mode 100644
index 00000000..28f08424
--- /dev/null
+++ b/bookwyrm/tests/test_templatetags.py
@@ -0,0 +1,209 @@
+''' style fixes and lookups for templates '''
+import re
+from unittest.mock import patch
+
+from dateutil.parser import parse
+from django.test import TestCase
+from django.utils import timezone
+
+from bookwyrm import models
+from bookwyrm.templatetags import bookwyrm_tags
+
+
+class TemplateTags(TestCase):
+    ''' lotta different things here '''
+    def setUp(self):
+        ''' create some filler objects '''
+        self.user = models.User.objects.create_user(
+            'mouse', 'mouse@mouse.mouse', 'mouseword', local=True)
+        with patch('bookwyrm.models.user.set_remote_server.delay'):
+            self.remote_user = models.User.objects.create_user(
+                'rat', 'rat@rat.rat', 'ratword',
+                remote_id='http://example.com/rat', local=False)
+        self.book = models.Edition.objects.create(title='Test Book')
+
+
+    def test_dict_key(self):
+        ''' just getting a value out of a dict '''
+        test_dict = {'a': 1, 'b': 3}
+        self.assertEqual(
+            bookwyrm_tags.dict_key(test_dict, 'a'), 1)
+        self.assertEqual(
+            bookwyrm_tags.dict_key(test_dict, 'c'), 0)
+
+
+    def test_get_rating(self):
+        ''' get a user's most recent rating of a book '''
+        models.Review.objects.create(
+            user=self.user, book=self.book, rating=3)
+        self.assertEqual(
+            bookwyrm_tags.get_rating(self.book, self.user), 3)
+
+
+    def test_get_rating_doesnt_exist(self):
+        ''' there is no rating available '''
+        self.assertEqual(
+            bookwyrm_tags.get_rating(self.book, self.user), 0)
+
+
+    def test_get_user_identifer_local(self):
+        ''' fall back to the simplest uid available '''
+        self.assertNotEqual(self.user.username, self.user.localname)
+        self.assertEqual(
+            bookwyrm_tags.get_user_identifier(self.user), 'mouse')
+
+    def test_get_user_identifer_remote(self):
+        ''' for a remote user, should be their full username '''
+        self.assertEqual(
+            bookwyrm_tags.get_user_identifier(self.remote_user),
+            'rat@example.com')
+
+    def test_get_notification_count(self):
+        ''' just countin' '''
+        self.assertEqual(bookwyrm_tags.get_notification_count(self.user), 0)
+
+        models.Notification.objects.create(
+            user=self.user, notification_type='FOLLOW')
+        models.Notification.objects.create(
+            user=self.user, notification_type='FOLLOW')
+
+        models.Notification.objects.create(
+            user=self.remote_user, notification_type='FOLLOW')
+
+        self.assertEqual(bookwyrm_tags.get_notification_count(self.user), 2)
+
+
+    def test_get_replies(self):
+        ''' direct replies to a status '''
+        parent = models.Review.objects.create(
+            user=self.user, book=self.book)
+        first_child = models.Status.objects.create(
+            reply_parent=parent, user=self.user)
+        second_child = models.Status.objects.create(
+            reply_parent=parent, user=self.user)
+        third_child = models.Status.objects.create(
+            reply_parent=parent, user=self.user, deleted=True)
+
+        replies = bookwyrm_tags.get_replies(parent)
+        self.assertEqual(len(replies), 2)
+        self.assertTrue(first_child in replies)
+        self.assertTrue(second_child in replies)
+        self.assertFalse(third_child in replies)
+
+
+    def test_get_parent(self):
+        ''' get the reply parent of a status '''
+        parent = models.Review.objects.create(
+            user=self.user, book=self.book)
+        child = models.Status.objects.create(
+            reply_parent=parent, user=self.user)
+
+        result = bookwyrm_tags.get_parent(child)
+        self.assertEqual(result, parent)
+        self.assertIsInstance(result, models.Review)
+
+
+    def test_get_user_liked(self):
+        ''' did a user like a status '''
+        status = models.Review.objects.create(
+            user=self.remote_user, book=self.book)
+
+        self.assertFalse(bookwyrm_tags.get_user_liked(self.user, status))
+        models.Favorite.objects.create(
+            user=self.user,
+            status=status
+        )
+        self.assertTrue(bookwyrm_tags.get_user_liked(self.user, status))
+
+
+    def test_get_user_boosted(self):
+        ''' did a user boost a status '''
+        status = models.Review.objects.create(
+            user=self.remote_user, book=self.book)
+
+        self.assertFalse(bookwyrm_tags.get_user_boosted(self.user, status))
+        models.Boost.objects.create(
+            user=self.user,
+            boosted_status=status
+        )
+        self.assertTrue(bookwyrm_tags.get_user_boosted(self.user, status))
+
+
+    def test_follow_request_exists(self):
+        ''' does a user want to follow '''
+        self.assertFalse(
+            bookwyrm_tags.follow_request_exists(self.user, self.remote_user))
+
+        models.UserFollowRequest.objects.create(
+            user_subject=self.user,
+            user_object=self.remote_user)
+
+        self.assertFalse(
+            bookwyrm_tags.follow_request_exists(self.user, self.remote_user))
+        self.assertTrue(
+            bookwyrm_tags.follow_request_exists(self.remote_user, self.user))
+
+
+    def test_get_boosted(self):
+        ''' load a boosted status '''
+        status = models.Review.objects.create(
+            user=self.remote_user, book=self.book)
+        boost = models.Boost.objects.create(
+            user=self.user,
+            boosted_status=status
+        )
+        boosted = bookwyrm_tags.get_boosted(boost)
+        self.assertIsInstance(boosted, models.Review)
+        self.assertEqual(boosted, status)
+
+
+    def test_get_edition_info(self):
+        ''' text slug about an edition '''
+        self.assertEqual(
+            bookwyrm_tags.get_edition_info(self.book), '')
+
+        self.book.physical_format = 'worm'
+        self.book.save()
+        self.assertEqual(
+            bookwyrm_tags.get_edition_info(self.book), 'worm')
+
+        self.book.languages = ['English']
+        self.book.save()
+        self.assertEqual(
+            bookwyrm_tags.get_edition_info(self.book), 'worm')
+
+        self.book.languages = ['Glorbish', 'English']
+        self.book.save()
+        self.assertEqual(
+            bookwyrm_tags.get_edition_info(self.book),
+            'worm, Glorbish language')
+
+        self.book.published_date = timezone.make_aware(parse('2020'))
+        self.book.save()
+        self.assertEqual(
+            bookwyrm_tags.get_edition_info(self.book),
+            'worm, Glorbish language, 2020')
+
+
+    def test_get_book_description(self):
+        ''' grab it from the edition or the parent '''
+        work = models.Work.objects.create(title='Test Work')
+        self.book.parent_work = work
+        self.book.save()
+
+        self.assertIsNone(bookwyrm_tags.get_book_description(self.book))
+
+        work.description = 'hi'
+        work.save()
+        self.assertEqual(bookwyrm_tags.get_book_description(self.book), 'hi')
+
+        self.book.description = 'hello'
+        self.book.save()
+        self.assertEqual(bookwyrm_tags.get_book_description(self.book), 'hello')
+
+
+    def test_get_uuid(self):
+        ''' uuid functionality '''
+        uuid = bookwyrm_tags.get_uuid('hi')
+        print(uuid)
+        self.assertTrue(re.match(r'hi[A-Za-z0-9\-]', uuid))

From 47981dc15e59a7f0a2f5442dcd985131c888b947 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Sat, 12 Dec 2020 20:02:14 -0800
Subject: [PATCH 5/5] Show year in time_since when ambiguous

---
 bookwyrm/templatetags/bookwyrm_tags.py |  2 ++
 bookwyrm/tests/test_templatetags.py    | 35 +++++++++++++++++++++++++-
 2 files changed, 36 insertions(+), 1 deletion(-)

diff --git a/bookwyrm/templatetags/bookwyrm_tags.py b/bookwyrm/templatetags/bookwyrm_tags.py
index deb90427..849974bf 100644
--- a/bookwyrm/templatetags/bookwyrm_tags.py
+++ b/bookwyrm/templatetags/bookwyrm_tags.py
@@ -132,6 +132,8 @@ def time_since(date):
     delta = now - date
 
     if date < (now - relativedelta(weeks=1)):
+        if date.year != now.year:
+            return date.strftime('%b %-d %Y')
         return date.strftime('%b %-d')
     delta = relativedelta(now, date)
     if delta.days:
diff --git a/bookwyrm/tests/test_templatetags.py b/bookwyrm/tests/test_templatetags.py
index 28f08424..67d28848 100644
--- a/bookwyrm/tests/test_templatetags.py
+++ b/bookwyrm/tests/test_templatetags.py
@@ -3,6 +3,7 @@ import re
 from unittest.mock import patch
 
 from dateutil.parser import parse
+from dateutil.relativedelta import relativedelta
 from django.test import TestCase
 from django.utils import timezone
 
@@ -205,5 +206,37 @@ class TemplateTags(TestCase):
     def test_get_uuid(self):
         ''' uuid functionality '''
         uuid = bookwyrm_tags.get_uuid('hi')
-        print(uuid)
         self.assertTrue(re.match(r'hi[A-Za-z0-9\-]', uuid))
+
+
+    def test_time_since(self):
+        ''' ultraconcise timestamps '''
+        self.assertEqual(bookwyrm_tags.time_since('bleh'), '')
+
+        now = timezone.now()
+        self.assertEqual(bookwyrm_tags.time_since(now), '0s')
+
+        seconds_ago = now - relativedelta(seconds=4)
+        self.assertEqual(bookwyrm_tags.time_since(seconds_ago), '4s')
+
+        minutes_ago = now - relativedelta(minutes=8)
+        self.assertEqual(bookwyrm_tags.time_since(minutes_ago), '8m')
+
+        hours_ago = now - relativedelta(hours=9)
+        self.assertEqual(bookwyrm_tags.time_since(hours_ago), '9h')
+
+        days_ago = now - relativedelta(days=3)
+        self.assertEqual(bookwyrm_tags.time_since(days_ago), '3d')
+
+        # I am not going to figure out how to mock dates tonight.
+        months_ago = now - relativedelta(months=5)
+        self.assertTrue(re.match(
+            r'[A-Z][a-z]{2} \d?\d',
+            bookwyrm_tags.time_since(months_ago)
+        ))
+
+        years_ago = now - relativedelta(years=10)
+        self.assertTrue(re.match(
+            r'[A-Z][a-z]{2} \d?\d \d{4}',
+            bookwyrm_tags.time_since(years_ago)
+        ))