Re: ModelAdmin API (formfield_for_*)

2016-01-23 Thread Yoong Kang Lim
Good point, I suppose it'll be better if it were in forms. I'll be looking into this. On Sunday, January 24, 2016 at 4:14:22 AM UTC+11, Tim Graham wrote: > > For a quick glance, the high-level idea looks okay (subject to a > deprecation path, of course), however, in my mind it would be more usef

Re: ModelAdmin API (formfield_for_*)

2016-01-23 Thread Tim Graham
For a quick glance, the high-level idea looks okay (subject to a deprecation path, of course), however, in my mind it would be more useful to try to move toward enhancing the forms library itself (if possible) so that less admin-specific code is needed. Have you thought about this at all? On Sa

ModelAdmin API (formfield_for_*)

2016-01-23 Thread Yoong Kang Lim
Currently the BaseModelAdmin class has a method called formfield_for_dbfield which has a conditional block that in turn calls different formfield_for_* methods depending on the type of database field it is handling. In order to do any custom logic to these fields we need to override these metho