Add more hooks in the ModelAdmin (get_extra, get_max_num)

[d.willy.c.c@…]

in class InlineModelAdmin and def get_formset in line 1401
defaults = {

"form": self.form,
"formset": self.formset,
"fk_name": self.fk_name,
"fields": fields,
"exclude": exclude,
"formfield_callback": partial(self.formfield_for_dbfield, request=request),
"extra": self.extra,
"max_num": self.max_num,
"can_delete": can_delete,

}

I think that is more customizable code should be as follows
"form": self.form,

"formset": self.formset,
"fk_name": self.fk_name,
"fields": fields,
"exclude": exclude,
"formfield_callback": partial(self.formfield_for_dbfield, request=request),
"extra": self.get_extra(),
"max_num": self.get_max_num(),
"can_delete": can_delete,

[rasca]

This is in tone with #17646, #17006 and #16841.

I'm accepting this based on that they are similar to the other ones, adding hooks to the ModelAdmin.

Maybe we can think a way to add this "hooks" in a more DRY way than adding a function get_FOO(self, request): return self.FOO

[anonymous]

I think 'GET' is the easiest way to personalize this, and also believe that all variables should have a 'GET'

[Lebedev Ilya]

This patch is correct. There was some garbage in previous one, sorry for that.

[javimansilla]

#19425 is proposing something similar and IMHO it's doing it on a better way

[Areski Belaid]

I agree also that #19425 is more elegant, I rebased the patch from that ticket and added max_num as per the suggestion here.
See patch in attachment and the PR : ​https://github.com/django/django/pull/1155

[Areski Belaid]

Just updated the patch to add some documentation

[Tim Graham]

InlineModelAdmin.get_extra was added in #19425. I'll try to review and commit the second half of this.

[Tim Graham <timograham@…>]

In 61524b09cfa3b51643d0e79cbf0e1e08ede357ae:

Fixed #18388 - Added InlineModelAdmin.get_max_num hook.

Thanks d.willy.c.c@ for the suggestion and Melevir and Areski Belaid for work
on the patch.