• 
      

    Fix {% template_hook_point %} when using RequestContext.

    Review Request #14155 — Created Sept. 11, 2024 and submitted — Latest diff uploaded

    Information

    Djblets
    release-5.x

    Reviewers

    In the case of unit tests, we may be rendering some templates that
    include {% template_hook_point %} without going through all our usual
    context processor machinery. In this case, we can sometimes hit cases
    where we're calling this tag with a RequestContext that does not
    actually have request in the dict contents. This was causing some unit
    test failures.

    This change makes it so our template tag will check if we have a
    RequestContext, and if so, get the request from the attribute instead
    of trying to pull it out of the dict. This fixes the cases where unit
    tests were crashing.

    Ran tests in both Djblets and Review Board (the latter being where I
    was hitting these issues).

    Commits

    Files