Opened 4 years ago

Closed 4 years ago

#21689 closed Cleanup/optimization (fixed)

Remove the only_with_models_module argument of get_app_config[s]

Reported by: Aymeric Augustin Owned by: nobody
Component: Core (Other) Version: master
Severity: Normal Keywords: app-loading
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


Once #21676 is done, the current logic that calls either populate_apps or populate_models won't make sense any more.

At this point it would be more straightforward to remove this argument and let callers filter the results by themselves.

Change History (2)

comment:1 Changed 4 years ago by Aymeric Augustin

Triage Stage: UnreviewedAccepted

comment:2 Changed 4 years ago by Aymeric Augustin <aymeric.augustin@…>

Resolution: fixed
Status: newclosed

In bfcc686d22df10c268752635947dd242317ba156:

Removed the only_with_models_module argument of get_model[s].

Now that the refactorings are complete, it isn't particularly useful any
more, nor very well named. Let's keep the API as simple as possible.

Fixed #21689.

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