Document (deprecate?) Paginator._get_page

[Josh Schneier]

This has been undocumented since it was introduced in ~1.6 although it is meant for public use to override the Page class used by a Paginator.

Assuming we keep it (is anyone using it?) I think I like something along the approach of the generic views. We could add a get_page_class method and a page_class attribute although I wonder if the latter isn't enough. We'd need the former to maintain feature parity.

[Carlton Gibson]

Moving to get_page_class() plus page_class seems reasonable.

• I know it's not public but maybe we should deprecate _get_page(), since it's been around a while. (Tim?)
• If so, there's a slight wiggle: _get_page() returns the instance where get_page_class() would presumably give us the class, which we then instantiate in page().

Not sure whether just having the attribute would be enough? Maybe someone is customising for first or last page, as one thought. We may as well leave it available (rather than needing to re-add it later.)

[Josh Schneier]

I regret suggesting get_page for the piece where we inlined the docs now. Maybe something like get_safe_page would have been better. That is a minor issue, I do wonder if that particular aspect of the functionality is used.