luci-app-ddns: Fix broken Custom provider select #4150

Fix broken Custom provider selection in luci-app-ddns.
Fixes: #4150

Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>

diff --git a/applications/luci-app-ddns/htdocs/luci-static/resources/view/ddns/overview.js b/applications/luci-app-ddns/htdocs/luci-static/resources/view/ddns/overview.js
index a15563e5b8263a0e55695f6e418a9ea87a901fd5..2bda3f3bc5e3c5ec68ca8b0084937eb3bd4d78b4 100644 (file)
--- a/applications/luci-app-ddns/htdocs/luci-static/resources/view/ddns/overview.js
+++ b/applications/luci-app-ddns/htdocs/luci-static/resources/view/ddns/overview.js
@@ -511,7 +511,14 @@ return view.extend({
                        o.value('-',"-- " + _("custom") + " --");
 
                o.cfgvalue = function(section_id) {
-                       return uci.get('ddns', section_id, 'service_name');
+                       var service = uci.get('ddns', section_id, 'service_name'),
+                               update_script = uci.get('ddns', section_id, 'update_script'),
+                               update_url = uci.get('ddns', section_id, 'update_url');
+
+                       if (!service && (update_script || update_url))
+                               return "-";
+
+                       return service;
                }
 
                o.write = function(section_id, formvalue) {
@@ -529,19 +536,41 @@ return view.extend({
                _("Update URL to be used for updating your DDNS Provider." + "<br />" +
                "Follow instructions you will find on their WEB page."));
                o.modalonly = true;
-               o.rmempty       = false;
+               o.rmempty       = true;
+               o.optional = true;
                o.depends("ipv6_service_name","-");
                o.depends("ipv4_service_name","-");
 
+               o.validate = function(section_id, value) {
+                       var other = this.section.children.filter(function(o) { return o.option == 'update_script' })[0].formvalue(section_id);
+
+                       if ((value == "" && other == "") || (value != "" && other != "")) {
+                               return _("Insert a Update Script OR a Update URL");
+                       }
+
+                       return true;
+               }
+
                // update_script
 
                o = s.taboption('basic', form.Value, 'update_script', _("Custom update-script"),
                _("Custom update script to be used for updating your DDNS Provider."));
                o.modalonly = true;
-               o.rmempty       = false;
+               o.rmempty       = true;
+               o.optional = true;
                o.depends("ipv6_service_name","-");
                o.depends("ipv4_service_name","-");
 
+               o.validate = function(section_id, value) {
+                       var other = this.section.children.filter(function(o) { return o.option == 'update_url' })[0].formvalue(section_id);
+
+                       if ((value == "" && other == "") || (value != "" && other != "")) {
+                               return _("Insert a Update Script OR a Update URL");
+                       }
+
+                       return true;
+               }
+
                // domain
 
                o = s.taboption('basic', form.Value, 'domain', _("Domain"),