From 9181e77d5507e3eb827726753608f6c2e24c370b Mon Sep 17 00:00:00 2001
From: Bartuccio Antoine <klmp200@klmp200.net>
Date: Mon, 9 Sep 2019 01:20:15 +0200
Subject: [PATCH] core: add some cache on birthdays and counters_activity

---
 com/templates/com/news_list.jinja |  3 +++
 core/templates/core/base.jinja    | 32 ++++++++++++++++---------------
 2 files changed, 20 insertions(+), 15 deletions(-)

diff --git a/com/templates/com/news_list.jinja b/com/templates/com/news_list.jinja
index d3e3016b..7c2ec629 100644
--- a/com/templates/com/news_list.jinja
+++ b/com/templates/com/news_list.jinja
@@ -42,6 +42,8 @@
         <div id="birthdays_title">{% trans %}Birthdays{% endtrans %}</div>
           <div id="birthdays_content">
             {% if user.is_subscribed %}
+              {# Cache request for 1 hour #}
+              {% cache 3600 birthdays %}
               <ul class="birthdays_year">
                 {% for d in birthdays.dates('date_of_birth', 'year', 'DESC') %}
                   <li>
@@ -54,6 +56,7 @@
                   </li>
                 {% endfor %}
               </ul>
+              {% endcache %}
             {% else %}
               <p>{% trans %}You need an up to date subscription to access this content{% endtrans %}</p>
             {% endif %}
diff --git a/core/templates/core/base.jinja b/core/templates/core/base.jinja
index 9d0e53cc..43d2a058 100644
--- a/core/templates/core/base.jinja
+++ b/core/templates/core/base.jinja
@@ -60,21 +60,23 @@
             </div>
             <div id="header_bar">
                 <ul id="header_bars_infos">
-                {% for bar in Counter.objects.filter(type="BAR").all() %}
-                    <li>
-                    <a href="{{ url('counter:activity', counter_id=bar.id) }}" style="padding: 0px">
-                    {% if bar.is_inactive(): %}
-                        <i class="fa fa-question" style="color: #f39c12"></i>
-                    {% elif bar.is_open(): %}
-                        <i class="fa fa-check" style="color: #2ecc71"></i>
-                    {% else %}
-                        <i class="fa fa-times" style="color: #eb2f06"></i>
-                    {% endif %}
-                        {{ bar }}
-                    </a>
-                    </li>
-                {% endfor %}
-                </ul>
+                {% cache 100 counters_activity %}
+                  {% for bar in Counter.objects.filter(type="BAR").all() %}
+                      <li>
+                      <a href="{{ url('counter:activity', counter_id=bar.id) }}" style="padding: 0px">
+                      {% if bar.is_inactive(): %}
+                          <i class="fa fa-question" style="color: #f39c12"></i>
+                      {% elif bar.is_open(): %}
+                          <i class="fa fa-check" style="color: #2ecc71"></i>
+                      {% else %}
+                          <i class="fa fa-times" style="color: #eb2f06"></i>
+                      {% endif %}
+                          {{ bar }}
+                      </a>
+                      </li>
+                  {% endfor %}
+                  </ul>
+                {% endcache %}
                 <form action="{{ url('core:search') }}" method="GET" id="header_search">
                     <input type="text" placeholder="{% trans %}Search{% endtrans %}" name="query" id="search" />
                     <input type="submit" value="{% trans %}Search{% endtrans %}" style="display: none;" />