mirrors/django
1
0
Fork 0
mirror of https://github.com/django/django.git synced 2025-03-25 16:50:45 +00:00
Code Issues 32 Releases Wiki Activity

[5.1.x] Refs #34609 -- Fixed deprecation warning stack level in format_html().

Browse Source 
Co-authored-by: Simon Charette <charette.s@gmail.com>

Backport of 2b71b2c8dcd40f2604310bb3914077320035b399 from main.
This commit is contained in:
Adam Johnson 2024-08-27 19:14:50 +01:00 committed by Natalia
parent f362c3dea2
commit 03e0ab5c64
2 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
django/utils/html.py
View File

@ -129,10 +129,11 @@ def format_html(format_string, *args, **kwargs):
""" """
if not (args or kwargs): if not (args or kwargs):
# RemovedInDjango60Warning: when the deprecation ends, replace with: # RemovedInDjango60Warning: when the deprecation ends, replace with:
# raise ValueError("args or kwargs must be provided.") # raise TypeError("args or kwargs must be provided.")
warnings.warn( warnings.warn(
"Calling format_html() without passing args or kwargs is deprecated.", "Calling format_html() without passing args or kwargs is deprecated.",
RemovedInDjango60Warning, RemovedInDjango60Warning,
stacklevel=2,
) )
args_safe = map(conditional_escape, args) args_safe = map(conditional_escape, args)
kwargs_safe = {k: conditional_escape(v) for (k, v) in kwargs.items()} kwargs_safe = {k: conditional_escape(v) for (k, v) in kwargs.items()}

5
tests/utils_tests/test_html.py
View File

@ -70,10 +70,11 @@ class TestUtilsHtml(SimpleTestCase):
msg = "Calling format_html() without passing args or kwargs is deprecated." msg = "Calling format_html() without passing args or kwargs is deprecated."
# RemovedInDjango60Warning: when the deprecation ends, replace with: # RemovedInDjango60Warning: when the deprecation ends, replace with:
# msg = "args or kwargs must be provided." # msg = "args or kwargs must be provided."
# with self.assertRaisesMessage(ValueError, msg): # with self.assertRaisesMessage(TypeError, msg):
with self.assertWarnsMessage(RemovedInDjango60Warning, msg): with self.assertWarnsMessage(RemovedInDjango60Warning, msg) as ctx:
name = "Adam" name = "Adam"
self.assertEqual(format_html(f"<i>{name}</i>"), "<i>Adam</i>") self.assertEqual(format_html(f"<i>{name}</i>"), "<i>Adam</i>")
self.assertEqual(ctx.filename, __file__)
def test_linebreaks(self): def test_linebreaks(self):
items = ( items = (