From 0d327127f5db33bf1f02382173406649f86b3812 Mon Sep 17 00:00:00 2001
From: Santiago Lamora <santiago@ribaguifi.com>
Date: Fri, 2 Jul 2021 13:08:06 +0200
Subject: [PATCH] Rename class `MailService` to `Address`

---
 musician/api.py    | 19 +++++++++----------
 musician/models.py |  3 +--
 musician/views.py  |  8 ++++----
 3 files changed, 14 insertions(+), 16 deletions(-)

diff --git a/musician/api.py b/musician/api.py
index 3d9755c..bffaa3e 100644
--- a/musician/api.py
+++ b/musician/api.py
@@ -1,14 +1,13 @@
-import requests
 import urllib.parse
-
 from itertools import groupby
+
+import requests
 from django.conf import settings
 from django.http import Http404
 from django.urls.exceptions import NoReverseMatch
 from django.utils.translation import gettext_lazy as _
 
-from .models import Domain, DatabaseService, MailService, SaasService, UserAccount, WebSite
-
+from .models import Address, DatabaseService, Domain, SaasService, UserAccount, WebSite
 
 DOMAINS_PATH = 'domains/'
 TOKEN_PATH = '/api-token-auth/'
@@ -114,7 +113,7 @@ class Orchestra(object):
         return bill_pdf
 
     def create_mail_address(self, data):
-        resource = '{}-list'.format(MailService.api_name)
+        resource = '{}-list'.format(Address.api_name)
         return self.request("POST", resource=resource, data=data)
 
     def retrieve_mail_address(self, pk):
@@ -124,7 +123,7 @@ class Orchestra(object):
         if status == 404:
             raise Http404(_("No object found matching the query"))
 
-        return MailService.new_from_json(data)
+        return Address.new_from_json(data)
 
     def update_mail_address(self, pk, data):
         path = API_PATHS.get('address-detail').format_map({'pk': pk})
@@ -143,7 +142,7 @@ class Orchestra(object):
 
         # retrieve mails applying filters (if any)
         raw_data = self.retrieve_service_list(
-            MailService.api_name,
+            Address.api_name,
             querystring=querystring,
         )
 
@@ -157,7 +156,7 @@ class Orchestra(object):
                 data = thing
 
             data['names'] = aliases
-            addresses.append(MailService.new_from_json(data))
+            addresses.append(Address.new_from_json(data))
 
         # PATCH to include Pangea addresses not shown by orchestra
         # described on issue #4
@@ -174,7 +173,7 @@ class Orchestra(object):
         #             },
         #             'mailboxes': [mailbox],
         #         }
-        #         pangea_address = MailService.new_from_json(address_data)
+        #         pangea_address = Address.new_from_json(address_data)
         #         addresses.append(pangea_address)
 
         return addresses
@@ -204,7 +203,7 @@ class Orchestra(object):
 
             # retrieve services associated to a domain
             domain_json['mails'] = self.retrieve_service_list(
-                MailService.api_name, querystring)
+                Address.api_name, querystring)
 
             # retrieve websites (as they cannot be filtered by domain on the API we should do it here)
             domain_json['websites'] = self.filter_websites_by_domain(websites, domain_json['id'])
diff --git a/musician/models.py b/musician/models.py
index 7ac681f..b0094b3 100644
--- a/musician/models.py
+++ b/musician/models.py
@@ -225,8 +225,7 @@ class DomainRecord(OrchestraModel):
         return '<%s: %s>' % (self.type, self.value)
 
 
-# TODO(@slamora) rename to Address
-class MailService(OrchestraModel):
+class Address(OrchestraModel):
     api_name = 'address'
     verbose_name = _('Mail addresses')
     description = _('Description details for mail addresses page.')
diff --git a/musician/views.py b/musician/views.py
index 196b688..bd147a4 100644
--- a/musician/views.py
+++ b/musician/views.py
@@ -19,7 +19,7 @@ from .auth import logout as auth_logout
 from .forms import LoginForm, MailForm
 from .mixins import (CustomContextMixin, ExtendedPaginationMixin,
                      UserTokenRequiredMixin)
-from .models import (Bill, DatabaseService, MailinglistService, MailService,
+from .models import (Address, Bill, DatabaseService, MailinglistService,
                      PaymentSource, SaasService, UserAccount)
 from .settings import ALLOWED_RESOURCES
 from .utils import get_bootstraped_percent
@@ -169,7 +169,7 @@ class BillDownloadView(CustomContextMixin, UserTokenRequiredMixin, View):
 
 
 class MailView(ServiceListView):
-    service_class = MailService
+    service_class = Address
     template_name = "musician/mail.html"
     extra_context = {
         # Translators: This message appears on the page title
@@ -203,7 +203,7 @@ class MailView(ServiceListView):
 
 
 class MailCreateView(CustomContextMixin, UserTokenRequiredMixin, FormView):
-    service_class = MailService
+    service_class = Address
     template_name = "musician/mail_form.html"
     form_class = MailForm
     success_url = reverse_lazy("musician:mails")
@@ -228,7 +228,7 @@ class MailCreateView(CustomContextMixin, UserTokenRequiredMixin, FormView):
 
 
 class MailUpdateView(CustomContextMixin, UserTokenRequiredMixin, FormView):
-    service_class = MailService
+    service_class = Address
     template_name = "musician/mail_form.html"
     form_class = MailForm
     success_url = reverse_lazy("musician:mails")