From 730719b61ff894f346b81fdb6bd1b6a6901a5c21 Mon Sep 17 00:00:00 2001
From: Leo Hemsted <leo.hemsted@digital.cabinet-office.gov.uk>
Date: Wed, 26 Oct 2016 16:56:51 +0100
Subject: [PATCH] platform admins can toggle services from seeing letters

---
 app/main/views/service_settings.py        | 13 ++++++++++++-
 app/notify_client/service_api_client.py   |  1 +
 app/templates/views/service-settings.html |  4 ++--
 3 files changed, 15 insertions(+), 3 deletions(-)

diff --git a/app/main/views/service_settings.py b/app/main/views/service_settings.py
index 71a038bed..b65f449d0 100644
--- a/app/main/views/service_settings.py
+++ b/app/main/views/service_settings.py
@@ -168,7 +168,18 @@ def service_switch_live(service_id):
 def service_switch_research_mode(service_id):
     service_api_client.update_service_with_properties(
         service_id,
-        {"research_mode": False if current_service['research_mode'] else True}
+        {"research_mode": not current_service['research_mode']}
+    )
+    return redirect(url_for('.service_settings', service_id=service_id))
+
+
+@main.route("/services/<service_id>/service-settings/can-send-letters")
+@login_required
+@user_has_permissions(admin_override=True)
+def service_switch_can_send_letters(service_id):
+    service_api_client.update_service_with_properties(
+        service_id,
+        {"can_send_letters": not current_service['can_send_letters']}
     )
     return redirect(url_for('.service_settings', service_id=service_id))
 
diff --git a/app/notify_client/service_api_client.py b/app/notify_client/service_api_client.py
index dfa64de51..17ff797be 100644
--- a/app/notify_client/service_api_client.py
+++ b/app/notify_client/service_api_client.py
@@ -88,6 +88,7 @@ class ServiceAPIClient(NotificationsAPIClient):
             'email_from',
             'reply_to_email_address',
             'research_mode',
+            'can_send_letters',
             'sms_sender',
             'created_by',
             'branding',
diff --git a/app/templates/views/service-settings.html b/app/templates/views/service-settings.html
index 41cf28563..3fda72be4 100644
--- a/app/templates/views/service-settings.html
+++ b/app/templates/views/service-settings.html
@@ -88,12 +88,12 @@
             {{ 'Make service live' if current_service.restricted else 'Revert service to trial mode' }}
           </a>
         </li>
-        <li>
+        <li class="bottom-gutter">
           <a href="{{ url_for('.service_switch_research_mode', service_id=current_service.id) }}" class="button">
             {{ 'Take service out of research mode' if current_service.research_mode else 'Put into research mode' }}
           </a>
         </li>
-        <li>
+        <li class="bottom-gutter">
           <a href="{{ url_for('.service_switch_can_send_letters', service_id=current_service.id) }}" class="button">
             {{ 'Stop sending letters' if current_service.can_send_letters else 'Allow to send letters' }}
           </a>