modules/freifunk: Fix errors on index and contact pages when running the Development...

diff --git a/modules/freifunk/luasrc/view/freifunk/contact.htm b/modules/freifunk/luasrc/view/freifunk/contact.htm
index 72d868b..23a5e7b 100644 (file)
--- a/modules/freifunk/luasrc/view/freifunk/contact.htm
+++ b/modules/freifunk/luasrc/view/freifunk/contact.htm
@@ -16,9 +16,20 @@ You may obtain a copy of the License at
 <% 
 local uci = require "luci.model.uci".cursor()
 local contact = uci:get_all("freifunk", "contact")
-local location = uci:get_first("system", "system", "location") or contact.location
+local nickname, name, mail, phone, location, note
 local lon = uci:get_first("system", "system", "longitude")
 local lat = uci:get_first("system", "system", "latitude")
+
+if not contact then
+       nickname, name, mail, phone, location, note = ""
+else
+       nickname = contact.nickname or ""
+       name = contact.name or ""
+       mail = contact.mail or ""
+       phone = contact.phone or ""
+       location = uci:get_first("system", "system", "location") or contact.location
+       note = contact.note or ""
+end
 %>
 
 <h2><a id="content" name="content"><%:Contact%></a></h2>
@@ -26,10 +37,10 @@ local lat = uci:get_first("system", "system", "latitude")
 <fieldset xmlns="http://www.w3.org/1999/xhtml" class="cbi-section">
 <legend><%:Operator%></legend>
        <table cellspacing="10" width="100%" style="text-align:left">
-               <tr><th width="33%"><%:Nickname%>:</th><td><%=contact.nickname%></td></tr>
-               <tr><th width="33%"><%:Realname%>:</th><td><%=contact.name%></td></tr>
-               <tr><th width="33%"><%:E-Mail%>:</th><td><a href="mailto:<%=contact.mail%>"><%=contact.mail%></a></td></tr>
-               <tr><th width="33%"><%:Phone%>:</th><td><%=contact.phone%></td></tr>
+               <tr><th width="33%"><%:Nickname%>:</th><td><%=nickname%></td></tr>
+               <tr><th width="33%"><%:Realname%>:</th><td><%=name%></td></tr>
+               <tr><th width="33%"><%:E-Mail%>:</th><td><a href="mailto:<%=mail%>"><%=mail%></a></td></tr>
+               <tr><th width="33%"><%:Phone%>:</th><td><%=phone%></td></tr>
        </table>
 </fieldset>
 
@@ -41,11 +52,11 @@ local lat = uci:get_first("system", "system", "latitude")
        </table>
 </fieldset>
 
-<% if contact.note then %>
+<% if note then %>
 <fieldset xmlns="http://www.w3.org/1999/xhtml" class="cbi-section">
 <legend><%:Notice%></legend>
        <table cellspacing="10" width="100%" style="text-align:left">
-               <tr><td><%=contact.note%></td></tr>
+               <tr><td><%=note%></td></tr>
        </table>
 </fieldset>
 <%end%>

diff --git a/modules/freifunk/luasrc/view/freifunk/index.htm b/modules/freifunk/luasrc/view/freifunk/index.htm
index 82261e5..e13d7f9 100644 (file)
--- a/modules/freifunk/luasrc/view/freifunk/index.htm
+++ b/modules/freifunk/luasrc/view/freifunk/index.htm
@@ -15,19 +15,28 @@ $Id$
 <%+header%>
 <% 
 local uci = require "luci.model.uci".cursor()
+local ff = {}
 local ff = uci:get_all("freifunk")
-if not ff.community.name then
-       ff.community.name = ""
+
+if not ff or not ff.community.name then
+       community = "Freifunk"
+       DefaultText = ""
+       nickname = "No Nickname set"
+else
+       community = ff.community.name
+       DefaultText = ff.community.DefaultText
+       nickname = ff.contact.nickname
 end
-local co = "profile_" .. ff.community.name
-local community = uci:get_first(co, "community", "name") or "Freifunk"
+
+local co = "profile_" .. community
+--local community = uci:get_first(co, "community", "name") or "Freifunk"
 local url = uci:get_first(co, "community", "homepage") or "http://www.freifunk.net"
 
 
 require("luci.fs")
 local usertext = luci.fs.readfile("/www/luci-static/index_user.html")
 
-if (ff.community.DefaultText or "") ~= "disabled" then
+if DefaultText ~= "disabled" then
 
        defaulttext = '<h2><a id="content" name="content">'..
        (translate("Hello and welcome in the network of"))..
@@ -44,7 +53,7 @@ if (ff.community.DefaultText or "") ~= "disabled" then
        ' <a href="'..
        luci.dispatcher.build_url("freifunk", "contact")..
        '">'..
-       (ff.contact.nickname or translate("Please set your contact information"))..
+       (nickname or translate("Please set your contact information"))..
        '</a>.</p><p>'..
        translate("You can find further information about the global Freifunk initiative at")..
        ' <a href="http://freifunk.net">Freifunk.net</a>.<br />'..