Subscription stats: Make sure there can be no division by zero errors

sitescripts/logs/template/subscription.html

 <!--
   - This file is part of the Adblock Plus web scripts,
   - Copyright (C) 2006-2012 Eyeo GmbH
   -
   - Adblock Plus is free software: you can redistribute it and/or modify
   - it under the terms of the GNU General Public License version 3 as
   - published by the Free Software Foundation.
   -
   - Adblock Plus is distributed in the hope that it will be useful,
   - but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 30 matching lines @@
       <tr>
         <td class="title" width="250">Reported period:</td>
         <td>Month {{month|monthname}} (<a href="{{overviewURL}}">To overview page</a>)</td>
       </tr>
     </table>
 
     <div class="block_title">Days of month</div>
     <div class="block">
       <table align="center" style="margin-bottom: 20px;">
         <tr align="bottom">
-          {%- set maxHits, maxBandwidth = day|max(attribute='hits'), day|max(attribute='bandwidth') %}
-          {%- set totalHits = day|sum(attribute='hits') %}
-          {%- set totalBandwidth = day|sum(attribute='bandwidth') %}
+          {%- set maxHits = day|max(attribute='hits')|ensuremin(1) %}
+          {%- set maxBandwidth = day|max(attribute='bandwidth')|ensuremin(1) %}
+          {%- set totalHits = day|sum(attribute='hits')|ensuremin(1) %}
+          {%- set totalBandwidth = day|sum(attribute='bandwidth')|ensuremin(1) %}
           {%- set dayCount = day|length %}
           {%- if currentMonth and dayCount > 1 %}
             {%- set totalHits = totalHits - day[-1].hits %}
             {%- set totalBandwidth = totalBandwidth - day[-1].bandwidth %}
             {%- set dayCount = dayCount - 1 %}
           {%- endif %}
           {%- for info in day %}
           <td align="center" style="vertical-align: bottom;">
             <div class="hitsChart vertical" title="Hits: {{info.hits}}" style="height: {{(info.hits / maxHits * 100)|round(method='ceil')|int}}px;"></div>
             {{- '' -}}
@@ skipping 64 matching lines @@
           <td align="right">&nbsp;</td>
           <td align="right">{{totalBandwidth|bytes}}</td>
         </tr>
       </table>
     </div>
 
     <div class="block_title">Days of week</div>
     <div class="block">
       <table align="center" style="margin-bottom: 20px;">
         <tr align="bottom">
-          {%- set maxHits, maxBandwidth = weekday|max(attribute='hits'),weekday|max(attribute='bandwidth') %}
+          {%- set maxHits = weekday|max(attribute='hits')|ensuremin(1) %}
+          {%- set maxBandwidth = weekday|max(attribute='bandwidth')|ensuremin(1) %}
           {%- for info in weekday %}
           <td align="center" style="vertical-align: bottom;">
             <div class="hitsChart vertical" title="Hits: {{info.hits}}" style="height: {{(info.hits / maxHits * 100)|round(method='ceil')|int}}px"></div>
             {{- '' -}}
             <div class="bandwidthChart vertical" alt="Bandwidth: {{info.bandwidth|bytes}}"" title="Bandwidth: {{info.bandwidth|bytes}}" style="height: {{(info.bandwidth / maxBandwidth * 100)|round(method='ceil')|int}}px"></div>
           </td>
           {%- endfor %}
         </tr>
         <tr>
           {%- for info in weekday %}
           <td align="center"{% if info.id > 4 %} class="special"{% endif %}>
             {{info.id|weekday}}
           </td>
           {%- endfor %}
         </tr>
       </table>
 
       <table align="center">
         <tr>
           <th bgcolor="#ECECEC">Day</th>
           <th bgcolor="#66DDEE" colspan="2">Hits</th>
           <th bgcolor="#2EA495">Bandwidth</th>
         </tr>
-        {%- set totalHits = weekday|sum(attribute='hits') %}
-        {%- set totalBandwidth = weekday|sum(attribute='bandwidth') %}
+        {%- set totalHits = weekday|sum(attribute='hits')|ensuremin(1) %}
+        {%- set totalBandwidth = weekday|sum(attribute='bandwidth')|ensuremin(1) %}
         {%- for info in weekday %}
         <tr{% if info.id > 4 %} class="special"{% endif %}>
           <td>{{info.id|weekday}}</td>
           <td align="right">{{info.hits}}</td>
           <td align="right">{{(info.hits / totalHits * 100)|round(precision=1)}}%</td>
           <td align="right">{{info.bandwidth|bytes}}</td>
         </tr>
         {%- endfor %}
       </table>
     </div>
 
     <div class="block_title">Hours</div>
     <div class="block">
       <table align="center" style="margin-bottom: 20px;">
         <tr align="bottom">
-          {%- set maxHits, maxBandwidth = hour|max(attribute='hits'),hour|max(attribute='bandwidth') %}
+          {%- set maxHits = hour|max(attribute='hits')|ensuremin(1) %}
+          {%- set maxBandwidth = hour|max(attribute='bandwidth')|ensuremin(1) %}
           {%- for info in hour %}
           <td align="center" style="vertical-align: bottom;">
             <div class="hitsChart vertical" title="Hits: {{info.hits}}" style="height: {{(info.hits / maxHits * 100)|round(method='ceil')|int}}px;"></div>
             {{- '' -}}
             <div class="bandwidthChart vertical" title="Bandwidth: {{info.bandwidth|bytes}}" style="height: {{(info.bandwidth / maxBandwidth * 100)|round(method='ceil')|int}}px;"></div>
           </td>
           {%- if loop.last and info.id|int < 23 %}
             {%- for dummyHour in range(info.id|int + 1, 24) %}
               <td align="center" style="vertical-align: bottom;">
                 <div class="hitsChart vertical" title="Hits: 0" style="height: 1px;"></div>
@@ skipping 21 matching lines @@
           {%- endfor %}
         </tr>
       </table>
 
       <table align="center">
         <tr>
           <th bgcolor="#ECECEC">Hour</th>
           <th bgcolor="#66DDEE" colspan="2">Hits</th>
           <th bgcolor="#2EA495">Bandwidth</th>
         </tr>
-        {%- set totalHits = hour|sum(attribute='hits') %}
-        {%- set totalBandwidth = hour|sum(attribute='bandwidth') %}
+        {%- set totalHits = hour|sum(attribute='hits')|ensuremin(1) %}
+        {%- set totalBandwidth = hour|sum(attribute='bandwidth')|ensuremin(1) %}
         {%- for info in hour %}
         <tr>
           <td>{{info.id}}</td>
           <td align="right">{{info.hits}}</td>
           <td align="right">{{(info.hits / totalHits * 100)|round(precision=1)}}%</td>
           <td align="right">{{info.bandwidth|bytes}}</td>
         </tr>
         {%- endfor %}
       </table>
     </div>
 
     {%- if app %}
     <div class="block_title">Browsers</div>
     <div class="block">
       <table align="center">
         <tr>
           <th bgcolor="#ECECEC">Browser</th>
           <th bgcolor="#66DDEE" colspan="2">Hits</th>
           <th bgcolor="#2EA495">Bandwidth</th>
         </tr>
-        {%- set maxHits, maxBandwidth = app|max(attribute='hits'),app|max(attribute='bandwidth') %}
-        {%- set totalHits = app|sum(attribute='hits') %}
-        {%- set totalBandwidth = app|sum(attribute='bandwidth') %}
+        {%- set maxHits = app|max(attribute='hits')|ensuremin(1) %}
+        {%- set maxBandwidth = app|max(attribute='bandwidth')|ensuremin(1) %}
+        {%- set totalHits = app|sum(attribute='hits')|ensuremin(1) %}
+        {%- set totalBandwidth = app|sum(attribute='bandwidth')|ensuremin(1) %}
         {%- for info in app[0:10] %}
         <tr>
           <td>{{info.id}}</td>
           <td align="right">{{info.hits}}</td>
           <td align="right">{{(info.hits / totalHits * 100)|round(precision=1)}}%</td>
           <td align="right">{{info.bandwidth|bytes}}</td>
           <td class="chart">
             <div class="hitsChart horizontal" style="width: {{(info.hits / maxHits * 100)|round(method='ceil')|int}}px;"></div><br />
             <div class="bandwidthChart horizontal" style="width: {{(info.bandwidth / maxBandwidth * 100)|round(method='ceil')|int}}px;"></div>
           </td>
@@ skipping 33 matching lines @@
 
     <div class="block_title">Countries</div>
     <div class="block">
       <table align="center">
         <tr>
           <th bgcolor="#ECECEC" colspan="3">Country</th>
           <th bgcolor="#66DDEE" colspan="2">Hits</th>
           <th bgcolor="#2EA495">Bandwidth</th>
           <th>&nbsp;</th>
         </tr>
-        {%- set maxHits, maxBandwidth = country|max(attribute='hits'),country|max(attribute='bandwidth') %}
-        {%- set totalHits = country|sum(attribute='hits') %}
-        {%- set totalBandwidth = country|sum(attribute='bandwidth') %}
+        {%- set maxHits = country|max(attribute='hits')|ensuremin(1) %}
+        {%- set maxBandwidth = country|max(attribute='bandwidth')|ensuremin(1) %}
+        {%- set totalHits = country|sum(attribute='hits')|ensuremin(1) %}
+        {%- set totalBandwidth = country|sum(attribute='bandwidth')|ensuremin(1) %}
         {%- for info in country[0:10] %}
         <tr>
           <td><div class="flag {{info.image}}"></div></td>
           <td>{{info.name}}</td>
           <td>{{info.id}}</td>
           <td align="right">{{info.hits}}</td>
           <td align="right">{{(info.hits / totalHits * 100)|round(precision=1)}}%</td>
           <td align="right">{{info.bandwidth|bytes}}</td>
           <td class="chart">
             <div class="hitsChart horizontal" style="width: {{(info.hits / maxHits * 100)|round(method='ceil')|int}}px;"></div><br />
@@ skipping 38 matching lines @@
 
     <div class="block_title">Download mirrors</div>
     <div class="block">
       <table align="center">
         <tr>
           <th bgcolor="#ECECEC">Mirror</th>
           <th bgcolor="#66DDEE" colspan="2">Hits</th>
           <th bgcolor="#2EA495">Bandwidth</th>
           <th>&nbsp;</th>
         </tr>
-        {%- set maxHits, maxBandwidth = mirror|max(attribute='hits'),mirror|max(attribute='bandwidth') %}
-        {%- set totalHits = mirror|sum(attribute='hits') %}
-        {%- set totalBandwidth = mirror|sum(attribute='bandwidth') %}
+        {%- set maxHits = mirror|max(attribute='hits')|ensuremin(1) %}
+        {%- set maxBandwidth = mirror|max(attribute='bandwidth')|ensuremin(1) %}
+        {%- set totalHits = mirror|sum(attribute='hits')|ensuremin(1) %}
+        {%- set totalBandwidth = mirror|sum(attribute='bandwidth')|ensuremin(1) %}
         {%- for info in mirror %}
         <tr>
           <td>{{info.id}}</td>
           <td align="right">{{info.hits}}</td>
           <td align="right">{{(info.hits / totalHits * 100)|round(precision=1)}}%</td>
           <td align="right">{{info.bandwidth|bytes}}</td>
           <td class="chart">
             <div class="hitsChart horizontal" style="width: {{(info.hits / maxHits * 100)|round(method='ceil')|int}}px;"></div><br />
             <div class="bandwidthChart horizontal" style="width: {{(info.bandwidth / maxBandwidth * 100)|round(method='ceil')|int}}px;"></div>
           </td>
         </tr>
         {%- endfor %}
       </table>
     </div>
   </body>
 </html>