#21677 closed Cleanup/optimization (fixed)

Review Apps.get_model[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


Following the recent work on the app loading project, it would be useful to review, and possibly optimize, the implementation of these methods and to add tests (probably based on https://code.djangoproject.com/attachment/ticket/3591/app-loading.2.diff#L1827).

Change History (5)

comment:1 Changed 22 months ago by aaugustin

  • Triage Stage changed from Unreviewed to Accepted

comment:2 Changed 22 months ago by aaugustin

Have a look at get_migratable_models too.

comment:4 Changed 22 months ago by aaugustin

only_installed is gone: https://github.com/django/django/compare/81a5e35c8d35...9f13c3328199

Now the next step is to change get_models to stop taking an models module as first argument.

comment:5 Changed 22 months ago by aaugustin

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.
Back to Top