provide clearer erroneous template filter use message (#86062)
gitea/wcs/pipeline/head This commit looks good
Details
gitea/wcs/pipeline/head This commit looks good
Details
This commit is contained in:
parent
4d5b309986
commit
82bb6d0e89
|
@ -1942,14 +1942,14 @@ def test_lazy_formdata_queryset_filter(pub, variable_test_data):
|
|||
assert tmpl.render(context) == 'None'
|
||||
assert pub.loggederror_class.count() == 1
|
||||
logged_error = pub.loggederror_class.select()[0]
|
||||
assert logged_error.summary == '|pending used on invalid queryset (\'\')'
|
||||
assert logged_error.summary == '|pending used on something else than a queryset (\'\')'
|
||||
|
||||
pub.loggederror_class.wipe()
|
||||
tmpl = Template('{{""|filter_value:"foo"}}')
|
||||
assert tmpl.render(context) == 'None'
|
||||
assert pub.loggederror_class.count() == 1
|
||||
logged_error = pub.loggederror_class.select()[0]
|
||||
assert logged_error.summary == '|filter_value used on invalid queryset (\'\')'
|
||||
assert logged_error.summary == '|filter_value used on something else than a queryset (\'\')'
|
||||
|
||||
|
||||
def test_lazy_formdata_queryset_filter_non_unique_varname(pub, variable_test_data):
|
||||
|
|
|
@ -750,7 +750,9 @@ def decorate_queryset_filter(func, name, attr):
|
|||
@functools.wraps(func)
|
||||
def f(queryset, *args, **kwargs):
|
||||
if not hasattr(queryset, attr):
|
||||
get_publisher().record_error(_('|%s used on invalid queryset (%r)') % (name, queryset))
|
||||
get_publisher().record_error(
|
||||
_('|%s used on something else than a queryset (%r)') % (name, queryset)
|
||||
)
|
||||
return None
|
||||
return func(queryset, *args, **kwargs)
|
||||
|
||||
|
|
Loading…
Reference in New Issue