Skip to content

Commit cd1a024

Browse files
committed
remove some obsolete models
1 parent 460bc0e commit cd1a024

15 files changed

Lines changed: 12 additions & 476 deletions

File tree

core/admin.py

Lines changed: 3 additions & 55 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,6 @@
77
from django.contrib.flatpages.models import FlatPage
88
from django.db.models import Q, TimeField
99
from django.forms import Textarea
10-
from django.urls import reverse
11-
from django.utils.html import format_html
12-
from django.utils.safestring import mark_safe
1310
from django.utils.translation import gettext_lazy as _
1411
from martor.widgets import AdminMartorWidget
1512
from reversion.admin import VersionAdmin
@@ -26,9 +23,8 @@
2623
UserAdminForm,
2724
UserCreationAdminForm,
2825
)
29-
from .models import Comment, StaffMember
26+
from .models import StaffMember
3027
from .utils.actions import (
31-
approve_comments,
3228
archive_page,
3329
normalize_late_start,
3430
resend_approval_email,
@@ -42,15 +38,13 @@
4238
set_event_visible,
4339
set_post_archived,
4440
set_post_unarchived,
45-
unapprove_comments,
4641
wipe_club_bios,
4742
)
4843
from .utils.admin import generic_post_formfield_for_manytomany
4944
from .utils.announcements import request_announcement_approval
5045
from .utils.filters import (
5146
BlogPostAuthorListFilter,
5247
OrganizationListFilter,
53-
PostTypeFilter,
5448
)
5549

5650
User = get_user_model()
@@ -207,8 +201,8 @@ def get_form(self, request, obj=None, **kwargs):
207201

208202

209203
class PostAdmin(admin.ModelAdmin):
210-
readonly_fields = ["like_count", "save_count", "comments"]
211-
fields = ["like_count", "save_count", "comments"]
204+
readonly_fields = ["like_count", "save_count"]
205+
fields = ["like_count", "save_count"]
212206

213207
def like_count(self, obj) -> int:
214208
return obj.like_count
@@ -223,15 +217,6 @@ def save_count(self, obj) -> int:
223217

224218
save_count.short_description = "Save Count"
225219

226-
def comments(self, obj):
227-
objs = [
228-
f'<a target="_blank" href="/admin/core/comment/{comment.pk}">{comment.body[:10]}</a>'
229-
for comment in obj.comments.all()
230-
]
231-
return mark_safe(",".join(objs))
232-
233-
comments.short_description = "Comments"
234-
235220
class Meta:
236221
abstract = True
237222

@@ -714,40 +699,6 @@ class CustomFlatPageAdmin(FlatPageAdmin):
714699
)
715700

716701

717-
class RaffleAdmin(admin.ModelAdmin):
718-
list_display = ["__str__", "open_start", "open_end"]
719-
720-
721-
class CommentAdmin(admin.ModelAdmin):
722-
formfield_overrides = {
723-
django.db.models.TextField: {"widget": AdminMartorWidget},
724-
}
725-
list_display = ["author", "content_object", "created_at"]
726-
search_fields = ["author__username", "body"]
727-
actions = [approve_comments, unapprove_comments]
728-
readonly_fields = ["created_at", "likes"]
729-
list_filter = ["live", PostTypeFilter]
730-
actions_on_top = True
731-
actions_on_bottom = True
732-
date_hierarchy = "created_at"
733-
734-
@staticmethod
735-
def likes(obj: Comment):
736-
return obj.like_count
737-
738-
def get_queryset(self, request):
739-
return Comment.objects.filter(author__isnull=False).order_by("-created_at")
740-
741-
def content_object(self, obj):
742-
url = reverse(
743-
f"admin:{obj.content_type.app_label}_{obj.content_type.model}_change",
744-
args=[obj.object_id],
745-
)
746-
return format_html('<a href="{}">{}</a>', url, str(obj.content_object))
747-
748-
content_object.short_description = "Associated Post"
749-
750-
751702
class BannerAdmin(admin.ModelAdmin):
752703
list_display = ["name", "start_date", "end_date"]
753704
search_fields = ["name"]
@@ -760,14 +711,11 @@ class BannerAdmin(admin.ModelAdmin):
760711
admin.site.register(models.SchedulePattern, SchedulePatternAdmin)
761712
admin.site.register(models.ScheduleOverride, ScheduleOverrideAdmin)
762713
admin.site.register(models.Organization, OrganizationAdmin)
763-
admin.site.register(models.DailyAnnouncement, DailyAnnouncementAdmin)
764714
admin.site.register(models.Announcement, AnnouncementAdmin)
765715
admin.site.register(models.BlogPost, BlogPostAdmin)
766716
admin.site.register(models.Exhibit, ExhibitAdmin)
767-
# admin.site.register(models.Comment, CommentAdmin) atm it's not used, so we don't need it
768717
admin.site.register(models.Tag, TagAdmin)
769718
admin.site.register(models.Event, EventAdmin)
770-
admin.site.register(models.Raffle, RaffleAdmin)
771719
admin.site.register(models.StaffMember)
772720
admin.site.register(models.Banner, BannerAdmin)
773721

core/api/serializers/custom/__init__.py

Lines changed: 1 addition & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77

88
from core.api.utils.github import get_model_choices
99
from core.api.utils.gravatar import gravatar_url
10-
from core.models import Comment, Organization, Tag, User
10+
from core.models import Organization, Tag, User
1111

1212

1313
class PrimaryKeyAndSlugRelatedField(serializers.SlugRelatedField):
@@ -92,33 +92,6 @@ def to_representation(self, obj):
9292
}
9393

9494

95-
class CommentSerializer(serializers.ModelSerializer):
96-
author = SingleUserSerializer()
97-
has_children = serializers.SerializerMethodField(read_only=True)
98-
edited = serializers.SerializerMethodField(read_only=True)
99-
likes = LikeField()
100-
101-
@staticmethod
102-
def get_edited(obj: Comment) -> bool:
103-
return obj.last_modified != obj.created_at
104-
105-
@staticmethod
106-
def get_has_children(obj: Comment) -> bool:
107-
return obj.children.exists()
108-
109-
class Meta:
110-
model = Comment
111-
fields = [
112-
"id",
113-
"body",
114-
"author",
115-
"has_children",
116-
"created_at",
117-
"edited",
118-
"likes",
119-
]
120-
121-
12295
class SingleUserField(ChoiceField):
12396
def __init__(self, **kwargs):
12497
self.slug_field = "username"
@@ -282,12 +255,3 @@ def to_internal_value(self, data):
282255
)
283256

284257
return Tag.objects.filter(id__in=data)
285-
286-
287-
class CommentField(Field):
288-
def __init__(self, **kwargs):
289-
kwargs["read_only"] = True
290-
super().__init__(**kwargs)
291-
292-
def to_representation(self, obj):
293-
return CommentSerializer(obj, many=True).data

core/api/utils/polymorphism.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@
1515
from core.api.v3.objects import (
1616
AnnouncementProvider,
1717
BlogPostProvider,
18-
CommentProvider,
1918
EventProvider,
2019
ExhibitProvider,
2120
FlatPageProvider,
@@ -85,7 +84,6 @@ def get_path_by_provider(provider: BaseProvider) -> str:
8584
"user": UserProvider,
8685
"tag": TagProvider,
8786
"timetable": TimetableProvider,
88-
"comment": CommentProvider,
8987
"like": LikeProvider,
9088
}
9189

@@ -118,7 +116,7 @@ def get_providers_by_operation(
118116
119117
Example:
120118
>>> get_providers_by_operation("single")
121-
["announcement", "blog-post", "exhibit", "event", "organization", "flatpage", "user", "tag", "timetable", "comment", "like"]
119+
["announcement", "blog-post", "exhibit", "event", "organization", "flatpage", "user", "tag", "timetable", "like"]
122120
"""
123121
operation = operation.lower()
124122
return [

core/api/v3/objects/__init__.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
from .exhibit import ExhibitProvider
55
from .flatpage import FlatPageProvider
66
from .organization import OrganizationProvider
7-
from .post_interactions import CommentProvider, LikeProvider
7+
from .post_interactions import LikeProvider
88
from .tag import TagProvider
99
from .timetable import TimetableProvider
1010
from .user import UserProvider
@@ -18,7 +18,6 @@
1818
"TagProvider",
1919
"TimetableProvider",
2020
"UserProvider",
21-
"CommentProvider",
2221
"LikeProvider",
2322
"FlatPageProvider",
2423
]

core/api/v3/objects/announcement.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
from rest_framework.exceptions import ValidationError
99

1010
from core.api.serializers.custom import (
11-
CommentField,
1211
LikeField,
1312
OrganizationField,
1413
SingleUserField,
@@ -33,7 +32,6 @@ def always_fail_validator(value, serializer_field):
3332

3433
class Serializer(serializers.ModelSerializer):
3534
message = serializers.CharField(read_only=True)
36-
comments = CommentField()
3735
likes = LikeField()
3836
tags = TagRelatedField()
3937
author = SingleUserField()
@@ -99,7 +97,6 @@ class Meta:
9997
"supervisor",
10098
"tags",
10199
"likes",
102-
"comments",
103100
"message",
104101
]
105102

core/api/v3/objects/blog_post.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
from rest_framework import permissions, serializers
44

55
from core.api.serializers.custom import (
6-
CommentField,
76
LikeField,
87
SingleUserField,
98
TagRelatedField,
@@ -15,7 +14,6 @@
1514

1615
class Serializer(serializers.ModelSerializer):
1716
likes = LikeField()
18-
comments = CommentField()
1917
author = SingleUserField()
2018
tags = TagRelatedField()
2119

@@ -44,7 +42,6 @@ class Meta:
4442
"is_published",
4543
"tags",
4644
"likes",
47-
"comments",
4845
]
4946

5047

core/api/v3/objects/exhibit.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
from rest_framework import permissions, serializers
33

44
from core.api.serializers.custom import (
5-
CommentField,
65
LikeField,
76
SingleUserField,
87
TagRelatedField,
@@ -14,7 +13,6 @@
1413

1514
class Serializer(serializers.ModelSerializer):
1615
likes = LikeField()
17-
comments = CommentField()
1816
tags = TagRelatedField()
1917
author = SingleUserField()
2018

@@ -34,7 +32,6 @@ class Meta:
3432
"show_after",
3533
"tags",
3634
"likes",
37-
"comments",
3835
]
3936

4037

0 commit comments

Comments
 (0)