modules/freifunk: Change default netmask to 255.255.0.0, change non-working dns, show error when user tries to edit profiles but there is no profile set yet

modules/freifunk/luasrc/controller/freifunk/freifunk.lua

@@ -98,6 +98,7 @@ function index()
 
 	entry({"freifunk", "map"}, template("freifunk-map/frame"), i18n("Map"), 50)
 	entry({"freifunk", "map", "content"}, template("freifunk-map/map"), nil, 51)
+	entry({"admin", "freifunk", "profile_error"}, template("freifunk/profile_error"))
 end
 
 local function fetch_olsrd()

modules/freifunk/luasrc/model/cbi/freifunk/profile.lua

@@ -11,45 +11,48 @@ You may obtain a copy of the License at
 ]]--
 
 local uci = require "luci.model.uci".cursor()
-local community = "profile_" .. uci:get("freifunk", "community", "name")
+local community = uci:get("freifunk", "community", "name")
---local community = "profile_augsburg"
 
+if community == nil then
+	luci.http.redirect(luci.dispatcher.build_url("admin", "freifunk", "profile_error"))
+	return
+else
+	community = "profile_" .. community
+	m = Map(community, translate("Community settings"), translate("These are the settings of your local community."))
+	c = m:section(NamedSection, "profile", "community")
 
-m = Map(community, translate("Community settings"), translate("These are the settings of your local community."))
+	name = c:option(Value, "name", "Name")
+	name.rmempty = false
 
-c = m:section(NamedSection, "profile", "community")
+	homepage = c:option(Value, "homepage", translate("Homepage"))
 
-name = c:option(Value, "name", "Name")
+	cc = c:option(Value, "country", translate("Country code"))
-name.rmempty = false
+	function cc.cfgvalue(self, section)
-
+		return uci:get(community, "wifi_device", "country")
-homepage = c:option(Value, "homepage", translate("Homepage"))
-
-cc = c:option(Value, "country", translate("Country code"))
-function cc.cfgvalue(self, section)
-	return uci:get(community, "wifi_device", "country")
-end
-function cc.write(self, sec, value)
-	if value then
-		uci:set(community, "wifi_device", "country", value)
-		uci:save(community)
 	end
+	function cc.write(self, sec, value)
+		if value then
+			uci:set(community, "wifi_device", "country", value)
+			uci:save(community)
+		end
+	end
+
+	ssid = c:option(Value, "ssid", translate("ESSID"))
+	ssid.rmempty = false
+
+	prefix = c:option(Value, "mesh_network", translate("Mesh prefix"))
+	prefix.rmempty = false
+
+	splash_net = c:option(Value, "splash_network", translate("Network for client DHCP addresses"))
+	splash_net.rmempty = false
+
+	splash_prefix = c:option(Value, "splash_prefix", translate("Client network size"))
+	splash_prefix.rmempty = false
+
+	lat = c:option(Value, "latitude", translate("Latitude"))
+	lat.rmempty = false
+
+	lon = c:option(Value, "longitude", translate("Longitude"))
+	lon.rmempty = false
+	return m
 end
-
-ssid = c:option(Value, "ssid", translate("ESSID"))
-ssid.rmempty = false
-
-prefix = c:option(Value, "mesh_network", translate("Mesh prefix"))
-prefix.rmempty = false
-
-splash_net = c:option(Value, "splash_network", translate("Network for client DHCP addresses"))
-splash_net.rmempty = false
-
-splash_prefix = c:option(Value, "splash_prefix", translate("Client network size"))
-splash_prefix.rmempty = false
-
-lat = c:option(Value, "latitude", translate("Latitude"))
-lat.rmempty = false
-
-lon = c:option(Value, "longitude", translate("Longitude"))
-lon.rmempty = false
-return m

modules/freifunk/luasrc/model/cbi/freifunk/profile_expert.lua

@@ -12,26 +12,30 @@ You may obtain a copy of the License at
 
 local fs = require "nixio.fs"
 local uci = require "luci.model.uci".cursor()
-local community = "/etc/config/profile_" .. uci:get("freifunk", "community", "name")
+local community = uci:get("freifunk", "community", "name")
 
-f = SimpleForm("community", translate("Community profile"), 
+if community == nil then
-	translate("You can manually edit the selected community profile here."))
+        luci.http.redirect(luci.dispatcher.build_url("admin", "freifunk", "profile_error"))
+        return
+else
+        community = "/etc/config/profile_" .. community
+	f = SimpleForm("community", translate("Community profile"), translate("You can manually edit the selected community profile here."))
 
-t = f:field(TextValue, "cop")
+	t = f:field(TextValue, "cop")
-t.rmempty = true
+	t.rmempty = true
-t.rows = 30
+	t.rows = 30
-function t.cfgvalue()
+	function t.cfgvalue()
-	return fs.readfile(community) or ""
+		return fs.readfile(community) or ""
-end
-
-function f.handle(self, state, data)
-	if state == FORM_VALID then
-		if data.cop then
-			fs.writefile(cop, data.rcs:gsub("\r\n", "\n"))
-		end
 	end
-	return true
+
+	function f.handle(self, state, data)
+		if state == FORM_VALID then
+			if data.cop then
+				fs.writefile(cop, data.rcs:gsub("\r\n", "\n"))
+			end
+		end
+		return true
+	end
+	return f
 end
 
-return f
-

modules/freifunk/luasrc/view/freifunk/profile_error.htm

@@ -0,0 +1,11 @@
+<%+header%>
+
+<%
+local profileurl = luci.dispatcher.build_url(luci.dispatcher.context.path[1], "freifunk", "basics")
+%>
+
+<h2><%:Error%></h2>
+<%:You need to select a profile before you can edit it. To select a profile go to%> <a href='<%=profileurl%>'><%:Basic settings%></a>.
+<p/>
+
+<%+footer%>

modules/freifunk/root/etc/config/freifunk

@@ -70,8 +70,8 @@ config 'defaults' 'wifi_iface'
 	option 'sw_merge' '1'
 
 config 'defaults' 'interface'
-	option 'netmask' '255.0.0.0'
+	option 'netmask' '255.255.0.0'
-	option 'dns' '88.198.178.18 141.54.1.1 212.204.49.83 208.67.220.220 208.67.222.222'
+	option 'dns' '8.8.8.8 212.204.49.83 141.1.1.1'
 
 config 'defaults' 'alias'
 	option 'netmask' '255.255.255.0'

modules/freifunk/root/etc/config/profile_aachen

@@ -7,6 +7,3 @@ config 'community' 'profile'
 	option 'mesh_network' '10.90.0.0/16'
 	option 'splash_network' '10.104.0.0/16'
 	option 'splash_prefix' '28'
-
-config 'defaults' 'interface'
-	option 'netmask' '255.255.0.0'

modules/freifunk/root/etc/config/profile_bergischesland

@@ -13,7 +13,3 @@ config 'defaults' 'wifi_device'
 
 config 'defaults' 'wifi_iface'
 	option 'bssid' '02:40:00:42:42:42'
-
-config 'defaults' 'interface'
-	option 'netmask' '255.255.0.0'
-

modules/freifunk/root/etc/config/profile_berlin

@@ -10,3 +10,7 @@ config 'community' 'profile'
 
 config 'defaults' 'wifi_device'
 	option 'channel' '10'
+
+config 'defaults' 'interface'
+        option 'netmask' '255.0.0.0'
+

modules/freifunk/root/etc/config/profile_duesseldorf

@@ -8,9 +8,6 @@ config 'community' 'profile'
 	option 'latitude' '51.22347'
 	option 'longitude' '6.78449'
 
-config 'defaults' 'interface'
-	option 'netmask' '255.255.0.0'
-
 config 'defaults' 'wifi_device'
 	option 'channel' '3'
 

modules/freifunk/root/etc/config/profile_hannover

@@ -10,7 +10,3 @@ config 'community' 'profile'
 
 config 'defaults' 'wifi_iface'
 	option 'bssid' 'CA:FF:EE:CA:FF:EE'
-
-config 'defaults' 'interface'
-	option 'netmask' '255.255.0.0'
-

modules/freifunk/root/etc/config/profile_kiberpipa

@@ -13,4 +13,3 @@ config 'defaults' 'wifi_device'
 
 config 'defaults' 'interface'
 	option 'dns' '10.14.0.1 208.67.222.220 208.67.220.222'
-	option 'netmask' '255.255.0.0'

modules/freifunk/root/etc/config/profile_mainz

@@ -13,7 +13,3 @@ config 'defaults' 'wifi_iface'
 
 config 'defaults' 'wifi_device'
 	option 'channel' '1'
-
-config 'defaults' 'interface'
-	option 'netmask' '255.255.0.0'
-

modules/freifunk/root/etc/config/profile_marburg

@@ -10,4 +10,3 @@ config 'community' 'profile'
 
 config 'defaults' 'interface'
 	option 'dns' '8.8.8.8 212.204.49.83'
-        option 'netmask' '255.255.0.0'

modules/freifunk/root/etc/config/profile_neuss

@@ -11,8 +11,5 @@ config 'community' 'profile'
 config 'defaults' 'wifi_device'
 	option 'channel' '11'
 
-config 'defaults' 'interface'
-	option 'netmask' '255.255.0.0'
-
 config 'defaults' 'wifi_iface'
 	option 'bssid' 'DE:AD:BE:EF:CA:FE'

modules/freifunk/root/etc/config/profile_openwireless_bern

@@ -9,7 +9,6 @@ config 'community' 'profile'
 	option 'longitude' '8.769239'
 
 config 'defaults' 'interface'
-	option 'netmask' '255.255.0.0'
 	option 'dns' '208.67.222.222 208.67.220.220'
 
 config 'defaults' 'wifi_device'

modules/freifunk/root/etc/config/profile_potsdam

@@ -8,8 +8,5 @@ config 'community' 'profile'
 	option 'latitude' '52.39349'
 	option 'longitude' '13.06489'
 
-config 'defaults' 'interface'
-	option 'netmask' '255.255.0.0'
-
 config 'defaults' 'wifi_device'
 	option 'channel' '13'

modules/freifunk/root/etc/config/profile_rosbach

@@ -13,6 +13,3 @@ config 'defaults' 'wifi_device'
 
 config 'defaults' 'wifi_iface'
 	option 'bssid' 'D2:CA:FF:EE:BA:BE'
-
-config 'defaults' 'interface'
-        option 'netmask' '255.255.0.0'

modules/freifunk/root/etc/config/profile_seefeld

@@ -8,9 +8,6 @@ config 'community' 'profile'
 	option 'latitude' '48.03485'
 	option 'longitude' '11.21279'
 
-config 'defaults' 'interface'
-	option 'netmask' '255.255.255.0'
-
 config 'defaults' 'wifi_device'
 	option 'channel' '1'
 	option 'bssid' '02:CA:FF:EE:BA:BB'

modules/freifunk/root/etc/config/profile_wlanljubljana

@@ -13,4 +13,3 @@ config 'defaults' 'wifi_device'
 
 config 'defaults' 'interface'
 	option 'dns' '10.254.0.1 10.254.0.2'
-	option 'netmask' '255.255.0.0'