Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#23078 closed Bug (fixed)

update_contenttypes interactive kwarg will always be False

Reported by: raymond@… Owned by: nobody
Component: contrib.contenttypes Version: 1.7-rc-1
Severity: Release blocker 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

I've been noticing that stale content types are no longer removed during syncdb / migrate. Looking at the master and the c1 branch I've found the problem.

django/contrib/contenttypes/management.py line 63: kwargs.get('interactive', False)

Will always be False due to the function signature which defines kwarg interactive=True.

Change History (10)

comment:1 by Tim Graham, 11 years ago

Component: Uncategorized → contrib.contenttypes
Easy pickings: unset
Severity: Normal → Release blocker
Triage Stage: Unreviewed → Accepted

The interactive kwarg was introduced in 00110904ac67050c639f93cfd7b5e19218bcef5b.

comment:2 by merb, 11 years ago

I added a pr for this: ​https://github.com/django/django/pull/2942

Edit: God Trac is so fucking dumb..

Last edited 11 years ago by merb (previous) (diff)

comment:3 by merb, 11 years ago

Needs tests: set
Resolution: → fixed
Status: new → closed

comment:4 by merb, 11 years ago

Resolution: fixed
Status: closed → new

comment:5 by Nick Sandford, 11 years ago

Has patch: set
Needs tests: unset

I added some tests, PR here: ​https://github.com/django/django/pull/2944

comment:6 by Nick Sandford, 11 years ago

Triage Stage: Accepted → Ready for checkin

comment:7 by Tim Graham <timograham@…>, 11 years ago

Resolution: → fixed
Status: new → closed

In fa3cf006b2912708e8c0b294cea91933cf35781f:

Fixed #23078 -- Regression in update_contenttypes() interactivity.

Thanks raymond at adaptiv.nl for the report.

comment:8 by Tim Graham <timograham@…>, 11 years ago

In 9638daec5cb584dc3b62b7373e0f7cd2fc808b7e:

[1.7.x] Fixed #23078 -- Regression in update_contenttypes() interactivity.

Thanks raymond at adaptiv.nl for the report.

Backport of fa3cf006b2 from master

comment:9 by Tim Graham <timograham@…>, 11 years ago

In 936fba5878075576570ab7d2c84bfba80d183c18:

Improved patching of sys.stdout in refs #23078.

comment:10 by Tim Graham <timograham@…>, 11 years ago

In 7cbb4f79d1e3f1690fb31b118f47590d9470b59b:

[1.7.x] Improved patching of sys.stdout in refs #23078.

Backport of 936fba5878 from master

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