From 3db5aa86214a4f9ea03067b12a9cedb6bcf6fe9a Mon Sep 17 00:00:00 2001
From: Santiago Lamora <slamora@users.noreply.github.com>
Date: Fri, 13 Dec 2019 15:08:01 +0100
Subject: [PATCH] Created domain-detail view.

---
 musician/api.py                               | 16 +++++++++--
 .../templates/musician/domain_detail.html     | 28 +++++++++++++++++++
 musician/urls.py                              |  1 +
 musician/views.py                             | 20 +++++++++++++
 4 files changed, 63 insertions(+), 2 deletions(-)
 create mode 100644 musician/templates/musician/domain_detail.html

diff --git a/musician/api.py b/musician/api.py
index 0536b5c..dcfe7e2 100644
--- a/musician/api.py
+++ b/musician/api.py
@@ -18,6 +18,7 @@ API_PATHS = {
     # services
     'database-list': 'databases/',
     'domain-list': 'domains/',
+    'domain-detail': 'domains/{pk}/',
     'address-list': 'addresses/',
     'mailbox-list': 'mailboxes/',
     'mailinglist-list': 'lists/',
@@ -55,9 +56,13 @@ class Orchestra(object):
 
         return response.json().get("token", None)
 
-    def request(self, verb, resource, querystring=None, raise_exception=True):
+    def request(self, verb, resource=None, querystring=None, url=None, raise_exception=True):
         assert verb in ["HEAD", "GET", "POST", "PATCH", "PUT", "DELETE"]
-        url = self.build_absolute_uri(resource)
+        if resource is not None:
+            url = self.build_absolute_uri(resource)
+        elif url is None:
+            raise AttributeError("Provide `resource` or `url` params")
+
         if querystring is not None:
             url = "{}?{}".format(url, querystring)
 
@@ -86,6 +91,13 @@ class Orchestra(object):
             raise PermissionError("Cannot retrieve profile of an anonymous user.")
         return UserAccount.new_from_json(output[0])
 
+    def retrieve_domain(self, pk):
+        path = API_PATHS.get('domain-detail').format_map({'pk': pk})
+
+        url = urllib.parse.urljoin(self.base_url, path)
+        status, domain_json = self.request("GET", url=url)
+        return Domain.new_from_json(domain_json)
+
     def retrieve_domain_list(self):
         output = self.retrieve_service_list(Domain.api_name)
         domains = []
diff --git a/musician/templates/musician/domain_detail.html b/musician/templates/musician/domain_detail.html
new file mode 100644
index 0000000..6cb459e
--- /dev/null
+++ b/musician/templates/musician/domain_detail.html
@@ -0,0 +1,28 @@
+{% extends "musician/base.html" %}
+{% load i18n %}
+
+{% block content %}
+<h1 class="service-name">{% trans "DNS settings for" %} <span class="font-weight-light">{{ object.name }}</span></h1>
+<p class="service-description">Litle description of what to be expected in this section to aid the user. Even a link to more help if there is one available.</p>
+
+<table class="table service-list">
+    <colgroup>
+      <col span="1" style="width: 12%;">
+      <col span="1" style="width: 88%;">
+  </colgroup>
+  <thead class="thead-dark">
+    <tr>
+      <th scope="col">{% trans "Type" %}</th>
+      <th scope="col">{% trans "Value" %}</th>
+    </tr>
+  </thead>
+  <tbody>
+    {% for record in object.records %}
+    <tr>
+      <td>{{ record.type }}</td>
+      <td>{{ record.value }}</td>
+    </tr>
+    {% endfor %}
+  </tbody>
+</table>
+{% endblock %}
diff --git a/musician/urls.py b/musician/urls.py
index 340d409..6bae570 100644
--- a/musician/urls.py
+++ b/musician/urls.py
@@ -15,6 +15,7 @@ urlpatterns = [
     path('auth/login/', views.LoginView.as_view(), name='login'),
     path('auth/logout/', views.LogoutView.as_view(), name='logout'),
     path('dashboard/', views.DashboardView.as_view(), name='dashboard'),
+    path('domains/<int:pk>/', views.DomainDetailView.as_view(), name='domain-detail'),
     path('billing/', views.BillingView.as_view(), name='billing'),
     path('profile/', views.ProfileView.as_view(), name='profile'),
     path('mails/', views.MailView.as_view(), name='mails'),
diff --git a/musician/views.py b/musician/views.py
index b6b4298..7d93509 100644
--- a/musician/views.py
+++ b/musician/views.py
@@ -186,6 +186,26 @@ class SaasView(ServiceListView):
     template_name = "musician/saas.html"
 
 
+class DomainDetailView(CustomContextMixin, UserTokenRequiredMixin, DetailView):
+    template_name = "musician/domain_detail.html"
+
+    def get_queryset(self):
+        return []  # self.orchestra.retrieve_domain_list()
+
+    def get_object(self, queryset=None):
+        if queryset is None:
+            queryset = self.get_queryset()
+
+        pk = self.kwargs.get(self.pk_url_kwarg)
+        # TODO try to retrieve object capturing exception
+        try:
+            domain = self.orchestra.retrieve_domain(pk)
+        except:
+            raise
+
+        return domain
+
+
 class LoginView(FormView):
     template_name = 'auth/login.html'
     form_class = LoginForm