From 6243cf0e4a763aacb5586338fbf6ef1127aa5349 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Mon, 26 Oct 2020 14:33:02 -0700
Subject: [PATCH 1/8] uses enum for post privacy database field

---
 bookwyrm/migrations/0057_auto_20201026_2131.py | 18 ++++++++++++++++++
 bookwyrm/models/status.py                      | 13 ++++++++++++-
 2 files changed, 30 insertions(+), 1 deletion(-)
 create mode 100644 bookwyrm/migrations/0057_auto_20201026_2131.py

diff --git a/bookwyrm/migrations/0057_auto_20201026_2131.py b/bookwyrm/migrations/0057_auto_20201026_2131.py
new file mode 100644
index 00000000..cc414e98
--- /dev/null
+++ b/bookwyrm/migrations/0057_auto_20201026_2131.py
@@ -0,0 +1,18 @@
+# Generated by Django 3.0.7 on 2020-10-26 21:31
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('bookwyrm', '0056_auto_20201021_0150'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='status',
+            name='privacy',
+            field=models.CharField(choices=[('public', 'Public'), ('unlisted', 'Unlisted'), ('followers', 'Followers'), ('direct', 'Direct')], default='public', max_length=255),
+        ),
+    ]
diff --git a/bookwyrm/models/status.py b/bookwyrm/models/status.py
index 61e9c500..dfc39194 100644
--- a/bookwyrm/models/status.py
+++ b/bookwyrm/models/status.py
@@ -10,6 +10,13 @@ from .base_model import ActivitypubMixin, OrderedCollectionPageMixin
 from .base_model import ActivityMapping, BookWyrmModel
 
 
+PrivacyLevels = models.TextChoices('Privacy', [
+    'public',
+    'unlisted',
+    'followers',
+    'direct'
+])
+
 class Status(OrderedCollectionPageMixin, BookWyrmModel):
     ''' any post, like a reply to a review, etc '''
     user = models.ForeignKey('User', on_delete=models.PROTECT)
@@ -18,7 +25,11 @@ class Status(OrderedCollectionPageMixin, BookWyrmModel):
     mention_books = models.ManyToManyField(
         'Edition', related_name='mention_book')
     local = models.BooleanField(default=True)
-    privacy = models.CharField(max_length=255, default='public')
+    privacy = models.CharField(
+        max_length=255,
+        default='public',
+        choices=PrivacyLevels.choices
+    )
     sensitive = models.BooleanField(default=False)
     # the created date can't be this, because of receiving federated posts
     published_date = models.DateTimeField(default=timezone.now)

From 2afa111b705eb319e3822e70d9c2f3ce0776d289 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Mon, 26 Oct 2020 15:00:15 -0700
Subject: [PATCH 2/8] Create statuses from django form

---
 bookwyrm/outgoing.py     | 59 +++++++++++-----------------------------
 bookwyrm/status.py       |  1 +
 bookwyrm/view_actions.py | 48 ++++++--------------------------
 3 files changed, 26 insertions(+), 82 deletions(-)

diff --git a/bookwyrm/outgoing.py b/bookwyrm/outgoing.py
index f496a211..118f90fe 100644
--- a/bookwyrm/outgoing.py
+++ b/bookwyrm/outgoing.py
@@ -9,9 +9,8 @@ import requests
 from bookwyrm import activitypub
 from bookwyrm import models
 from bookwyrm.broadcast import broadcast
-from bookwyrm.status import create_review, create_status
-from bookwyrm.status import create_quotation, create_comment
-from bookwyrm.status import create_tag, create_notification, create_rating
+from bookwyrm.status import create_status
+from bookwyrm.status import create_tag, create_notification
 from bookwyrm.status import create_generated_note
 from bookwyrm.status import delete_status
 from bookwyrm.remote_user import get_or_create_remote_user
@@ -178,16 +177,18 @@ def handle_import_books(user, items):
                 broadcast(user, activity)
 
                 if item.rating or item.review:
-                    review_title = "Review of {!r} on Goodreads".format(
-                        item.book.title,
-                    ) if item.review else ""
-                    handle_review(
-                        user,
-                        item.book,
-                        review_title,
-                        item.review,
-                        item.rating,
-                    )
+                    pass
+                    #review_title = "Review of {!r} on Goodreads".format(
+                    #    item.book.title,
+                    #) if item.review else ""
+                    # TODO
+                    #handle_review(
+                    #    user,
+                    #    item.book,
+                    #    review_title,
+                    #    item.review,
+                    #    item.rating,
+                    #)
                 for read in item.reads:
                     read.book = item.book
                     read.user = user
@@ -209,37 +210,9 @@ def handle_delete_status(user, status):
     broadcast(user, status.to_activity())
 
 
-def handle_rate(user, book, rating):
-    ''' a review that's just a rating '''
-    builder = create_rating
-    handle_status(user, book, builder, rating)
-
-
-def handle_review(user, book, name, content, rating):
-    ''' post a review '''
-    # validated and saves the review in the database so it has an id
-    builder = create_review
-    handle_status(user, book, builder, name, content, rating)
-
-
-def handle_quotation(user, book, content, quote):
-    ''' post a review '''
-    # validated and saves the review in the database so it has an id
-    builder = create_quotation
-    handle_status(user, book, builder, content, quote)
-
-
-def handle_comment(user, book, content):
-    ''' post a comment '''
-    # validated and saves the review in the database so it has an id
-    builder = create_comment
-    handle_status(user, book, builder, content)
-
-
-def handle_status(user, book_id, builder, *args):
+def handle_status(user, form):
     ''' generic handler for statuses '''
-    book = models.Edition.objects.get(id=book_id)
-    status = builder(user, book, *args)
+    status = form.save()
 
     broadcast(user, status.to_create_activity(user), software='bookwyrm')
 
diff --git a/bookwyrm/status.py b/bookwyrm/status.py
index 25619839..57c800e5 100644
--- a/bookwyrm/status.py
+++ b/bookwyrm/status.py
@@ -13,6 +13,7 @@ def delete_status(status):
     status.deleted_date = datetime.now()
     status.save()
 
+
 def create_rating(user, book, rating):
     ''' a review that's just a rating '''
     if not rating or rating < 1 or rating > 5:
diff --git a/bookwyrm/view_actions.py b/bookwyrm/view_actions.py
index 54ed353a..e8a3b1fc 100644
--- a/bookwyrm/view_actions.py
+++ b/bookwyrm/view_actions.py
@@ -296,67 +296,37 @@ def shelve(request):
 def rate(request):
     ''' just a star rating for a book '''
     form = forms.RatingForm(request.POST)
-    book_id = request.POST.get('book')
-    # TODO: better failure behavior
-    if not form.is_valid():
-        return redirect('/book/%s' % book_id)
-
-    rating = form.cleaned_data.get('rating')
-    # throws a value error if the book is not found
-
-    outgoing.handle_rate(request.user, book_id, rating)
-    return redirect('/book/%s' % book_id)
+    return handle_status(request, form)
 
 
 @login_required
 def review(request):
     ''' create a book review '''
     form = forms.ReviewForm(request.POST)
-    book_id = request.POST.get('book')
-    if not form.is_valid():
-        return redirect('/book/%s' % book_id)
-
-    # TODO: validation, htmlification
-    name = form.cleaned_data.get('name')
-    content = form.cleaned_data.get('content')
-    rating = form.data.get('rating', None)
-    try:
-        rating = int(rating)
-    except ValueError:
-        rating = None
-
-    outgoing.handle_review(request.user, book_id, name, content, rating)
-    return redirect('/book/%s' % book_id)
+    return handle_status(request, form)
 
 
 @login_required
 def quotate(request):
     ''' create a book quotation '''
     form = forms.QuotationForm(request.POST)
-    book_id = request.POST.get('book')
-    if not form.is_valid():
-        return redirect('/book/%s' % book_id)
-
-    quote = form.cleaned_data.get('quote')
-    content = form.cleaned_data.get('content')
-
-    outgoing.handle_quotation(request.user, book_id, content, quote)
-    return redirect('/book/%s' % book_id)
+    return handle_status(request, form)
 
 
 @login_required
 def comment(request):
     ''' create a book comment '''
     form = forms.CommentForm(request.POST)
+    return handle_status(request, form)
+
+
+def handle_status(request, form):
+    ''' all the review/comment/quote etc functions are the same '''
     book_id = request.POST.get('book')
-    # TODO: better failure behavior
     if not form.is_valid():
         return redirect('/book/%s' % book_id)
 
-    # TODO: validation, htmlification
-    content = form.data.get('content')
-
-    outgoing.handle_comment(request.user, book_id, content)
+    outgoing.handle_status(request.user, form)
     return redirect('/book/%s' % book_id)
 
 

From 39b9fe8f4a671bad786e757ea85fd5757184bdc2 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Mon, 26 Oct 2020 15:10:32 -0700
Subject: [PATCH 3/8] Fixes serializing reviews with no rating

---
 bookwyrm/models/status.py | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/bookwyrm/models/status.py b/bookwyrm/models/status.py
index dfc39194..b1283c9d 100644
--- a/bookwyrm/models/status.py
+++ b/bookwyrm/models/status.py
@@ -192,9 +192,14 @@ class Review(Status):
     @property
     def ap_pure_name(self):
         ''' clarify review names for mastodon serialization '''
-        return 'Review of "%s" (%d stars): %s' % (
+        if self.rating:
+            return 'Review of "%s" (%d stars): %s' % (
+                self.book.title,
+                self.rating,
+                self.name
+            )
+        return 'Review of "%s": %s' % (
             self.book.title,
-            self.rating,
             self.name
         )
 

From b7061c0f4d13576729a568d647be62e22ab40d4b Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Tue, 27 Oct 2020 11:32:15 -0700
Subject: [PATCH 4/8] Fixes create status forms

---
 bookwyrm/forms.py                             | 26 +++-------------
 bookwyrm/outgoing.py                          | 31 +++++++------------
 .../templates/snippets/create_status.html     |  8 +++++
 bookwyrm/templates/snippets/interaction.html  |  4 ++-
 bookwyrm/view_actions.py                      | 26 ++++++----------
 5 files changed, 38 insertions(+), 57 deletions(-)

diff --git a/bookwyrm/forms.py b/bookwyrm/forms.py
index 7b18a2ff..0c3b19ba 100644
--- a/bookwyrm/forms.py
+++ b/bookwyrm/forms.py
@@ -52,47 +52,31 @@ class RegisterForm(CustomForm):
 class RatingForm(CustomForm):
     class Meta:
         model = models.Review
-        fields = ['rating']
+        fields = ['user', 'book', 'content', 'rating', 'privacy']
 
 
 class ReviewForm(CustomForm):
     class Meta:
         model = models.Review
-        fields = ['name', 'content']
-        help_texts = {f: None for f in fields}
-        labels = {
-            'name': 'Title',
-            'content': 'Review',
-        }
+        fields = ['user', 'book', 'name', 'content', 'rating', 'privacy']
 
 
 class CommentForm(CustomForm):
     class Meta:
         model = models.Comment
-        fields = ['content']
-        help_texts = {f: None for f in fields}
-        labels = {
-            'content': 'Comment',
-        }
+        fields = ['user', 'book', 'content', 'privacy']
 
 
 class QuotationForm(CustomForm):
     class Meta:
         model = models.Quotation
-        fields = ['quote', 'content']
-        help_texts = {f: None for f in fields}
-        labels = {
-            'quote': 'Quote',
-            'content': 'Comment',
-        }
+        fields = ['user', 'book', 'quote', 'content', 'privacy']
 
 
 class ReplyForm(CustomForm):
     class Meta:
         model = models.Status
-        fields = ['content']
-        help_texts = {f: None for f in fields}
-        labels = {'content': 'Comment'}
+        fields = ['user', 'content', 'reply_parent', 'privacy']
 
 
 class EditUserForm(CustomForm):
diff --git a/bookwyrm/outgoing.py b/bookwyrm/outgoing.py
index 118f90fe..bcdf7170 100644
--- a/bookwyrm/outgoing.py
+++ b/bookwyrm/outgoing.py
@@ -9,7 +9,6 @@ import requests
 from bookwyrm import activitypub
 from bookwyrm import models
 from bookwyrm.broadcast import broadcast
-from bookwyrm.status import create_status
 from bookwyrm.status import create_tag, create_notification
 from bookwyrm.status import create_generated_note
 from bookwyrm.status import delete_status
@@ -214,12 +213,21 @@ def handle_status(user, form):
     ''' generic handler for statuses '''
     status = form.save()
 
+    # notify reply parent or (TODO) tagged users
+    if status.reply_parent and status.reply_parent.user.local:
+        create_notification(
+            status.reply_parent.user,
+            'REPLY',
+            related_user=user,
+            related_status=status
+        )
+
     broadcast(user, status.to_create_activity(user), software='bookwyrm')
 
     # re-format the activity for non-bookwyrm servers
-    remote_activity = status.to_create_activity(user, pure=True)
-
-    broadcast(user, remote_activity, software='other')
+    if hasattr(status, 'pure_activity_serializer'):
+        remote_activity = status.to_create_activity(user, pure=True)
+        broadcast(user, remote_activity, software='other')
 
 
 def handle_tag(user, book, name):
@@ -238,21 +246,6 @@ def handle_untag(user, book, name):
     broadcast(user, tag_activity)
 
 
-def handle_reply(user, review, content):
-    ''' respond to a review or status '''
-    # validated and saves the comment in the database so it has an id
-    reply = create_status(user, content, reply_parent=review)
-    if reply.reply_parent:
-        create_notification(
-            reply.reply_parent.user,
-            'REPLY',
-            related_user=user,
-            related_status=reply,
-        )
-
-    broadcast(user, reply.to_create_activity(user))
-
-
 def handle_favorite(user, status):
     ''' a user likes a status '''
     try:
diff --git a/bookwyrm/templates/snippets/create_status.html b/bookwyrm/templates/snippets/create_status.html
index 88558df9..28eded97 100644
--- a/bookwyrm/templates/snippets/create_status.html
+++ b/bookwyrm/templates/snippets/create_status.html
@@ -22,6 +22,8 @@
             <form class="toggle-content hidden tab-option-{{ book.id }}" name="review" action="/review/" method="post" id="tab-review-{{ book.id }}">
                 {% csrf_token %}
                 <input type="hidden" name="book" value="{{ book.id }}">
+                <input type="hidden" name="user" value="{{ request.user.id }}">
+                <input type="hidden" name="privacy" value="public">
                 <div class="control">
                     <label class="label" for="id_name_{{ book.id }}_review">Title:</label>
                     <input type="text" name="name" maxlength="255" class="input" required="" id="id_name_{{ book.id }}_review" placeholder="My review of '{{ book.title }}'">
@@ -51,6 +53,9 @@
             <form class="toggle-content hidden tab-option-{{ book.id }}" name="comment" action="/comment/" method="post" id="tab-comment-{{ book.id }}">
                 {% csrf_token %}
                 <input type="hidden" name="book" value="{{ book.id }}">
+                <input type="hidden" name="book" value="{{ book.id }}">
+                <input type="hidden" name="user" value="{{ request.user.id }}">
+                <input type="hidden" name="privacy" value="public">
                 <div class="control">
                     <label class="label" for="id_content_{{ book.id }}_comment">Comment:</label>
                     <textarea name="content" class="textarea" id="id_content_{{ book.id }}_comment" placeholder="Some thoughts on '{{ book.title }}'"></textarea>
@@ -64,6 +69,9 @@
             <form class="toggle-content hidden tab-option-{{ book.id }}" name="quotation" action="/quotate/" method="post" id="tab-quotation-{{ book.id }}">
                 {% csrf_token %}
                 <input type="hidden" name="book" value="{{ book.id }}">
+                <input type="hidden" name="book" value="{{ book.id }}">
+                <input type="hidden" name="user" value="{{ request.user.id }}">
+                <input type="hidden" name="privacy" value="public">
                 <div class="control">
                     <label class="label" for="id_quote_{{ book.id }}_quote">Quote:</label>
                     <textarea name="quote" class="textarea" required="" id="id_quote_{{ book.id }}_quote" placeholder="An except from '{{ book.title }}'"></textarea>
diff --git a/bookwyrm/templates/snippets/interaction.html b/bookwyrm/templates/snippets/interaction.html
index 8680df8d..27bbbf8b 100644
--- a/bookwyrm/templates/snippets/interaction.html
+++ b/bookwyrm/templates/snippets/interaction.html
@@ -4,7 +4,9 @@
     <form name="reply" action="/reply" method="post" onsubmit="return reply(event)">
         <div class="field is-grouped">
         {% csrf_token %}
-        <input type="hidden" name="parent" value="{{ activity.id }}">
+        <input type="hidden" name="reply_parent" value="{{ activity.id }}">
+        <input type="hidden" name="user" value="{{ request.user.id }}">
+        <input type="hidden" name="privacy" value="public">
         <textarea name="content" placeholder="Leave a comment..." id="id_content_{{ activity.id }}" required="true"></textarea>
         <button class="button" type="submit">
             <span class="icon icon-comment">
diff --git a/bookwyrm/view_actions.py b/bookwyrm/view_actions.py
index e8a3b1fc..f82c97c0 100644
--- a/bookwyrm/view_actions.py
+++ b/bookwyrm/view_actions.py
@@ -320,14 +320,21 @@ def comment(request):
     return handle_status(request, form)
 
 
+@login_required
+def reply(request):
+    ''' respond to a book review '''
+    form = forms.ReplyForm(request.POST)
+    return handle_status(request, form)
+
+
 def handle_status(request, form):
-    ''' all the review/comment/quote etc functions are the same '''
+    ''' all the "create a status" functions are the same '''
     book_id = request.POST.get('book')
     if not form.is_valid():
-        return redirect('/book/%s' % book_id)
+        return redirect(request.headers.get('Referer', '/'))
 
     outgoing.handle_status(request.user, form)
-    return redirect('/book/%s' % book_id)
+    return redirect(request.headers.get('Referer', '/'))
 
 
 @login_required
@@ -353,19 +360,6 @@ def untag(request):
     return redirect('/book/%s' % book_id)
 
 
-@login_required
-def reply(request):
-    ''' respond to a book review '''
-    form = forms.ReplyForm(request.POST)
-    # this is a bit of a formality, the form is just one text field
-    if not form.is_valid():
-        return redirect('/')
-    parent_id = request.POST['parent']
-    parent = models.Status.objects.get(id=parent_id)
-    outgoing.handle_reply(request.user, parent, form.data['content'])
-    return redirect('/')
-
-
 @login_required
 def favorite(request, status_id):
     ''' like a status '''

From 5a09ce68eca1e13ae47b1a5e006eca8df5abc1b8 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Tue, 27 Oct 2020 11:52:23 -0700
Subject: [PATCH 5/8] Removed unused status generation code

---
 bookwyrm/status.py | 117 ---------------------------------------------
 1 file changed, 117 deletions(-)

diff --git a/bookwyrm/status.py b/bookwyrm/status.py
index 57c800e5..69abfc9a 100644
--- a/bookwyrm/status.py
+++ b/bookwyrm/status.py
@@ -14,94 +14,6 @@ def delete_status(status):
     status.save()
 
 
-def create_rating(user, book, rating):
-    ''' a review that's just a rating '''
-    if not rating or rating < 1 or rating > 5:
-        raise ValueError('Invalid rating')
-    return models.Review.objects.create(
-        user=user,
-        book=book,
-        rating=rating,
-    )
-
-
-def create_review(user, book, name, content, rating):
-    ''' a book review has been added '''
-    name = sanitize(name)
-    content = sanitize(content)
-
-    # no ratings outside of 0-5
-    if rating:
-        rating = rating if 1 <= rating <= 5 else None
-    else:
-        rating = None
-
-    return models.Review.objects.create(
-        user=user,
-        book=book,
-        name=name,
-        rating=rating,
-        content=content,
-    )
-
-
-def create_quotation_from_activity(author, activity):
-    ''' parse an activity json blob into a status '''
-    book_id = activity['inReplyToBook']
-    book = get_or_create_book(book_id)
-    quote = activity.get('quote')
-    content = activity.get('content')
-    published = activity.get('published')
-    remote_id = activity['id']
-
-    quotation = create_quotation(author, book, content, quote)
-    quotation.published_date = published
-    quotation.remote_id = remote_id
-    quotation.save()
-    return quotation
-
-
-def create_quotation(user, book, content, quote):
-    ''' a quotation has been added '''
-    # throws a value error if the book is not found
-    content = sanitize(content)
-    quote = sanitize(quote)
-
-    return models.Quotation.objects.create(
-        user=user,
-        book=book,
-        content=content,
-        quote=quote,
-    )
-
-
-def create_comment_from_activity(author, activity):
-    ''' parse an activity json blob into a status '''
-    book_id = activity['inReplyToBook']
-    book = get_or_create_book(book_id)
-    content = activity.get('content')
-    published = activity.get('published')
-    remote_id = activity['id']
-
-    comment = create_comment(author, book, content)
-    comment.published_date = published
-    comment.remote_id = remote_id
-    comment.save()
-    return comment
-
-
-def create_comment(user, book, content):
-    ''' a book comment has been added '''
-    # throws a value error if the book is not found
-    content = sanitize(content)
-
-    return models.Comment.objects.create(
-        user=user,
-        book=book,
-        content=content,
-    )
-
-
 def get_status(remote_id):
     ''' find a status in the database '''
     return models.Status.objects.select_subclasses().filter(
@@ -128,28 +40,6 @@ def create_generated_note(user, content, mention_books=None):
     return status
 
 
-def create_status(user, content, reply_parent=None, mention_books=None):
-    ''' a status update '''
-    # TODO: handle @'ing users
-
-    # sanitize input html
-    parser = InputHtmlParser()
-    parser.feed(content)
-    content = parser.get_output()
-
-    status = models.Status.objects.create(
-        user=user,
-        content=content,
-        reply_parent=reply_parent,
-    )
-
-    if mention_books:
-        for book in mention_books:
-            status.mention_books.add(book)
-
-    return status
-
-
 def create_tag(user, possible_book, name):
     ''' add a tag to a book '''
     book = get_or_create_book(possible_book)
@@ -175,10 +65,3 @@ def create_notification(user, notification_type, related_user=None, \
         related_import=related_import,
         notification_type=notification_type,
     )
-
-
-def sanitize(content):
-    ''' remove invalid html from free text '''
-    parser = InputHtmlParser()
-    parser.feed(content)
-    return parser.get_output()

From c43c56678371294bf0a5373292ac449a880c5af4 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Tue, 27 Oct 2020 11:55:15 -0700
Subject: [PATCH 6/8] Removes tests of deleted code

---
 bookwyrm/tests/status/__init__.py       |  1 -
 bookwyrm/tests/status/test_comment.py   | 18 ------------
 bookwyrm/tests/status/test_quotation.py | 24 ---------------
 bookwyrm/tests/status/test_review.py    | 39 -------------------------
 bookwyrm/tests/status/test_status.py    | 28 ------------------
 5 files changed, 110 deletions(-)
 delete mode 100644 bookwyrm/tests/status/__init__.py
 delete mode 100644 bookwyrm/tests/status/test_comment.py
 delete mode 100644 bookwyrm/tests/status/test_quotation.py
 delete mode 100644 bookwyrm/tests/status/test_review.py
 delete mode 100644 bookwyrm/tests/status/test_status.py

diff --git a/bookwyrm/tests/status/__init__.py b/bookwyrm/tests/status/__init__.py
deleted file mode 100644
index b6e690fd..00000000
--- a/bookwyrm/tests/status/__init__.py
+++ /dev/null
@@ -1 +0,0 @@
-from . import *
diff --git a/bookwyrm/tests/status/test_comment.py b/bookwyrm/tests/status/test_comment.py
deleted file mode 100644
index be127d88..00000000
--- a/bookwyrm/tests/status/test_comment.py
+++ /dev/null
@@ -1,18 +0,0 @@
-from django.test import TestCase
-
-from bookwyrm import models
-from bookwyrm import status as status_builder
-
-
-class Comment(TestCase):
-    ''' we have hecka ways to create statuses '''
-    def setUp(self):
-        self.user = models.User.objects.create_user(
-            'mouse', 'mouse@mouse.mouse', 'mouseword')
-        self.book = models.Edition.objects.create(title='Example Edition')
-
-
-    def test_create_comment(self):
-        comment = status_builder.create_comment(
-            self.user, self.book, 'commentary')
-        self.assertEqual(comment.content, 'commentary')
diff --git a/bookwyrm/tests/status/test_quotation.py b/bookwyrm/tests/status/test_quotation.py
deleted file mode 100644
index 4892e21d..00000000
--- a/bookwyrm/tests/status/test_quotation.py
+++ /dev/null
@@ -1,24 +0,0 @@
-from django.test import TestCase
-
-from bookwyrm import models
-from bookwyrm import status as status_builder
-
-
-class Quotation(TestCase):
-    ''' we have hecka ways to create statuses '''
-    def setUp(self):
-        self.user = models.User.objects.create_user(
-            'mouse', 'mouse@mouse.mouse', 'mouseword',
-            remote_id='https://example.com/user/mouse'
-        )
-        self.book = models.Edition.objects.create(
-            title='Example Edition',
-            remote_id='https://example.com/book/1',
-        )
-
-
-    def test_create_quotation(self):
-        quotation = status_builder.create_quotation(
-            self.user, self.book, 'commentary', 'a quote')
-        self.assertEqual(quotation.quote, 'a quote')
-        self.assertEqual(quotation.content, 'commentary')
diff --git a/bookwyrm/tests/status/test_review.py b/bookwyrm/tests/status/test_review.py
deleted file mode 100644
index 263fef97..00000000
--- a/bookwyrm/tests/status/test_review.py
+++ /dev/null
@@ -1,39 +0,0 @@
-from django.test import TestCase
-
-from bookwyrm import models
-from bookwyrm import status as status_builder
-
-
-class Review(TestCase):
-    ''' we have hecka ways to create statuses '''
-    def setUp(self):
-        self.user = models.User.objects.create_user(
-            'mouse', 'mouse@mouse.mouse', 'mouseword')
-        self.book = models.Edition.objects.create(title='Example Edition')
-
-
-    def test_create_review(self):
-        review = status_builder.create_review(
-            self.user, self.book, 'review name', 'content', 5)
-        self.assertEqual(review.name, 'review name')
-        self.assertEqual(review.content, 'content')
-        self.assertEqual(review.rating, 5)
-
-        review = status_builder.create_review(
-            self.user, self.book, '<div>review</div> name', '<b>content', 5)
-        self.assertEqual(review.name, 'review name')
-        self.assertEqual(review.content, 'content')
-        self.assertEqual(review.rating, 5)
-
-    def test_review_rating(self):
-        review = status_builder.create_review(
-            self.user, self.book, 'review name', 'content', -1)
-        self.assertEqual(review.name, 'review name')
-        self.assertEqual(review.content, 'content')
-        self.assertEqual(review.rating, None)
-
-        review = status_builder.create_review(
-            self.user, self.book, 'review name', 'content', 6)
-        self.assertEqual(review.name, 'review name')
-        self.assertEqual(review.content, 'content')
-        self.assertEqual(review.rating, None)
diff --git a/bookwyrm/tests/status/test_status.py b/bookwyrm/tests/status/test_status.py
deleted file mode 100644
index cb49cb12..00000000
--- a/bookwyrm/tests/status/test_status.py
+++ /dev/null
@@ -1,28 +0,0 @@
-from django.test import TestCase
-
-from bookwyrm import models
-from bookwyrm import status as status_builder
-
-
-class Status(TestCase):
-    ''' we have hecka ways to create statuses '''
-    def setUp(self):
-        self.user = models.User.objects.create_user(
-            'mouse', 'mouse@mouse.mouse', 'mouseword',
-            local=False,
-            inbox='https://example.com/user/mouse/inbox',
-            outbox='https://example.com/user/mouse/outbox',
-            remote_id='https://example.com/user/mouse'
-        )
-
-
-    def test_create_status(self):
-        content = 'statuses are usually <i>replies</i>'
-        status = status_builder.create_status(
-            self.user, content)
-        self.assertEqual(status.content, content)
-
-        reply = status_builder.create_status(
-            self.user, content, reply_parent=status)
-        self.assertEqual(reply.content, content)
-        self.assertEqual(reply.reply_parent, status)

From e3317ff7711da4398c453488a092c2b9682882cf Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Tue, 27 Oct 2020 12:01:41 -0700
Subject: [PATCH 7/8] restores creating reviews in goodreads import

---
 bookwyrm/outgoing.py | 23 +++++++++++------------
 1 file changed, 11 insertions(+), 12 deletions(-)

diff --git a/bookwyrm/outgoing.py b/bookwyrm/outgoing.py
index bcdf7170..4681a670 100644
--- a/bookwyrm/outgoing.py
+++ b/bookwyrm/outgoing.py
@@ -176,18 +176,17 @@ def handle_import_books(user, items):
                 broadcast(user, activity)
 
                 if item.rating or item.review:
-                    pass
-                    #review_title = "Review of {!r} on Goodreads".format(
-                    #    item.book.title,
-                    #) if item.review else ""
-                    # TODO
-                    #handle_review(
-                    #    user,
-                    #    item.book,
-                    #    review_title,
-                    #    item.review,
-                    #    item.rating,
-                    #)
+                    review_title = 'Review of {!r} on Goodreads'.format(
+                        item.book.title,
+                    ) if item.review else ''
+
+                    models.Review.objects.create(
+                        user=user,
+                        book=item.book,
+                        name=review_title,
+                        content=item.review,
+                        rating=item.rating,
+                    )
                 for read in item.reads:
                     read.book = item.book
                     read.user = user

From 003740ca21fb00ef7c989d7a99eff5a7175d9be0 Mon Sep 17 00:00:00 2001
From: Mouse Reeve <mousereeve@riseup.net>
Date: Tue, 27 Oct 2020 12:02:24 -0700
Subject: [PATCH 8/8] One last function we don't need

---
 bookwyrm/status.py | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/bookwyrm/status.py b/bookwyrm/status.py
index 69abfc9a..258734b3 100644
--- a/bookwyrm/status.py
+++ b/bookwyrm/status.py
@@ -14,13 +14,6 @@ def delete_status(status):
     status.save()
 
 
-def get_status(remote_id):
-    ''' find a status in the database '''
-    return models.Status.objects.select_subclasses().filter(
-        remote_id=remote_id
-    ).first()
-
-
 def create_generated_note(user, content, mention_books=None):
     ''' a note created by the app about user activity '''
     # sanitize input html