Views should be given the instance namespace retrieved during the resolve process

[GaretJax]

During the URL resolving process, the instance namespace is returned as part of the match object but then never propagated to the final view or made available to other middlewares (see ​https://github.com/django/django/blob/master/django/core/handlers/base.py#L103).

This value could be made available to the view as part of the request object (an attribute?) or another way which does not involve a second resolve call. It can then also be used as a default value for the 'current_app' argument of the 'render' shortcut or the RequestContext class.

[GaretJax]

I already create a PR for this (it got rejected as explained in ​https://github.com/django/django/pull/260) but I can edit to comply with the requested points.

Before proceeding though, I'd like to discuss the addition of the 'current_app' attribute to the HttpRequest class. Someone sees a better way to design this?

[GaretJax]

Looking at how other solutions which add attributes to the Request instance resolved the problem, it seems like that the attribute should be set by the get_response method but without initialization in the HttpRequest constructor.

[Malcolm Tredinnick]

Since the initial days of namespaces, this has been a "might be nice to have one day" kind of feature, so I'm not against the idea. The current patch -- ignoring any deficiencies -- looks like it's not going to be an incredibly intrusive change, either. So a good patch with tests would probably be worth committing for this.

I guess we need to work out what the exact format of the passed-through data is, but that's not going to change the implementation much. I'm currently updating the namespace documentation to ... well ... make sense, so this is going to come up more in the future as people start to use it.

[GaretJax]

Beside regrouping everything in a single commit, is there anything I can adapt in my proposal (​https://github.com/django/django/pull/260/files)?

[GaretJax]

Regarding the tests, there already is one checking for the presence of the attribute on the request instance. I can add one to check for the correct behavior of the render shortcut. Is there anything else?

[Luke Plant]

Isn't this superseded by #15695 ? The ResolverMatch object seems to have all the info you want, from a quick look.

[Luke Plant]

Given no response, I'm closing as 'fixed'. Please re-open if #15695 doesn't fix your problem.