* General code cleanup

* Removed unnecessary module dependencies
* Completed SGI abstraction
* Reworked ffluci.sgi.webuci

diff --git a/contrib/package/lua-luci/patches/200-lua-path b/contrib/package/lua-luci/patches/200-lua-path
new file mode 100644 (file)
index 0000000..62dd00e
--- /dev/null
+++ b/contrib/package/lua-luci/patches/200-lua-path
@@ -0,0 +1,15 @@
+--- b/src/luaconf.h    2008-05-06 20:10:46.000000000 +0200
++++ a/src/luaconf.h    2008-05-06 20:10:27.000000000 +0200
+@@ -95,9 +95,9 @@
+       ".\\?.dll;"  LUA_CDIR"?.dll;" LUA_CDIR"loadall.dll"
+ 
+ #else
+-#define LUA_ROOT      "/usr/local/"
+-#define LUA_LDIR      LUA_ROOT "share/lua/5.1/"
+-#define LUA_CDIR      LUA_ROOT "lib/lua/5.1/"
++#define LUA_ROOT      "/usr/"
++#define LUA_LDIR      LUA_ROOT "share/lua/"
++#define LUA_CDIR      LUA_ROOT "lib/lua/"
+ #define LUA_PATH_DEFAULT  \
+               "./?.lua;"  LUA_LDIR"?.lua;"  LUA_LDIR"?/init.lua;" \
+                           LUA_CDIR"?.lua;"  LUA_CDIR"?/init.lua"

diff --git a/core/contrib/webuci_bootstrap.lua b/core/contrib/webuci_bootstrap.lua
index 1733437..fd4c47a 100644 (file)
--- a/core/contrib/webuci_bootstrap.lua
+++ b/core/contrib/webuci_bootstrap.lua
@@ -1,2 +1,22 @@
 package.path  = "/usr/lib/lua/?.lua;/usr/lib/lua/?/init.lua;" .. package.path
 package.cpath = "/usr/lib/lua/?.so;" .. package.cpath
 package.path  = "/usr/lib/lua/?.lua;/usr/lib/lua/?/init.lua;" .. package.path
 package.cpath = "/usr/lib/lua/?.so;" .. package.cpath

+module("webuci", package.seeall)
+
+function prepare_req(uri)
+       require("ffluci.menu").get()
+       REQUEST_URI = uri
+end
+
+function init_req(context)
+       SERVER_PROTOCOL = context.server_proto
+       REMOTE_ADDR     = context.remote_addr
+       REQUEST_METHOD  = context.request_method
+       PATH_INFO       = "/" .. context.uri
+       REMOTE_PORT     = context.remote_port
+       SERVER_ADDR     = context.server_addr
+       SCRIPT_NAME     = REQUEST_URI:sub(1, #REQUEST_URI - #context.uri)
+end
+
+function handle_req(context)
+       require("ffluci.dispatcher").httpdispatch()
+end
\ No newline at end of file

diff --git a/core/src/ffluci/cbi.lua b/core/src/ffluci/cbi.lua
index 4d6d25c..3384e5c 100644 (file)
--- a/core/src/ffluci/cbi.lua
+++ b/core/src/ffluci/cbi.lua
@@ -39,8 +39,9 @@ function load(cbimap)
        require("ffluci.fs")
        require("ffluci.i18n")
        require("ffluci.config")
        require("ffluci.fs")
        require("ffluci.i18n")
        require("ffluci.config")

+       require("ffluci.sys")

        
        

-       local cbidir = ffluci.config.path .. "/model/cbi/"
+       local cbidir = ffluci.sys.libpath() .. "/model/cbi/"

        local func, err = loadfile(cbidir..cbimap..".lua")
        
        if not func then
        local func, err = loadfile(cbidir..cbimap..".lua")
        
        if not func then

diff --git a/core/src/ffluci/config.lua b/core/src/ffluci/config.lua
index 90fa6b1..0db45ac 100644 (file)
--- a/core/src/ffluci/config.lua
+++ b/core/src/ffluci/config.lua
@@ -28,10 +28,7 @@ limitations under the License.
 module("ffluci.config", package.seeall)
 require("ffluci.model.uci")
 require("ffluci.util")
 module("ffluci.config", package.seeall)
 require("ffluci.model.uci")
 require("ffluci.util")

-require("ffluci.debug")
-
--- Our path (wtf Lua lacks __file__ support)
-path = ffluci.debug.path
+require("ffluci.sys")

 
 -- Warning! This is only for fallback and compatibility purporses! --
 main = {}
 
 -- Warning! This is only for fallback and compatibility purporses! --
 main = {}

diff --git a/core/src/ffluci/debug.lua b/core/src/ffluci/debug.lua
index f1132ed..1be4034 100644 (file)
--- a/core/src/ffluci/debug.lua
+++ b/core/src/ffluci/debug.lua
@@ -1,2 +1,2 @@
 module("ffluci.debug", package.seeall)
 module("ffluci.debug", package.seeall)

-path = require("ffluci.fs").dirname(debug.getinfo(1, 'S').source:sub(2))
\ No newline at end of file
+__file__ = debug.getinfo(1, 'S').source:sub(2)
\ No newline at end of file

diff --git a/core/src/ffluci/dispatcher.lua b/core/src/ffluci/dispatcher.lua
index 15ac3c7..813e35d 100644 (file)
--- a/core/src/ffluci/dispatcher.lua
+++ b/core/src/ffluci/dispatcher.lua
@@ -150,7 +150,7 @@ end
 
 -- Dispatches a request depending on the PATH_INFO variable
 function httpdispatch()
 
 -- Dispatches a request depending on the PATH_INFO variable
 function httpdispatch()

-       local pathinfo = os.getenv("PATH_INFO") or ""
+       local pathinfo = ffluci.http.get_path_info() or ""

        local parts = pathinfo:gmatch("/[%w-]+")
        
        local sanitize = function(s, default)
        local parts = pathinfo:gmatch("/[%w-]+")
        
        local sanitize = function(s, default)

diff --git a/core/src/ffluci/fs.lua b/core/src/ffluci/fs.lua
index 6e8859a..3eb562b 100644 (file)
--- a/core/src/ffluci/fs.lua
+++ b/core/src/ffluci/fs.lua
@@ -28,6 +28,11 @@ module("ffluci.fs", package.seeall)
 
 require("posix")
 
 
 require("posix")
 

+-- Glob
+function glob(pattern)
+       return posix.glob(pattern)
+end
+

 -- Checks whether a file exists
 function isfile(filename)
        local fp = io.open(path, "r")
 -- Checks whether a file exists
 function isfile(filename)
        local fp = io.open(path, "r")

diff --git a/core/src/ffluci/i18n.lua b/core/src/ffluci/i18n.lua
index 11f4afe..489308c 100644 (file)
--- a/core/src/ffluci/i18n.lua
+++ b/core/src/ffluci/i18n.lua
@@ -25,11 +25,11 @@ limitations under the License.
 ]]--
 
 module("ffluci.i18n", package.seeall)
 ]]--
 
 module("ffluci.i18n", package.seeall)

-

 require("ffluci.config")
 require("ffluci.config")

+require("ffluci.sys")

 
 table   = {}
 
 table   = {}

-i18ndir = ffluci.config.path .. "/i18n/"
+i18ndir = ffluci.sys.libpath() .. "/i18n/"

 
 -- Clears the translation table
 function clear()
 
 -- Clears the translation table
 function clear()

diff --git a/core/src/ffluci/menu.lua b/core/src/ffluci/menu.lua
index e55c0e7..5cbb725 100644 (file)
--- a/core/src/ffluci/menu.lua
+++ b/core/src/ffluci/menu.lua
@@ -27,18 +27,15 @@ module("ffluci.menu", package.seeall)
 
 require("ffluci.fs")
 require("ffluci.util")
 
 require("ffluci.fs")
 require("ffluci.util")

-require("ffluci.template")
-require("ffluci.i18n")
-require("ffluci.config")
-require("ffluci.model.ipkg")
+require("ffluci.sys")

 
 -- Default modelpath
 
 -- Default modelpath

-modelpath = ffluci.config.path .. "/model/menu/"
+modelpath = ffluci.sys.libpath() .. "/model/menu/"

 
 -- Menu definition extra scope
 scope = {
 
 -- Menu definition extra scope
 scope = {

-       translate = ffluci.i18n.translate,
-       loadtrans = ffluci.i18n.loadc,
+       translate = function(...) return require("ffluci.i18n").translate(...) end,
+       loadtrans = function(...) return require("ffluci.i18n").loadc(...) end,

        isfile    = ffluci.fs.mtime
 }
 
        isfile    = ffluci.fs.mtime
 }
 


@@ -101,26 +98,40 @@ end
 
 -- Collect all menu information provided in the model dir
 function collect()
 
 -- Collect all menu information provided in the model dir
 function collect()

+       local generators = {}
+       

        for k, menu in pairs(ffluci.fs.dir(modelpath)) do
                if menu:sub(1, 1) ~= "." then
                        local f = loadfile(modelpath.."/"..menu)
        for k, menu in pairs(ffluci.fs.dir(modelpath)) do
                if menu:sub(1, 1) ~= "." then
                        local f = loadfile(modelpath.."/"..menu)

-                       local env = ffluci.util.clone(scope)
-                       
-                       env.add = add
-                       env.sel = sel
-                       env.act = act
-                       
-                       setfenv(f, env)
-                       f()
+                       if f then
+                               table.insert(generators, f)
+                       end

                end
        end
                end
        end

+       
+       return generators
+end
+
+-- Parse the collected information
+function parse(generators)
+       menu = {}
+       for i, f in pairs(generators) do
+               local env = ffluci.util.clone(scope)
+               
+               env.add = add
+               env.sel = sel
+               env.act = act
+               
+               setfenv(f, env)
+               f()
+       end
+       return menu

 end
 
 -- Returns the menu information
 function get()
        if not menu then
 end
 
 -- Returns the menu information
 function get()
        if not menu then

-               menu = {}
-               collect()
+               menu = parse(collect())

        end
        return menu
 end
\ No newline at end of file
        end
        return menu
 end
\ No newline at end of file

diff --git a/core/src/ffluci/sgi/haserl.lua b/core/src/ffluci/sgi/haserl.lua
index 76dc385..e58189d 100644 (file)
--- a/core/src/ffluci/sgi/haserl.lua
+++ b/core/src/ffluci/sgi/haserl.lua
@@ -54,12 +54,21 @@ function ffluci.http.formvaluetable(prefix)
        return ffluci.http.formvalue(prefix, {})
 end
 
        return ffluci.http.formvalue(prefix, {})
 end
 

+-- Returns the path info
+function ffluci.http.get_path_info()
+       return ENV.PATH_INFO
+end

 
 -- Returns the User's IP
 function ffluci.http.get_remote_addr()
        return ENV.REMOTE_ADDR
 end
 
 
 -- Returns the User's IP
 function ffluci.http.get_remote_addr()
        return ENV.REMOTE_ADDR
 end
 

+-- Returns the request URI
+function ffluci.http.get_request_uri()
+       return ENV.REQUEST_URI
+end
+

 -- Returns the script name
 function ffluci.http.get_script_name()
        return ENV.SCRIPT_NAME
 -- Returns the script name
 function ffluci.http.get_script_name()
        return ENV.SCRIPT_NAME

diff --git a/core/src/ffluci/sgi/webuci.lua b/core/src/ffluci/sgi/webuci.lua
index 2977802..c609646 100644 (file)
--- a/core/src/ffluci/sgi/webuci.lua
+++ b/core/src/ffluci/sgi/webuci.lua
@@ -51,15 +51,25 @@ function ffluci.http.formvaluetable(prefix)
        return vals
 end
 
        return vals
 end
 

+-- Returns the path info
+function ffluci.http.get_path_info()
+       return webuci.PATH_INFO
+end

 
 -- Returns the User's IP
 function ffluci.http.get_remote_addr()
 
 -- Returns the User's IP
 function ffluci.http.get_remote_addr()

-       return os.getenv("REMOTE_ADDR")
+       return webuci.REMOTE_ADDR
+end
+
+-- Returns the request URI
+function ffluci.http.get_request_uri()
+       return webuci.REQUEST_URI

 end
 
 end
 

+

 -- Returns the script name
 function ffluci.http.get_script_name()
 -- Returns the script name
 function ffluci.http.get_script_name()

-       return os.getenv("SCRIPT_NAME")
+       return webuci.SCRIPT_NAME

 end
 
 
 end
 
 


@@ -81,5 +91,5 @@ end
 
 -- Sets HTTP-Status-Header
 function ffluci.http.set_status(code, message)
 
 -- Sets HTTP-Status-Header
 function ffluci.http.set_status(code, message)

-       print("Status: " .. tostring(code) .. " " .. message)
+       print(webuci.REQUEST_METHOD .. " " .. tostring(code) .. " " .. message)

 end
 end

diff --git a/core/src/ffluci/sys.lua b/core/src/ffluci/sys.lua
index d71bce7..3766548 100644 (file)
--- a/core/src/ffluci/sys.lua
+++ b/core/src/ffluci/sys.lua
@@ -82,6 +82,11 @@ function httpget(url)
        return exec("wget -qO- '"..url:gsub("'", "").."'")
 end
 
        return exec("wget -qO- '"..url:gsub("'", "").."'")
 end
 

+-- Returns the FFLuci-Basedir
+function libpath()
+       return ffluci.fs.dirname(require("ffluci.debug").__file__)
+end
+

 -- Returns the load average
 function loadavg()
        local loadavg = io.lines("/proc/loadavg")()
 -- Returns the load average
 function loadavg()
        local loadavg = io.lines("/proc/loadavg")()

diff --git a/core/src/ffluci/template.lua b/core/src/ffluci/template.lua
index 04cc07d..a972f2b 100644 (file)
--- a/core/src/ffluci/template.lua
+++ b/core/src/ffluci/template.lua
@@ -28,11 +28,9 @@ module("ffluci.template", package.seeall)
 require("ffluci.config")
 require("ffluci.util")
 require("ffluci.fs")
 require("ffluci.config")
 require("ffluci.util")
 require("ffluci.fs")

-require("ffluci.i18n")

 require("ffluci.http")
 require("ffluci.http")

-require("ffluci.model.uci")

 
 

-viewdir = ffluci.config.path .. "/view/"
+viewdir = ffluci.sys.libpath() .. "/view/"

 
 
 -- Compile modes:
 
 
 -- Compile modes:


@@ -52,8 +50,8 @@ compiler_enable_bytecode = false
 
 -- Define the namespace for template modules
 viewns = {
 
 -- Define the namespace for template modules
 viewns = {

-       translate  = ffluci.i18n.translate,
-       config     = function(...) return ffluci.model.uci.get(...) or "" end,
+       translate  = function(...) return require("ffluci.i18n").translate(...) end,
+       config     = function(...) return require("ffluci.model.uci").get(...) or "" end,

        controller = ffluci.http.get_script_name(),
        media      = ffluci.config.main.mediaurlbase,
        write      = io.write,
        controller = ffluci.http.get_script_name(),
        media      = ffluci.config.main.mediaurlbase,
        write      = io.write,

diff --git a/core/src/ffluci/util.lua b/core/src/ffluci/util.lua
index 3ff7bc2..b76278d 100644 (file)
--- a/core/src/ffluci/util.lua
+++ b/core/src/ffluci/util.lua
@@ -139,12 +139,6 @@ function resfenv(f)
 end 
 
 
 end 
 
 

--- Returns the Haserl unique sessionid
-function sessionid()
-       return ENV.SESSIONID
-end
-
-

 -- Splits a string into an array
 function split(str, pat, max, regex)
        pat = pat or "\n"
 -- Splits a string into an array
 function split(str, pat, max, regex)
        pat = pat or "\n"

diff --git a/core/src/ffluci/view/cbi/header.htm b/core/src/ffluci/view/cbi/header.htm
index 1b69a3a..97542f0 100644 (file)
--- a/core/src/ffluci/view/cbi/header.htm
+++ b/core/src/ffluci/view/cbi/header.htm
@@ -1,5 +1,5 @@
 <%+header%>
 <%+header%>

-               <form method="post" action="<%=os.getenv("REQUEST_URI")%>">
+               <form method="post" action="<%=ffluci.http.get_request_uri()%>">

                        <div>
                                <script type="text/javascript" src="<%=media%>/cbi.js"></script>
                                <input type="hidden" name="cbi.submit" value="1" />
                        <div>
                                <script type="text/javascript" src="<%=media%>/cbi.js"></script>
                                <input type="hidden" name="cbi.submit" value="1" />

diff --git a/core/src/ffluci/view/error404.htm b/core/src/ffluci/view/error404.htm
index adc671d..1a9d74a 100644 (file)
--- a/core/src/ffluci/view/error404.htm
+++ b/core/src/ffluci/view/error404.htm
@@ -1,5 +1,5 @@
 <%+header%>
 <h1>404 Not Found</h1>
 <p>Sorry, the object you requested was not found.</p>
 <%+header%>
 <h1>404 Not Found</h1>
 <p>Sorry, the object you requested was not found.</p>

-<tt>Unable to dispatch: <%=os.getenv("PATH_INFO")%></tt>
+<tt>Unable to dispatch: <%=ffluci.http.get_path_info()%></tt>

 <%+footer%>
\ No newline at end of file
 <%+footer%>
\ No newline at end of file