Opened 10 years ago

Closed 10 years ago

#24138 closed Bug (fixed)

Expose modelform_factory in __all__

Reported by: Josh Schneier Owned by: nobody
Component: Forms Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

Today I ran across the fact that modelform_factory is not exposed in __all__/on the top level django.forms module. This doesn't really fit with the rest of the documented forms api and all of the other exposed *factory functions (granted they are all formsets). I think it was an oversight that wasn't corrected when modelform_factory was documented in #10239.

Change History (2)

comment:1 by Collin Anderson, 10 years ago

Has patch: set
Triage Stage: UnreviewedReady for checkin
Type: UncategorizedBug

comment:2 by Tim Graham <timograham@…>, 10 years ago

Resolution: fixed
Status: newclosed

In 3844ccc958f59aef731dbb054a51fa37aa1d16ec:

Fixed #24138 -- Added modelform_factory to all.

Note: See TracTickets for help on using tickets.
Back to Top