From 10e3faf191d8f230dde8534d1c8fad8c8717816e Mon Sep 17 00:00:00 2001 From: Simon Charette Date: Wed, 22 Jan 2014 01:43:33 -0500 Subject: Fixed #19774 -- Deprecated the contenttypes.generic module. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit It contained models, forms and admin objects causing undesirable import side effects. Refs #16368. Thanks to Ramiro, Carl and Loïc for the review. --- tests/generic_inline_admin/admin.py | 12 ++++++------ tests/generic_inline_admin/models.py | 10 ++++++---- tests/generic_inline_admin/tests.py | 4 ++-- 3 files changed, 14 insertions(+), 12 deletions(-) (limited to 'tests/generic_inline_admin') diff --git a/tests/generic_inline_admin/admin.py b/tests/generic_inline_admin/admin.py index c701e4a9eb..d0ce4c70ef 100644 --- a/tests/generic_inline_admin/admin.py +++ b/tests/generic_inline_admin/admin.py @@ -1,5 +1,5 @@ from django.contrib import admin -from django.contrib.contenttypes import generic +from django.contrib.contenttypes.admin import GenericTabularInline from .models import (Media, PhoneNumber, Episode, EpisodeExtra, Contact, Category, EpisodePermanent, EpisodeMaxNum) @@ -8,7 +8,7 @@ from .models import (Media, PhoneNumber, Episode, EpisodeExtra, Contact, site = admin.AdminSite(name="admin") -class MediaInline(generic.GenericTabularInline): +class MediaInline(GenericTabularInline): model = Media @@ -18,22 +18,22 @@ class EpisodeAdmin(admin.ModelAdmin): ] -class MediaExtraInline(generic.GenericTabularInline): +class MediaExtraInline(GenericTabularInline): model = Media extra = 0 -class MediaMaxNumInline(generic.GenericTabularInline): +class MediaMaxNumInline(GenericTabularInline): model = Media extra = 5 max_num = 2 -class PhoneNumberInline(generic.GenericTabularInline): +class PhoneNumberInline(GenericTabularInline): model = PhoneNumber -class MediaPermanentInline(generic.GenericTabularInline): +class MediaPermanentInline(GenericTabularInline): model = Media can_delete = False diff --git a/tests/generic_inline_admin/models.py b/tests/generic_inline_admin/models.py index af3a474cdf..0ae3c4b052 100644 --- a/tests/generic_inline_admin/models.py +++ b/tests/generic_inline_admin/models.py @@ -1,4 +1,6 @@ -from django.contrib.contenttypes import generic +from django.contrib.contenttypes.fields import ( + GenericForeignKey, GenericRelation +) from django.contrib.contenttypes.models import ContentType from django.db import models from django.utils.encoding import python_2_unicode_compatible @@ -17,7 +19,7 @@ class Media(models.Model): """ content_type = models.ForeignKey(ContentType) object_id = models.PositiveIntegerField() - content_object = generic.GenericForeignKey() + content_object = GenericForeignKey() url = models.URLField() description = models.CharField(max_length=100, blank=True) keywords = models.CharField(max_length=100, blank=True) @@ -56,7 +58,7 @@ class Category(models.Model): class PhoneNumber(models.Model): content_type = models.ForeignKey(ContentType) object_id = models.PositiveIntegerField() - content_object = generic.GenericForeignKey('content_type', 'object_id') + content_object = GenericForeignKey('content_type', 'object_id') phone_number = models.CharField(max_length=30) category = models.ForeignKey(Category, null=True, blank=True) @@ -66,7 +68,7 @@ class PhoneNumber(models.Model): class Contact(models.Model): name = models.CharField(max_length=50) - phone_numbers = generic.GenericRelation(PhoneNumber) + phone_numbers = GenericRelation(PhoneNumber) # diff --git a/tests/generic_inline_admin/tests.py b/tests/generic_inline_admin/tests.py index e35f0d639a..b223c98298 100644 --- a/tests/generic_inline_admin/tests.py +++ b/tests/generic_inline_admin/tests.py @@ -4,8 +4,8 @@ import warnings from django.contrib import admin from django.contrib.admin.sites import AdminSite -from django.contrib.contenttypes.generic import ( - generic_inlineformset_factory, GenericTabularInline) +from django.contrib.contenttypes.admin import GenericTabularInline +from django.contrib.contenttypes.forms import generic_inlineformset_factory from django.forms.formsets import DEFAULT_MAX_NUM from django.forms.models import ModelForm from django.test import TestCase, override_settings -- cgit v1.3