From 78809d44bcb02408732fe9b543d1130e3c2b6e13 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ben=20Gr=C3=A4f?= <ben@gra3f.de>
Date: Wed, 24 Jul 2019 13:58:28 -0700
Subject: [PATCH] Delete a template from Dashboard (#257)

* Integrated embedded config parsing #245

* cleanup

* Cleanup in remotelog

* Fixed MS Visual Studio 2019 compatibility

* Embedded config parsing only for miner not server

* wip

* Finished delete template
---
 index.html         | 53 +++++++++++++++++++++++++++++++++++++++++++---
 src/cc/Service.cpp | 16 ++++++++++++++
 src/cc/Service.h   |  1 +
 3 files changed, 67 insertions(+), 3 deletions(-)

diff --git a/index.html b/index.html
index 0fde83b0..58e12adf 100755
--- a/index.html
+++ b/index.html
@@ -292,12 +292,14 @@
 
                                 if (arrayLength > 0 ) {
                                     $('#templateEditorSave').prop('disabled', false);
+                                    $('#templateEditorDeleteDialog').prop('disabled', false);
 
                                     $('#templateSelector').html(htmlContent);
                                     $('#templateSelector').selectpicker('refresh');
                                     $('#templateSelector').trigger('change');
                                 } else {
                                     $('#templateEditorSave').prop('disabled', true);
+                                    $('#templateEditorDeleteDialog').prop('disabled', true);
                                 }
 
                                 $('#templateEditor').modal('show');
@@ -511,6 +513,10 @@
             $('#templateDialogSaveAs').modal('show');
         });
 
+        $('#templateEditorDeleteDialog').click(function(event) {
+            $('#templateDialogDelete').modal('show');
+        });
+
         $('#templateEditorSaveAs').click(function(event) {
             var templateId = $('#templateName').val();
             var template = $('#template').val();
@@ -520,6 +526,12 @@
             $('#templateEditor').modal('hide');
         });
 
+        $('#templateEditorDelete').click(function(event) {
+            var templateId = $('#templateSelector').val();
+
+            deleteTemplateConfig(templateId, template);
+        });
+
         $('#clientStatusList tbody').on( 'click', 'button#LOG', function () {
             var data = table.row( $(this).parents('tr') ).data();
             var clientId = data['client_status']['client_id'];
@@ -856,6 +868,20 @@
             }
         });
     }
+
+    function deleteTemplateConfig(templateId) {
+        $.ajax({
+            url: "/admin/deleteClientConfig?clientId=template_" + templateId,
+            type: 'POST',
+            dataType: "text",
+            success: function(data){
+                setSuccess('<strong>Successfully deleted template: ' + templateId + '</strong>');
+            },
+            error: function (data) {
+                setError('<strong>Failed to delete template: ' + templateId + '</strong> \nError: ' +  JSON.stringify(data,undefined, 2));
+            }
+        });
+    }
 </script>
 
 </head>
@@ -982,7 +1008,8 @@
                 <div class="modal-footer">
                     <button id="templateEditorSave" type="button" class="btn btn-success" data-dismiss="modal">Save</button>
                     <button id="templateEditorSaveAsDialog" type="button" class="btn btn-primary">Save As...</button>
-                    <button type="button" class="btn btn-danger" data-dismiss="modal">Cancel</button>
+                    <button id="templateEditorDeleteDialog" type="button" class="btn btn-danger" data-dismiss="modal">Delete</button>
+                    <button type="button" class="btn btn-default" data-dismiss="modal">Cancel</button>
                 </div>
             </div>
         </div>
@@ -1003,7 +1030,27 @@
                 </div>
                 <div class="modal-footer">
                     <button id="templateEditorSaveAs" type="button" class="btn btn-success" data-dismiss="modal">Save</button>
-                    <button type="button" class="btn btn-danger" data-dismiss="modal">Cancel</button>
+                    <button type="button" class="btn btn-default" data-dismiss="modal">Cancel</button>
+                </div>
+            </div>
+        </div>
+    </div>
+
+    <div class="modal fade" id="templateDialogDelete" role="dialog">
+        <div class="modal-dialog modal-lg">
+            <div class="modal-content">
+                <div class="modal-header">
+                    <button type="button" class="close" data-dismiss="modal">&times;</button>
+                    <h4 class="modal-title">Delete</h4>
+                </div>
+                <div class="modal-body">
+                    <div class="form-group">
+                        <label for="templateName">Do you really want to delete this template?</label>
+                    </div>
+                </div>
+                <div class="modal-footer">
+                    <button id="templateEditorDelete" type="button" class="btn btn-danger" data-dismiss="modal">Delete</button>
+                    <button type="button" class="btn btn-default" data-dismiss="modal">Cancel</button>
                 </div>
             </div>
         </div>
@@ -1034,7 +1081,7 @@
                 </div>
                 <div class="modal-footer">
                     <button id="assignTemplate" type="button" class="btn btn-success" data-dismiss="modal">Assign</button>
-                    <button type="button" class="btn btn-danger" data-dismiss="modal">Cancel</button>
+                    <button type="button" class="btn btn-default" data-dismiss="modal">Cancel</button>
                 </div>
             </div>
         </div>
diff --git a/src/cc/Service.cpp b/src/cc/Service.cpp
index 3bf47457..6520a3b0 100644
--- a/src/cc/Service.cpp
+++ b/src/cc/Service.cpp
@@ -155,6 +155,8 @@ unsigned Service::handlePOST(const Options* options, const std::string& url, con
             resultCode = setClientConfig(options, clientId, data, resp);
         } else if (url.rfind("/admin/setClientCommand", 0) == 0) {
             resultCode = setClientCommand(clientId, data, resp);
+        } else if (url.rfind("/admin/deleteClientConfig", 0) == 0) {
+            resultCode = deleteClientConfig(options, clientId, resp);
         } else {
             LOG_WARN("[%s] 400 BAD REQUEST - Request does not contain clientId (%s)", clientIp.c_str(), url.c_str());
         }
@@ -482,6 +484,20 @@ void Service::setClientLog(size_t maxRows, const std::string& clientId, const st
     }
 }
 
+unsigned Service::deleteClientConfig(const Options* options, const std::string& clientId, std::string& resp)
+{
+    if (!clientId.empty()) {
+        std::string clientConfigFileName = getClientConfigFileName(options, clientId);
+        if (!clientConfigFileName.empty() && remove(clientConfigFileName.c_str()) == 0) {
+            return MHD_HTTP_OK;
+        } else {
+            return MHD_HTTP_NOT_FOUND;
+        }
+    } else {
+        return MHD_HTTP_BAD_REQUEST;
+    }
+}
+
 unsigned Service::resetClientStatusList(const std::string& data, std::string& resp)
 {
     m_clientStatus.clear();
diff --git a/src/cc/Service.h b/src/cc/Service.h
index 7c755ee4..88f5fecf 100644
--- a/src/cc/Service.h
+++ b/src/cc/Service.h
@@ -60,6 +60,7 @@ private:
     static unsigned setClientStatus(const Options* options, const std::string& clientIp, const std::string& clientId, const std::string& data, std::string& resp);
     static unsigned setClientCommand(const std::string& clientId, const std::string& data, std::string& resp);
     static unsigned setClientConfig(const Options* options, const std::string &clientId, const std::string &data, std::string &resp);
+    static unsigned deleteClientConfig(const Options* options, const std::string& clientId, std::string& resp);
     static unsigned resetClientStatusList(const std::string& data, std::string& resp);
 
     static void setClientLog(size_t maxRows, const std::string& clientId, const std::string& log);