Skip to content

Commit

Permalink
Made global DHCLIENT_SET_HOSTNAME to work again
Browse files Browse the repository at this point in the history
also fixed naming inconsistency in naming DHCP related widgets caused
by introducing more detailed setup for setting DHCP hostname
  • Loading branch information
mchf committed Nov 22, 2016
1 parent 406ea4b commit 5f8ffab
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions src/include/network/services/dns.rb
Expand Up @@ -310,7 +310,7 @@ def InitSettings
settings = {
"HOSTNAME" => DNS.hostname,
"DOMAIN" => DNS.domain,
"DHCP_HOSTNAME" => DNS.dhcp_hostname,
"DHCP_DEFAULT" => DNS.dhcp_hostname,
"WRITE_HOSTNAME" => DNS.write_hostname,
"PLAIN_POLICY" => DNS.resolv_conf_policy
}
Expand Down Expand Up @@ -351,7 +351,7 @@ def StoreSettings(settings)
DNS.domain = Ops.get_string(settings, "DOMAIN", "")
DNS.nameservers = NonEmpty(nameservers)
DNS.searchlist = NonEmpty(searchlist)
DNS.dhcp_hostname = Ops.get_boolean(settings, "DHCP_HOSTNAME", false)
DNS.dhcp_hostname = settings["DHCP_DEFAULT"]
DNS.write_hostname = Ops.get_boolean(settings, "WRITE_HOSTNAME", true)
DNS.resolv_conf_policy = settings["PLAIN_POLICY"]

Expand Down Expand Up @@ -452,7 +452,7 @@ def InitDhcpHostname(_key)
UI.ChangeWidget(Id("DHCP_HOSTNAME"), :Enabled, has_dhcp?)

# TODO: extend this
dhcp_hostname = @hn_settings.fetch("DHCP_HOSTNAME", false)
dhcp_hostname = @hn_settings.fetch("DHCP_DEFAULT", false)
UI.ChangeWidget(Id("DHCP_HOSTNAME"), :Value, dhcp_hostname)
end

Expand Down Expand Up @@ -486,12 +486,12 @@ def InitDhcpIfaces(_key)
def InitDhcpDefault(_key)
UI.ChangeWidget(Id("DHCP_DEFAULT"), :Enabled, use_dhcp_hostname?)

items = ["yes", "no"].map do |item|
items = [true, false].map do |item|
# TODO: fix inconsistency in naming
syscfg = @hn_settings.fetch("DHCP_HOSTNAME", false) ? "yes" : "no"
syscfg = @hn_settings.fetch("DHCP_DEFAULT", false)
Item(
Id(item),
_(item),
item ? _("yes") : _("no"),
syscfg == item
)
end
Expand Down

0 comments on commit 5f8ffab

Please sign in to comment.