update error message for after_this_request and copy_current_request_context

This commit is contained in:
David Lord 2021-12-22 18:44:37 -07:00
parent 633449a36c
commit b1a00ebc4c
No known key found for this signature in database
GPG Key ID: 7A1C87E3F5BC42A8
2 changed files with 9 additions and 8 deletions

View File

@ -15,7 +15,7 @@ Unreleased
- ``app.json_encoder`` and ``json_decoder`` are only passed to
``dumps`` and ``loads`` if they have custom behavior. This improves
performance, mainly on PyPy. :issue:`4349`
- Fix error message for ``after_this_request`` when it used outside
- Clearer error message when ``after_this_request`` is used outside a
request context. :issue:`4333`

View File

@ -131,14 +131,14 @@ def after_this_request(f: AfterRequestCallable) -> AfterRequestCallable:
.. versionadded:: 0.9
"""
top = _request_ctx_stack.top
if top is None:
raise RuntimeError(
"This decorator can only be used at local scopes "
"when a request context is on the stack. For instance within "
"view functions."
"This decorator can only be used when a request context is"
" active, such as within a view function."
)
top._after_request_functions.append(f)
top._after_request_functions.append(f)
return f
@ -167,12 +167,13 @@ def copy_current_request_context(f: t.Callable) -> t.Callable:
.. versionadded:: 0.10
"""
top = _request_ctx_stack.top
if top is None:
raise RuntimeError(
"This decorator can only be used at local scopes "
"when a request context is on the stack. For instance within "
"view functions."
"This decorator can only be used when a request context is"
" active, such as within a view function."
)
reqctx = top.copy()
def wrapper(*args, **kwargs):