ResolverMatch repr is incorrect for Class Based Views

[Keryn Knight]

The following test applies cleanly to 3.2.9, and AFAIK would apply roughly correctly all the way back to when CBVs were introduced (I can't easily test without going back to a super old Python and finding the test cases, which have moved around):

"""
add to class: tests.urlpatterns_reverse.tests.ResolverMatchTests
"""
    @override_settings(ROOT_URLCONF='urlpatterns_reverse.reverse_lazy_urls')
    def test_classbased_repr(self):
        self.assertEqual(
            repr(resolve('/redirect/')),
            "ResolverMatch(func=urlpatterns_reverse.views.LazyRedirectView, "
            "args=(), kwargs={}, url_name=None, app_names=[], "
            "namespaces=[], route=redirect/)",
        )

The _func_path as AFAIK always been a representation to the fully qualified dotted callable where possible, that is for a CBV it's the CBV module + the class name.

As of 4.0, the _func_path has become urlpatterns_reverse.views.view, I believe because of #32260 removing the use of update_wrapper and intentionally not setting the __name__ and __qualname__ in favour using the view_class attribute, as per the comment view_class should be used to robustly determine the name of the view (see ​Pull Request)

Unfortunately I think that means the detection of class based views in ResolverMatch no longer works correctly, and this can probably only be resolved by making ResolverMatch CBV aware again by embedding detection of view_class therein.

Noted it as a question in ​this PR for ticket #33396, but hoisting it here properly to be considered separately.

The fix appears to ostensibly the same as for #33425 (see ​PR)

class ResolverMatch:
    def __init__(...):
        # ...
        if hasattr(func, 'view_class'):
            func = func.view_class
        if not hasattr(func, '__name__'):
        # ...

I have a branch which I shall push shortly to confirm it works generally.

[Mariusz Felisiak]

Thanks for the report! ​PR

Regression in 7c08f26bf0439c1ed593b51b51ad847f7e262bc1.

[Mariusz Felisiak <felisiak.mariusz@…>]

In f4b06a3:

Fixed #33426 -- Fixed ResolverMatch.repr_() for class-based views.

Regression in 7c08f26bf0439c1ed593b51b51ad847f7e262bc1.

[Mariusz Felisiak <felisiak.mariusz@…>]

In c8a6bf95:

[4.0.x] Fixed #33426 -- Fixed ResolverMatch.repr_() for class-based views.

Regression in 7c08f26bf0439c1ed593b51b51ad847f7e262bc1.

Backport of f4b06a3cc1e54888ff86f36a1f9a3ddf21292314 from main

[Mariusz Felisiak <felisiak.mariusz@…>]

In d05ab13c:

Refs #33426 -- Simplified technical_404_response() with ResolverMatch._func_path.