Indent filter fails confusing if input is not string about jinja HOT 3 CLOSED

In general, Python does not do type checking just to show special error messages. If you pass in wrong data to something it might fail in weird ways at a weird spot. This filter isn't special in this regard.

from jinja.

@davidism If you look through the code, you have yourself already added a number of FilterArgumentExceptions—correctly, according to good design—to ensure that failures are graceful. Also, the idea that Python programmers normally don't bother to handle their exceptions and just let things blow up confusingly is .... not true.

from jinja.

Here's the standard library's own indent function, failing on invalid input without talking about str. Most other functions result in similar behavior. Convince the cpython maintainers that they should add runtime type checks for the arguments of each function they maintain, and I'll consider adding them all over the place in Jinja too.

>>> import textwrap
>>> textwrap.indent({}, "  ")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.10/textwrap.py", line 488, in indent
    return ''.join(prefixed_lines())
  File "/usr/local/lib/python3.10/textwrap.py", line 486, in prefixed_lines
    for line in text.splitlines(True):
AttributeError: 'dict' object has no attribute 'splitlines'

from jinja.