From 6a4ca127a4b1c6b9977635a41201b6b977cd9dad Mon Sep 17 00:00:00 2001
From: Alexandre Aubin <alex.aubin@mailoo.org>
Date: Thu, 17 Oct 2019 19:56:36 +0200
Subject: [PATCH] Moooar buttons (domain info view)

---
 src/js/yunohost/controllers/domains.js | 84 ++++++++++++--------------
 src/views/domain/domain_info.ms        | 13 ++--
 2 files changed, 45 insertions(+), 52 deletions(-)

diff --git a/src/js/yunohost/controllers/domains.js b/src/js/yunohost/controllers/domains.js
index bfd41e2f..17df4f75 100644
--- a/src/js/yunohost/controllers/domains.js
+++ b/src/js/yunohost/controllers/domains.js
@@ -97,14 +97,52 @@
                     }
                 });
 
-
                 var domain = {
                     name: c.params['domain'],
                     main: (c.params['domain'] == dataMain.current_main_domain) ? true : false,
                     url: "https://"+c.params['domain'],
                     enable_cert_management: enable_cert_management_
                 };
-                c.view('domain/domain_info', domain);
+                c.view('domain/domain_info', domain, function() {
+
+                    // Configure "set default" button
+                    $('button[data-action="set_default"]').on("click", function() {
+                        var domain = $(this).data("domain");
+                        c.confirm(
+                            y18n.t('domains'),
+                            y18n.t('confirm_change_maindomain'),
+                            function() {
+                                var params = {
+                                    new_domain: c.params['domain']
+                                };
+                                c.api('PUT', '/domains/main', params, function(data) {
+                                    c.redirect_to('#/domains');
+                                });
+
+                                // WTF ... why is this hack needed v_v
+
+                                // Wait 15s and refresh the page
+                                var refreshDomain = window.setTimeout(function(){
+                                    c.redirect_to('#/domains');
+                                }, 15000);
+                            }
+                        );
+                    });
+
+                    // Configure delete button
+                    $('button[data-action="delete"]').on("click", function() {
+                        var domain = $(this).data("domain");
+                        c.confirm(
+                            y18n.t('domains'),
+                            y18n.t('confirm_delete', [domain]),
+                            function(){
+                                c.api('DELETE', '/domains/'+ domain, {}, function(data) {
+                                    c.redirect_to('#/domains');
+                                });
+                            }
+                        );
+                    });
+                });
             });
         });
     });
@@ -241,46 +279,4 @@
         });
     });
 
-
-    // Remove existing domain
-    app.get('#/domains/:domain/delete', function (c) {
-        c.confirm(
-            y18n.t('domains'),
-            y18n.t('confirm_delete', [c.params['domain']]),
-            function(){
-                c.api('DELETE', '/domains/'+ c.params['domain'], {}, function(data) {
-                    c.redirect_to('#/domains');
-                });
-            }
-        );
-    });
-
-    // Set default domain
-    app.post('#/domains', function (c) {
-        if (c.params['domain'] === '') {
-            c.flash('fail', y18n.t('error_select_domain'));
-            c.redirect_to('#/domains', {slide:false});
-        } else {
-            c.confirm(
-                y18n.t('domains'),
-                y18n.t('confirm_change_maindomain'),
-                function(){
-                    var params = {
-                        new_domain: c.params['domain']
-                    };
-                    c.api('PUT', '/domains/main', params, function(data) {
-                        c.redirect_to('#/domains');
-                    });
-
-                    // WTF ... why is this hack needed v_v
-
-                    // Wait 15s and refresh the page
-                    var refreshDomain = window.setTimeout(function(){
-                        c.redirect_to('#/domains');
-                    }, 15000);
-                }
-            );
-        }
-    });
-
 })();
diff --git a/src/views/domain/domain_info.ms b/src/views/domain/domain_info.ms
index 6eed7627..f22bd3c0 100644
--- a/src/views/domain/domain_info.ms
+++ b/src/views/domain/domain_info.ms
@@ -40,12 +40,9 @@
         <hr>
         <div class="container">
             <p>{{t 'domain_default_desc'}}</p>
-            <form method="POST" action="#/domains">
-                    <input type="hidden" name="domain" value="{{name}}" required class="form-control">
-                    <button type="submit" class="btn btn-primary slide back" value="{{t 'set_default'}}">
-                        {{t 'set_default'}} <span class="fa-fw fa-star"></span>
-                    </button>
-            </form>
+            <button class="btn btn-primary" data-action="set_default" data-domain="{{name}}">
+                {{t 'set_default'}} <span class="fa-fw fa-star"></span>
+            </button>
         </div>
         {{/unless}}
         <hr>
@@ -70,9 +67,9 @@
         <hr>
         <div class="container">
             <p>{{t 'domain_delete_longdesc' name}}</p>
-            <a href="#/domains/{{name}}/delete" role="button" class="btn btn-danger slide back">
+            <button class="btn btn-danger" data-action="delete" data-domain="{{name}}">
                 {{t 'delete'}} <span class="fa-fw fa-trash-o"></span>
-            </a>
+            </button>
         </div>
     </div>
 </div>