Skip to content

Commit 89314a0

Browse files
committed
Move new message models from announcements to a new message app
- Legacy-Id: 3856
1 parent 980d4e6 commit 89314a0

8 files changed

Lines changed: 10 additions & 79 deletions

File tree

ietf/announcements/admin.py

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
#coding: utf-8
22
from django.contrib import admin
3-
from django.conf import settings
43
from ietf.announcements.models import *
54

65
class AnnouncedFromAdmin(admin.ModelAdmin):
@@ -21,23 +20,3 @@ class AnnouncementAdmin(admin.ModelAdmin):
2120
class ScheduledAnnouncementAdmin(admin.ModelAdmin):
2221
pass
2322
admin.site.register(ScheduledAnnouncement, ScheduledAnnouncementAdmin)
24-
25-
26-
if settings.USE_DB_REDESIGN_PROXY_CLASSES:
27-
class MessageAdmin(admin.ModelAdmin):
28-
list_display = ["time", "by", "subject", "groups"]
29-
search_fields = ["body"]
30-
raw_id_fields = ["by"]
31-
32-
def groups(self, instance):
33-
return ", ".join(g.acronym for g in related_groups.all())
34-
35-
admin.site.register(Message, MessageAdmin)
36-
37-
class SendQueueAdmin(admin.ModelAdmin):
38-
list_display = ["time", "by", "message", "send_at", "sent_at"]
39-
list_filter = ["time", "send_at", "sent_at"]
40-
search_fields = ["message__body"]
41-
raw_id_fields = ["by"]
42-
43-
admin.site.register(SendQueue, SendQueueAdmin)

ietf/announcements/models.py

Lines changed: 0 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -86,45 +86,3 @@ def __str__(self):
8686
return "Scheduled Announcement from %s to %s on %s %s" % (self.from_val, self.to_val, self.to_be_sent_date, self.to_be_sent_time)
8787
class Meta:
8888
db_table = 'scheduled_announcements'
89-
90-
91-
if settings.USE_DB_REDESIGN_PROXY_CLASSES or hasattr(settings, "IMPORTING_FROM_OLD_SCHEMA"):
92-
import datetime
93-
94-
from ietf.person.models import Email, Person
95-
from ietf.group.models import Group
96-
97-
class Message(models.Model):
98-
time = models.DateTimeField(default=datetime.datetime.now)
99-
by = models.ForeignKey(Person)
100-
101-
subject = models.CharField(max_length=255)
102-
frm = models.CharField(max_length=255)
103-
to = models.CharField(max_length=1024)
104-
cc = models.CharField(max_length=1024, blank=True)
105-
bcc = models.CharField(max_length=255, blank=True)
106-
reply_to = models.CharField(max_length=255, blank=True)
107-
body = models.TextField()
108-
content_type = models.CharField(max_length=255, blank=True)
109-
110-
related_groups = models.ManyToManyField(Group, blank=True)
111-
112-
class Meta:
113-
ordering = ['time']
114-
115-
def __unicode__(self):
116-
return "'%s' %s -> %s" % (self.subject, self.frm, self.to)
117-
118-
class SendQueue(models.Model):
119-
time = models.DateTimeField(default=datetime.datetime.now)
120-
by = models.ForeignKey(Person)
121-
122-
message = models.ForeignKey(Message)
123-
124-
send_at = models.DateTimeField(blank=True, null=True)
125-
sent_at = models.DateTimeField(blank=True, null=True)
126-
127-
note = models.TextField(blank=True)
128-
129-
class Meta:
130-
ordering = ['time']

ietf/announcements/sitemaps.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,12 +3,12 @@
33
from django.conf import settings
44
from django.contrib.sitemaps import Sitemap
55
from ietf.announcements.models import Announcement
6+
from ietf.message.models import Message
67

78
class NOMCOMAnnouncementsMap(Sitemap):
89
changefreq = "never"
910
def items(self):
1011
if settings.USE_DB_REDESIGN_PROXY_CLASSES:
11-
from ietf.announcements.models import Message
1212
return Message.objects.filter(related_groups__acronym__startswith="nomcom").exclude(related_groups__acronym="nomcom").order_by('-time')
1313
return Announcement.objects.all().filter(nomcom=True)
1414
def location(self, obj):

ietf/announcements/tests.py

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,9 @@
99

1010
from ietf.announcements.models import ScheduledAnnouncement
1111

12+
from ietf.message.models import Message, SendQueue
13+
from ietf.person.models import Person
14+
1215
class AnnouncementsUrlTestCase(SimpleUrlTestCase):
1316
def testUrls(self):
1417
self.doTestUrls(__file__)
@@ -67,9 +70,6 @@ class SendScheduledAnnouncementsTestCaseREDESIGN(django.test.TestCase):
6770
fixtures = ["names"]
6871

6972
def test_send_plain_announcement(self):
70-
from ietf.announcements.models import Message, SendQueue
71-
from ietf.person.models import Person
72-
7373
make_test_data()
7474

7575
msg = Message.objects.create(
@@ -99,9 +99,6 @@ def test_send_plain_announcement(self):
9999
self.assertTrue(SendQueue.objects.get(id=q.id).sent_at)
100100

101101
def test_send_mime_announcement(self):
102-
from ietf.announcements.models import Message, SendQueue
103-
from ietf.person.models import Person
104-
105102
make_test_data()
106103

107104
msg = Message.objects.create(

ietf/announcements/urls.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,5 +12,5 @@
1212
urlpatterns = patterns('',
1313
# (r'^nomcom/$', 'django.views.generic.simple.redirect_to', {'url': 'http://www.ietf.org/nomcom/index.html'} ),
1414
(r'^nomcom/$', 'ietf.announcements.views.nomcom'),
15-
(r'^nomcom/(?P<object_id>\d+)/$', 'ietf.announcements.views.message_detail' if settings.USE_DB_REDESIGN_PROXY_CLASSES else 'django.views.generic.list_detail.object_detail', nomcom_dict)
15+
(r'^nomcom/(?P<object_id>\d+)/$', 'ietf.announcements.views.message_detail') if settings.USE_DB_REDESIGN_PROXY_CLASSES else (r'^nomcom/(?P<object_id>\d+)/$', 'django.views.generic.list_detail.object_detail', nomcom_dict)
1616
)

ietf/announcements/views.py

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,8 @@
1010
from ietf.idtracker.models import ChairsHistory
1111
from ietf.idtracker.models import Role
1212
from ietf.announcements.models import Announcement
13+
from ietf.group.models import Group, GroupEvent
14+
from ietf.message.models import Message
1315

1416
def nomcom(request):
1517
curr_chair = (ChairsHistory.objects.
@@ -35,9 +37,6 @@ def nomcom(request):
3537
'regimes' : regimes })
3638

3739
def nomcomREDESIGN(request):
38-
from ietf.group.models import Group, GroupEvent
39-
from ietf.announcements.models import Message
40-
4140
address_re = re.compile("<.*>")
4241

4342
nomcoms = list(Group.objects.filter(acronym__startswith="nomcom").exclude(name="nomcom"))
@@ -73,10 +72,7 @@ def nomcomREDESIGN(request):
7372
nomcom = nomcomREDESIGN
7473

7574

76-
def message_detail(request, object_id, queryset):
77-
from ietf.group.models import Group
78-
from ietf.announcements.models import Message
79-
75+
def message_detail(request, object_id):
8076
# restrict to nomcom announcements for the time being
8177
nomcoms = Group.objects.filter(acronym__startswith="nomcom").exclude(acronym="nomcom")
8278
m = get_object_or_404(Message, id=object_id,

ietf/settings.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -139,6 +139,7 @@
139139
'ietf.name',
140140
'ietf.group',
141141
'ietf.doc',
142+
'ietf.message',
142143
'ietf.announcements',
143144
'ietf.idindex',
144145
'ietf.idtracker',

redesign/importing/import-announcements.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@
1515
from ietf.person.models import *
1616
from ietf.group.models import *
1717
from ietf.name.utils import name
18+
from ietf.message.models import Message, SendQueue
1819
from ietf.importing.utils import old_person_to_person
19-
from ietf.announcements.models import Message, SendQueue
2020
from ietf.announcements.models import Announcement, PersonOrOrgInfo, AnnouncedTo, AnnouncedFrom, ScheduledAnnouncement
2121
from ietf.idtracker.models import IESGLogin
2222

0 commit comments

Comments
 (0)