From 0a02421dd593f0466226b9defc33337cdbf672ef Mon Sep 17 00:00:00 2001
From: Jan Szejko <janek37@gmail.com>
Date: Wed, 11 Jul 2018 13:24:36 +0200
Subject: [PATCH 1/1] rework book box caching

---
 src/catalogue/templates/catalogue/book_detail.html | 5 +----
 src/catalogue/templates/catalogue/book_short.html  | 6 ++++++
 src/catalogue/templates/catalogue/book_text.html   | 5 +----
 src/catalogue/templates/catalogue/work-list.html   | 6 +-----
 4 files changed, 9 insertions(+), 13 deletions(-)

diff --git a/src/catalogue/templates/catalogue/book_detail.html b/src/catalogue/templates/catalogue/book_detail.html
index cf26b125a..865aca2bb 100644
--- a/src/catalogue/templates/catalogue/book_detail.html
+++ b/src/catalogue/templates/catalogue/book_detail.html
@@ -13,10 +13,7 @@
 {% block bodyid %}book-detail{% endblock %}
 
 {% block body %}
-  {% get_current_language as LANGUAGE_CODE %}
-  {% cache 86400 book_wide book.pk book|status:user LANGUAGE_CODE %}
-    {% include 'catalogue/book_wide.html' %}
-  {% endcache %}
+  {% include 'catalogue/book_wide.html' %}
 
   {% work_list book_children %}
 
diff --git a/src/catalogue/templates/catalogue/book_short.html b/src/catalogue/templates/catalogue/book_short.html
index fff3fc0ff..4f186e8ff 100644
--- a/src/catalogue/templates/catalogue/book_short.html
+++ b/src/catalogue/templates/catalogue/book_short.html
@@ -1,5 +1,6 @@
 {% spaceless %}
   {% load i18n %}
+  {% load cache %}
   {% load catalogue_tags %}
   {% load book_shelf_tags from social_tags %}
   {% load static %}
@@ -28,6 +29,8 @@
           {% block cover-area-extra %}{% endblock %}
         </div>
 
+        {% get_current_language as LANGUAGE_CODE %}
+        {% cache 86400 book_box_head_tags book.pk LANGUAGE_CODE %}
         <div class="book-box-head">
           <div class="author">
             {% for tag in tags.author %}
@@ -94,9 +97,11 @@
           {% endif %}
           {% endwith %}
         </div>
+        {% endcache %}
       </div>
       {% book_shelf_tags book.pk %}
 
+      {% cache 86400 book_box_tools book.pk book|status:request.user LANGUAGE_CODE %}
       {% if book|status:request.user != 'closed' %}
         <ul class="book-box-tools">
           <li class="book-box-read">
@@ -142,6 +147,7 @@
       {% else %}
         <p class="book-box-tools">{% trans "For now this work is only available for our subscribers." %}</p>
       {% endif %}
+      {% endcache %}
       {% block book-box-extra-info %}{% endblock %}
       {% block box-append %}{% endblock %}
     </div>
diff --git a/src/catalogue/templates/catalogue/book_text.html b/src/catalogue/templates/catalogue/book_text.html
index 5894fe825..e958d7060 100644
--- a/src/catalogue/templates/catalogue/book_text.html
+++ b/src/catalogue/templates/catalogue/book_text.html
@@ -110,9 +110,6 @@
   </div>
 
   <div class="box" id="book-short">
-    {% get_current_language as LANGUAGE_CODE %}
-    {% cache 86400 book_short book.pk book|status:user LANGUAGE_CODE %}
-      {% include 'catalogue/book_short.html' %}
-    {% endcache %}
+    {% include 'catalogue/book_short.html' %}
   </div>
 {% endblock footer %}
diff --git a/src/catalogue/templates/catalogue/work-list.html b/src/catalogue/templates/catalogue/work-list.html
index 178654c70..5e3f3b931 100755
--- a/src/catalogue/templates/catalogue/work-list.html
+++ b/src/catalogue/templates/catalogue/work-list.html
@@ -1,7 +1,6 @@
 {% spaceless %}
 
   {% load cache %}
-  {% load i18n %}
   {% load pagination_tags %}
   {% load class_name from catalogue_tags %}
   {% load status from catalogue_tags %}
@@ -13,10 +12,7 @@
     {% for item in object_list %}
       <li class='{{ item|class_name }}-item'>
         {% if item|class_name == 'Book' %}
-          {% get_current_language as LANGUAGE_CODE %}
-          {% cache 86400 book_short item.pk item|status:request.user LANGUAGE_CODE %}
-            {% include "catalogue/book_short.html" with book=item %}
-          {% endcache %}
+          {% include "catalogue/book_short.html" with book=item %}
         {% else %}
           {% ssi_include item.short_html_url_name pk=item.pk %}
         {% endif %}
-- 
2.20.1