Remove mailing list feature

com/models.py

@@ -259,10 +259,7 @@ class Weekmail(models.Model):
         return f"Weekmail {self.id} (sent: {self.sent}) - {self.title}"
 
     def send(self):
-        """Send the weekmail to all users with the receive weekmail option opt-in.
-
-        Also send the weekmail to the mailing list in settings.SITH_COM_EMAIL.
-        """
+        """Send the weekmail to all users with the receive weekmail option opt-in."""
         dest = [
             i[0]
             for i in Preferences.objects.filter(receive_weekmail=True).values_list(

com/templates/com/mailing_admin.jinja

@@ -1,45 +0,0 @@
-{% extends "core/base.jinja" %}
-
-{% block title %}
-  {% trans %}Mailing lists administration{% endtrans %}
-{% endblock %}
-
-{% macro display_mailings(list) %}
-  <table>
-    <tr>
-      <th>{% trans %}Email{% endtrans %}</th>
-      <th>{% trans %}Club{%endtrans%}</th>
-      <th>{% trans %}Actions{% endtrans %}</th>
-    </tr>
-    {% for mailing in list %}
-      <tr>
-        <td>{{ mailing.email_full }}</td>
-        <td><a href="{{ url('club:mailing', club_id=mailing.club.id) }}">{{ mailing.club }}</a></td>
-        <td>
-          <a href="{{ url('com:mailing_delete', mailing_id=mailing.id) }}">{% trans %}Delete{% endtrans %}</a> - {% if not mailing.is_moderated %}<a href="{{ url('com:mailing_moderate', mailing_id=mailing.id) }}">{% trans %}Moderate{% endtrans %}</a>{% else %}{% trans user=mailing.moderator %}Moderated by {{ user }}{% endtrans %}{% endif %}
-        </td>
-      </tr>
-    {% endfor %}
-  </table>
-
-{% endmacro %}
-
-{% block content %}
-  <h1>{% trans %}This page lists all mailing lists{% endtrans %}</h1>
-
-  {% if has_unmoderated %}
-    <h2>{% trans %}Not moderated mailing lists{% endtrans %}</h2>
-    {{ display_mailings(unmoderated) }}
-  {% endif %}
-
-  <h2>{% trans %}Moderated mailing lists{% endtrans %}</h2>
-  {% if has_moderated %}
-    {{ display_mailings(moderated) }}
-  {% else %}
-    <p>{% trans %}No mailing list existing{% endtrans %}</p>
-  {% endif %}
-
-{% endblock %}
-
-
-

com/urls.py

@@ -15,12 +15,9 @@
 
 from django.urls import path
 
-from club.views import MailingDeleteView
 from com.views import (
     AlertMsgEditView,
     InfoMsgEditView,
-    MailingListAdminView,
-    MailingModerateView,
     NewsAdminListView,
     NewsCreateView,
     NewsDeleteView,
@@ -82,17 +79,6 @@ urlpatterns = [
         "news/<int:news_id>/moderate/", NewsModerateView.as_view(), name="news_moderate"
     ),
     path("news/<int:news_id>/", NewsDetailView.as_view(), name="news_detail"),
-    path("mailings/", MailingListAdminView.as_view(), name="mailing_admin"),
-    path(
-        "mailings/<int:mailing_id>/moderate/",
-        MailingModerateView.as_view(),
-        name="mailing_moderate",
-    ),
-    path(
-        "mailings/<int:mailing_id>/delete/",
-        MailingDeleteView.as_view(redirect_page="com:mailing_admin"),
-        name="mailing_delete",
-    ),
     path("poster/", PosterListView.as_view(), name="poster_list"),
     path("poster/create/", PosterCreateView.as_view(), name="poster_create"),
     path("poster/<int:poster_id>/edit/", PosterEditView.as_view(), name="poster_edit"),

com/views.py

@@ -30,11 +30,9 @@ from urllib.parse import urlparse
 from dateutil.relativedelta import relativedelta
 from django.conf import settings
 from django.contrib import messages
-from django.contrib.auth.mixins import (
-    PermissionRequiredMixin,
-)
+from django.contrib.auth.mixins import PermissionRequiredMixin
 from django.contrib.syndication.views import Feed
-from django.core.exceptions import PermissionDenied, ValidationError
+from django.core.exceptions import ValidationError
 from django.db.models import Exists, Max, OuterRef, Value
 from django.forms.models import modelform_factory
 from django.http import HttpResponseRedirect
@@ -46,7 +44,7 @@ from django.utils.translation import gettext_lazy as _
 from django.views.generic import DetailView, ListView, TemplateView, View
 from django.views.generic.edit import CreateView, DeleteView, UpdateView
 
-from club.models import Club, Mailing, Membership
+from club.models import Club, Membership
 from com.forms import NewsDateForm, NewsForm, PosterForm
 from com.ics_calendar import IcsCalendar
 from com.models import News, NewsDate, Poster, Screen, Sith, Weekmail, WeekmailArticle
@@ -86,11 +84,6 @@ class ComTabsMixin(TabedViewMixin):
                 "slug": "alert",
                 "name": _("Alert message"),
             },
-            {
-                "url": reverse("com:mailing_admin"),
-                "slug": "mailings",
-                "name": _("Mailing lists administration"),
-            },
             {
                 "url": reverse("com:poster_list"),
                 "slug": "posters",
@@ -533,37 +526,6 @@ class WeekmailArticleDeleteView(CanEditPropMixin, DeleteView):
     pk_url_kwarg = "article_id"
 
 
-class MailingListAdminView(ComTabsMixin, ListView):
-    template_name = "com/mailing_admin.jinja"
-    model = Mailing
-    current_tab = "mailings"
-
-    def dispatch(self, request, *args, **kwargs):
-        if not (request.user.is_com_admin or request.user.is_root):
-            raise PermissionDenied
-        return super().dispatch(request, *args, **kwargs)
-
-    def get_context_data(self, **kwargs):
-        kwargs = super().get_context_data(**kwargs)
-        kwargs["moderated"] = self.get_queryset().filter(is_moderated=True).all()
-        kwargs["unmoderated"] = self.get_queryset().filter(is_moderated=False).all()
-        kwargs["has_moderated"] = len(kwargs["moderated"]) > 0
-        kwargs["has_unmoderated"] = len(kwargs["unmoderated"]) > 0
-        return kwargs
-
-
-class MailingModerateView(View):
-    def get(self, request, *args, **kwargs):
-        mailing = get_object_or_404(Mailing, pk=kwargs["mailing_id"])
-        if mailing.can_moderate(request.user):
-            mailing.is_moderated = True
-            mailing.moderator = request.user
-            mailing.save()
-            return redirect("com:mailing_admin")
-
-        raise PermissionDenied
-
-
 class PosterListBaseView(ListView):
     """List communication posters."""