darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-23 15:57:23 -04:00
Code Issues Packages Projects Releases Wiki Activity

use markup so html link renders in template

Browse Source
This commit is contained in:
Kenneth Kehl
2023-11-28 14:43:12 -08:00
parent 1cacd3b850
commit 65e6812385
1 changed files with 13 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
app/main/views/templates.py
View File

@@ -2,6 +2,7 @@ from functools import partial
from flask import abort, flash, jsonify, redirect, render_template, request, url_for from flask import abort, flash, jsonify, redirect, render_template, request, url_for
from flask_login import current_user from flask_login import current_user
from markupsafe import Markup
from notifications_python_client.errors import HTTPError from notifications_python_client.errors import HTTPError
from notifications_utils import SMS_CHAR_COUNT_LIMIT from notifications_utils import SMS_CHAR_COUNT_LIMIT
@@ -684,10 +685,11 @@ def _is_latin1(s):
def _get_content_count_error_and_message_for_template(template): def _get_content_count_error_and_message_for_template(template):
url = "https://en.wikipedia.org/wiki/ISO/IEC_8859-1"
if template.template_type == "sms": if template.template_type == "sms":
s1 = " Use of characters outside the iso-latin-1 character set may increase " s1 = f"<html><body>Use of characters outside the <a href='{url}'>IEC_8859-1</a> character set may increase "
s2 = "the message fragment count, resulting in additional charges, and these non iso-latin-1 " s2 = "the message fragment count, resulting in additional charges, and these IEC_8859-1 "
s3 = "characters may not display properly on older phones." s3 = "characters may not display properly on some older mobile devices.</body></html>"
warning = "" warning = ""
try: try:
@@ -703,11 +705,16 @@ def _get_content_count_error_and_message_for_template(template):
) )
if template.placeholders: if template.placeholders:
return False, ( return False, (
f"Will be charged as {message_count(template.fragment_count, template.template_type)} " Markup(
f"(not including personalization). {warning}" f"Will be charged as {message_count(template.fragment_count, template.template_type)} "
f"(not including personalization). {warning}"
)
) )
return False, ( return False, (
f"Will be charged as {message_count(template.fragment_count, template.template_type)}. {warning} " # Markup marks html contents safe so that they render properly. Don't use it if there is user input.
Markup(
f"Will be charged as {message_count(template.fragment_count, template.template_type)}. {warning} "
)
) )