GenericInlineModelAdmin doesn't respect ModelAdmin.get_exclude()

[Tim Graham]

The ModelAdmin.get_exclude() hook was added in #24941, however, GenericInlineModelAdmin ​doesn't use it.

[Elias Stinson]

Going to move this triage to accepted, I would recommend someone who knows more about the admin model to go and fix it. I’m also going to change the status to clean up just because the feature is working just not using the efficient function. Good luck!

[MANAS MADESHIYA]

I have submitted a PR for this ticket:
​PR

[Elias Stinson]

The PR looks great, and I'll just recommend to add yourself to the contributors section, just for your own benefit. Someone else can check me on this, but I think this would warrant creating a documentation page for this, as the feature I don't think has one already, or at least it should be improved to now state that the function is called by django. Anyways, I'll mark it both for ready to checkin, and needs documentation for said fix.

[Jacob Walls]

If you're suggesting the patch needs documentation, that's inconsistent with marking it Ready for Checkin.

I'm going to second the advice I just mentioned in another ticket to ask you to slow down and read the triage workflow docs, and reduce the frequency of your participation until you demonstrate more understanding. Thanks.

[MANAS MADESHIYA]

Hi @jacobtylerwalls ,

I apologize for the incomplete AI disclosure.
I used Claude (claude.ai) as an AI assistant
while working on this fix. I reviewed and
verified all code changes myself and ran
the full test suite locally (21 tests passing).

I would like to submit a new PR with proper
disclosure. May I do so?

Thank you for your patience.

[Jacob Walls]

Original PR reopened.