From 01cfcf0f3b6dc26aa67a93e00e8e06205fedf431 Mon Sep 17 00:00:00 2001 From: "Sergio C. Orozco Torres" Date: Tue, 7 Jun 2016 01:03:59 -0500 Subject: [PATCH] Undefined word_per_minutes parameter division by zero If the parameter words_per_minute at _config.yml is not defined and the template says that must show read_time then will be divide a number by zero, a fix is verify that parameter and show a message when this is empty or zero. already localized for spanish --- _data/ui-text.yml | 2 ++ _includes/read-time.html | 16 +++++++++++----- 2 files changed, 13 insertions(+), 5 deletions(-) diff --git a/_data/ui-text.yml b/_data/ui-text.yml index 99daed15..18867f13 100644 --- a/_data/ui-text.yml +++ b/_data/ui-text.yml @@ -27,6 +27,7 @@ en: &DEFAULT_EN website_label : "Website" email_label : "Email" recent_posts : "Recent Posts" + undefined_wpm : "Undefined parameter words_per_minute at _config.yml" en-US: <<: *DEFAULT_EN en-UK: @@ -59,6 +60,7 @@ es: &DEFAULT_ES website_label : "Sitio web" email_label : "Email" recent_posts : "Entradas recientes" + undefined_wpm : "Parametro words_per_minute (Palabras por minuto) no definido en _config.yml" es-ES: <<: *DEFAULT_ES es-CO: diff --git a/_includes/read-time.html b/_includes/read-time.html index 86e7db84..b0dd0626 100644 --- a/_includes/read-time.html +++ b/_includes/read-time.html @@ -1,13 +1,19 @@ + + {% if post.read_time %} {% assign words = post.content | strip_html | number_of_words %} {% elsif page.read_time %} {% assign words = page.content | strip_html | number_of_words %} {% endif %} -{% if words < 180 %} - {{ site.data.ui-text[site.locale].less_than }} 1 {{ site.data.ui-text[site.locale].minute_read }} -{% elsif words < 360 %} - 1 {{ site.data.ui-text[site.locale].minute_read }} +{% if site.words_per_minute %} + {% if words < 180 %} + {{ site.data.ui-text[site.locale].less_than }} 1 {{ site.data.ui-text[site.locale].minute_read }} + {% elsif words < 360 %} + 1 {{ site.data.ui-text[site.locale].minute_read }} + {% else %} + {{ words | divided_by:site.words_per_minute }} {{ site.data.ui-text[site.locale].minutes_read }} + {% endif %} {% else %} - {{ words | divided_by:site.words_per_minute }} {{ site.data.ui-text[site.locale].minutes_read }} + {{ site.data.ui-text[site.locale].undefined_words_per_minute }} {% endif %} \ No newline at end of file