Opened 5 years ago

Last modified 5 years ago

#31123 closed New feature

Add option to remove_stale_contenttypes to remove entries for nonexistent apps. — at Version 3

Reported by: Gowtham Kommineni Owned by: nobody
Component: contrib.contenttypes 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: no UI/UX: no

Description (last modified by Javier Buzzi)

Add an option (disabled by default) to remove_stale_contenttypes command to remove entries also for nonexistent apps.

Based on discussion.

PR

Change History (3)

comment:1 by Mariusz Felisiak, 5 years ago

Resolution: wontfix
Status: newclosed
Summary: remove_stale_contenttypes doesn't remove entries for stale appsremove_stale_contenttypes doesn't remove entries for nonexistent apps.

I don't think that remove_stale_contenttypes should remove content types for nonexistent apps. We don't automatically remove models for nonexistent apps and I don't think we should do this with content types. This can also cause a data loss for a custom content types (not related to real apps). You can start a discussion on DevelopersMailingList if you don't agree.

comment:2 by Mariusz Felisiak, 5 years ago

Description: modified (diff)
Has patch: unset
Resolution: wontfix
Status: closednew
Summary: remove_stale_contenttypes doesn't remove entries for nonexistent apps.Add option to remove_stale_contenttypes to remove entries for nonexistent apps.
Triage Stage: UnreviewedAccepted
Type: BugNew feature
Version: 3.0master

Accepted based on discussion .

comment:3 by Javier Buzzi, 5 years ago

Description: modified (diff)
Has patch: set
Last edited 5 years ago by Mariusz Felisiak (previous) (diff)
Note: See TracTickets for help on using tickets.
Back to Top