#21689 closed Cleanup/optimization (fixed)

Remove the only_with_models_module argument of get_app_config[s]

Reported by: aaugustin 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

Description

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 17 months ago by aaugustin

  • Triage Stage changed from Unreviewed to Accepted

comment:2 Changed 17 months ago by Aymeric Augustin <aymeric.augustin@…>

  • Resolution set to fixed
  • Status changed from new to closed

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