Adds openlibrary update for book

bookwyrm/connectors/abstract_connector.py

@@ -196,12 +196,12 @@ class AbstractConnector(AbstractMinimalConnector):
 
     def update_author_from_remote(self, obj):
         """load the remote data from this connector and add it to an existing author"""
-        remote_id = obj.getattr(self, "generated_remote_link_field")
+        remote_id = getattr(obj, getattr(self, "generated_remote_link_field"))
         return self.get_or_create_author(remote_id, instance=obj)
 
     def update_book_from_remote(self, obj):
         """load the remote data from this connector and add it to an existing book"""
-        remote_id = obj.getattr(self, "generated_remote_link_field")
+        remote_id = getattr(obj, getattr(self, "generated_remote_link_field"))
         data = self.get_book_data(remote_id)
         return self.create_edition_from_data(obj.parent_work, data, instance=obj)
 

bookwyrm/templates/author/author.html

@@ -77,6 +77,9 @@
                 <a itemprop="sameAs" href="{{ author.openlibrary_link }}" target="_blank" rel="noopener">
                     {% trans "View on OpenLibrary" %}
                 </a>
+                {% if request.user.is_authenticated and perms.bookwyrm.edit_book %}
+                <button class="button is-small" type="button">{% trans "Update from OpenLibrary" %}</button>
+                {% endif %}
             </p>
             {% endif %}
 
@@ -85,6 +88,9 @@
                 <a itemprop="sameAs" href="{{ author.inventaire_link }}" target="_blank" rel="noopener">
                     {% trans "View on Inventaire" %}
                 </a>
+                {% if request.user.is_authenticated and perms.bookwyrm.edit_book %}
+                <button class="button is-small" type="button">{% trans "Update from Inventaire" %}</button>
+                {% endif %}
             </p>
             {% endif %}
 

bookwyrm/templates/book/book.html

@@ -91,7 +91,13 @@
                 {% endwith %}
 
                 {% if book.openlibrary_key %}
-                <p><a href="{{ book.openlibrary_link }}" target="_blank" rel="noopener">{% trans "View on OpenLibrary" %}</a></p>
+                <p>
+                    <a href="{{ book.openlibrary_link }}" target="_blank" rel="noopener">{% trans "View on OpenLibrary" %}</a>
+                    <form name="ol-update" method="POST" action="{% url 'book-update-remote' book.id 'openlibrary.org' %}">
+                        {% csrf_token %}
+                        <button class="button is-small" type="submit">{% trans "Update from OpenLibrary" %}</button>
+                    </form>
+                </p>
                 {% endif %}
                 {% if book.inventaire_id %}
                 <p><a href="{{ book.inventaire_link }}" target="_blank" rel="noopener">{% trans "View on Inventaire" %}</a></p>

bookwyrm/urls.py

@@ -422,8 +422,13 @@ urlpatterns = [
         r"^upload-cover/(?P<book_id>\d+)/?$", views.upload_cover, name="upload-cover"
     ),
     re_path(r"^add-description/(?P<book_id>\d+)/?$", views.add_description),
-    re_path(r"^resolve-book/?$", views.resolve_book),
+    re_path(r"^resolve-book/?$", views.resolve_book, name="resolve-book"),
-    re_path(r"^switch-edition/?$", views.switch_edition),
+    re_path(r"^switch-edition/?$", views.switch_edition, name="switch-edition"),
+    re_path(
+        rf"{BOOK_PATH}/update/(?P<connector_identifier>[\w\.]+)?$",
+        views.update_book_from_remote,
+        name="book-update-remote"
+    ),
     # isbn
     re_path(r"^isbn/(?P<isbn>\d+)(.json)?/?$", views.Isbn.as_view()),
     # author

bookwyrm/views/__init__.py

@@ -29,6 +29,7 @@ from .preferences.block import Block, unblock
 
 # books
 from .books.books import Book, upload_cover, add_description, resolve_book
+from .books.books import update_book_from_remote
 from .books.edit_book import EditBook, ConfirmEditBook
 from .books.editions import Editions, switch_edition
 

bookwyrm/views/author.py

@@ -81,7 +81,7 @@ class EditAuthor(View):
 @require_POST
 @permission_required("bookwyrm.edit_book", raise_exception=True)
 # pylint: disable=unused-argument
-def update_author_from_remote(request, connector_identifier, author_id):
+def update_author_from_remote(request, author_id, connector_identifier):
     """load the remote data for this author"""
     connector = connector_manager.load_connector(
         get_object_or_404(models.Connector, identifier=connector_identifier)

bookwyrm/views/books/books.py

@@ -184,7 +184,7 @@ def resolve_book(request):
 @require_POST
 @permission_required("bookwyrm.edit_book", raise_exception=True)
 # pylint: disable=unused-argument
-def update_book_from_remote(request, connector_identifier, book_id):
+def update_book_from_remote(request, book_id, connector_identifier):
     """load the remote data for this book"""
     connector = connector_manager.load_connector(
         get_object_or_404(models.Connector, identifier=connector_identifier)