From: Jo-Philipp Wich <redacted>
Date: Sat, 20 Mar 2021 23:36:24 +0000 (+0100)
Subject: luci-mod-network: restore DNS option semantics for proto static
X-Git-Url: http://git.99rst.org/?a=commitdiff_plain;h=2bfd4908a9cfeac74c7abc31d8cef0bf1e58af52;p=openwrt-luci.git

luci-mod-network: restore DNS option semantics for proto static

The peerdns settings makes little practical sense for proto:static
interfaces, so revert to allow setting the DNS server list directly.

Fixes: faad7464a8 ("luci-mod-network: add support for network.device sections")
Signed-off-by: Jo-Philipp Wich <redacted>
---

diff --git a/modules/luci-mod-network/htdocs/luci-static/resources/view/network/interfaces.js b/modules/luci-mod-network/htdocs/luci-static/resources/view/network/interfaces.js
index 19021c4ef1..02a2a89a57 100644
--- a/modules/luci-mod-network/htdocs/luci-static/resources/view/network/interfaces.js
+++ b/modules/luci-mod-network/htdocs/luci-static/resources/view/network/interfaces.js
@@ -596,9 +596,11 @@ return view.extend({
 				o.default = o.enabled;
 
 				o = nettools.replaceOption(s, 'advanced', form.Flag, 'peerdns', _('Use DNS servers advertised by peer'), _('If unchecked, the advertised DNS server addresses are ignored'));
+				o.depends({ 'proto': 'static', '!reverse': true });
 				o.default = o.enabled;
 
 				o = nettools.replaceOption(s, 'advanced', form.DynamicList, 'dns', _('Use custom DNS servers'));
+				o.depends('proto', 'static');
 				o.depends('peerdns', '0');
 				o.datatype = 'ipaddr';