autodoc: fix ordering of class and static methods for groupwise order

[picnixz]

I don't have an issue yet (but I'll create one and update the changelog once I'm done) but I've found the issue while working on #13200.

Class and static methods were meant to be ordered before regular methods when using groupwise order but this was not respected until now. The reason is that we call sort_members before calling generate (which is fine). However, generate() actually calls import_object which itself modifies member_order (because that's the first time we have the actual object and we can inspect it without relying on static analysis).

So, we need to decouple import_object from generate. To avoid breaking stuff for now, I've added a private _generate() method which is the actual implementation of generate() so that I can call it separately. Current uses of generate() remain the same.

[AA-Turner]

Looks good! As noted in the PR description, this should have some documentation (at least a CHANGES entry).

A

[picnixz]

Looks good! As noted in the PR description, this should have some documentation (at least a CHANGES entry).

Yes, I planned to do that but I wanted to first hear your thoughts on this. I'll open a separate issue if you want but I'll definitely add the CHANGELOG and update the docs.

[AA-Turner]

No need for a distinct issue.