From b4f51867b625b1c00adc085a1a42c596e2b5aa89 Mon Sep 17 00:00:00 2001
From: zorun <github@bitsofnetworks.org>
Date: Sun, 26 Apr 2020 23:12:33 +0200
Subject: [PATCH] Fix crash when a localized email template is missing (#592)

(cherry picked from commit 08bb95422bd43ce729992673b27117088f5454d9)
---
 ihatemoney/utils.py | 15 ++++++++++++++-
 ihatemoney/web.py   | 24 ++++++++++++------------
 2 files changed, 26 insertions(+), 13 deletions(-)

diff --git a/ihatemoney/utils.py b/ihatemoney/utils.py
index 393667e5..c0864a11 100644
--- a/ihatemoney/utils.py
+++ b/ihatemoney/utils.py
@@ -8,7 +8,8 @@ import operator
 from io import BytesIO, StringIO
 import jinja2
 from json import dumps, JSONEncoder
-from flask import redirect, current_app
+from flask import redirect, current_app, render_template
+from flask_babel import get_locale
 from babel import Locale
 from werkzeug.routing import HTTPException, RoutingException
 import six
@@ -251,3 +252,15 @@ def eval_arithmetic_expression(expr):
         raise ValueError("Error evaluating expression: {}".format(expr))
 
     return result
+
+
+def render_localized_template(template_name_prefix, **context):
+    """Like render_template(), but selects the right template according to the
+    current user language.  Fallback to English if a template for the
+    current language does not exist.
+    """
+    fallback = "en"
+    templates = ["{}.{}.j2".format(template_name_prefix, lang)
+                 for lang in (get_locale().language, fallback)]
+    # render_template() supports a list of templates to try in order
+    return render_template(templates, **context)
diff --git a/ihatemoney/web.py b/ihatemoney/web.py
index 3e74362e..fa8d7f50 100644
--- a/ihatemoney/web.py
+++ b/ihatemoney/web.py
@@ -27,7 +27,7 @@ from ihatemoney.forms import (
     AdminAuthenticationForm, AuthenticationForm, EditProjectForm,
     InviteForm, MemberForm, PasswordReminder, ResetPasswordForm, ProjectForm, get_billform_for
 )
-from ihatemoney.utils import Redirect303, list_of_dicts2json, list_of_dicts2csv, LoginThrottler
+from ihatemoney.utils import Redirect303, list_of_dicts2json, list_of_dicts2csv, LoginThrottler, render_localized_template
 
 main = Blueprint("main", __name__)
 
@@ -242,8 +242,7 @@ def create_project():
             message_title = _("You have just created '%(project)s' "
                               "to share your expenses", project=g.project.name)
 
-            message_body = render_template("reminder_mail.%s.j2" %
-                                           get_locale().language)
+            message_body = render_localized_template("reminder_mail")
 
             msg = Message(message_title,
                           body=message_body,
@@ -272,11 +271,15 @@ def remind_password():
             # get the project
             project = Project.query.get(form.id.data)
             # send a link to reset the password
-            password_reminder = "password_reminder.%s.j2" % get_locale().language
-            current_app.mail.send(Message(
-                "password recovery",
-                body=render_template(password_reminder, project=project),
-                recipients=[project.contact_email]))
+            current_app.mail.send(
+                Message(
+                    "password recovery",
+                    body=render_localized_template(
+                        "password_reminder", project=project
+                    ),
+                    recipients=[project.contact_email],
+                )
+            )
             return redirect(url_for(".password_reminder_sent"))
 
     return render_template("password_reminder.html", form=form)
@@ -402,10 +405,7 @@ def invite():
     if request.method == "POST":
         if form.validate():
             # send the email
-
-            message_body = render_template("invitation_mail.%s.j2" %
-                                           get_locale().language)
-
+            message_body = render_localized_template("invitation_mail")
             message_title = _("You have been invited to share your "
                               "expenses for %(project)s", project=g.project.name)
             msg = Message(message_title,