Difuse/luci
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

luci-base, luci-lua-runtime: fix "dispatched" and "requested" properties

Browse source 
Ensure to properly emulate the "dispatched" and "requested" properties
which refer to the executed and initially resolved menu node respectivey.

Also stop exposing a `node` property in Lua context to maintain full
compatibility with the old Lua runtime.

Signed-off-by: Jo-Philipp Wich <jo@mein.io>
This commit is contained in:
Jo-Philipp Wich 2022-11-21 12:24:31 +01:00
parent c4d86c9f67
commit 08ae8d40a9
2 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
modules/luci-base/ucode/dispatcher.uc
View file

@ -856,7 +856,7 @@ dispatch = function(_http, path) {
let version = determine_version();
let lang = determine_request_language();
runtime = LuCIRuntime({
runtime = runtime || LuCIRuntime({
http,
ubus,
uci,
@ -892,7 +892,8 @@ dispatch = function(_http, path) {
let resolved = resolve_page(menu, path);
runtime.env.ctx = resolved.ctx;
runtime.env.node = resolved.node;
runtime.env.dispatched = resolved.node;
runtime.env.requested ??= resolved.node;
if (length(resolved.ctx.auth)) {
let session = is_authenticated(resolved.ctx.auth);

2
modules/luci-lua-runtime/luasrc/dispatcher.lua
View file

@ -12,8 +12,6 @@ context = setmetatable({}, {
return _G.L.ctx.request_path
elseif k == "requestargs" then
return _G.L.ctx.request_args
elseif k == "requested" or k == "dispatched" then
return _G.L.node
else
return _G.L.ctx[k]
end