From 85ff9db2bffd3d2bf119300d9297f1b524608e1b Mon Sep 17 00:00:00 2001 From: Julien Cassette Date: Sun, 2 Oct 2022 20:14:19 +0200 Subject: [PATCH] luci-app-ddns: fix creation of custom service (fix #4926) The `service_name` option must be empty for custom services, otherwise an error is raised in `dynamic_dns_updater.sh`. Signed-off-by: Julien Cassette --- .../htdocs/luci-static/resources/view/ddns/overview.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) 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 8a8c27c718..d395f411a3 100644 --- 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 @@ -477,7 +477,9 @@ return view.extend({ return m.save(function() { uci.add('ddns', 'service', section_id); - uci.set('ddns', section_id, 'service_name', service_value); + if (service_value != '-') { + uci.set('ddns', section_id, 'service_name', service_value); + } uci.set('ddns', section_id, 'use_ipv6', ipv6_value); }).then(L.bind(m.children[1].renderMoreOptionsModal, m.children[1], section_id)); }; -- 2.30.2