Opened 3 years ago

Closed 3 years ago

#21677 closed Cleanup/optimization (fixed)

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

Description

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 3 years ago by Aymeric Augustin

Triage Stage: UnreviewedAccepted

comment:2 Changed 3 years ago by Aymeric Augustin

Have a look at get_migratable_models too.

comment:4 Changed 3 years ago by Aymeric Augustin

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 3 years ago by Aymeric Augustin

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.
Back to Top