diff options
| author | Paul Spooren <paul@spooren.de> | 2017-08-14 19:13:23 +0200 |
|---|---|---|
| committer | Paul Spooren <paul@spooren.de> | 2017-08-14 19:14:52 +0200 |
| commit | 5cdab68f25f92dd26eb3f218ebe648740eaef4dd (patch) | |
| tree | 6e1dfa9a7e1acf1a3323bbba7262b73dbecf3be6 /applications/luci-app-attendedsysupgrade | |
| parent | 09e0519fca7ac3bbb6d391ca5c04a1735ee1d49f (diff) | |
luci-app-attendedsysupgrade: use JSON.stringify
now uses JSON.stringify instead of creating pseudo JSON data via
`'{' + variable + '}'`
also simplify uci naming
Signed-off-by: Paul Spooren <paul@spooren.de>
Diffstat (limited to 'applications/luci-app-attendedsysupgrade')
| -rw-r--r-- | applications/luci-app-attendedsysupgrade/luasrc/view/attendedsysupgrade.htm | 22 |
1 files changed, 13 insertions, 9 deletions
diff --git a/applications/luci-app-attendedsysupgrade/luasrc/view/attendedsysupgrade.htm b/applications/luci-app-attendedsysupgrade/luasrc/view/attendedsysupgrade.htm index ea97e9a51..1edafa7c7 100644 --- a/applications/luci-app-attendedsysupgrade/luasrc/view/attendedsysupgrade.htm +++ b/applications/luci-app-attendedsysupgrade/luasrc/view/attendedsysupgrade.htm @@ -41,7 +41,11 @@ function server_request(request_dict, path, callback) { // requests ubus via rpcd function ubus_request(command, argument, params, callback) { - var request_data = '{ "jsonrpc": "2.0", "id": ' + ubus_counter + ', "method": "call", "params": [ "'+ data["ubus_rpc_session"] +'", "' + command + '", "' + argument + '", ' + params + ' ] }' + request_data = {}; + request_data.jsonrpc = "2.0"; + request_data.id = ubus_counter; + request_data.method = "call"; + request_data.params = [ data.ubus_rpc_session, command, argument, params ] ubus_counter++ var xmlhttp = new XMLHttpRequest(); xmlhttp.open("POST", ubus_url, true); @@ -56,7 +60,7 @@ function ubus_request(command, argument, params, callback) { ubus_request_callback(xmlhttp, callback) } }); - xmlhttp.send(request_data); + xmlhttp.send(JSON.stringify(request_data)); } // handle ubus_requests, set variables or perform functions @@ -87,11 +91,11 @@ function ubus_request_callback_uci(response_object, callback) { // initial setup, get system information function setup() { data["ubus_rpc_session"] = "<%=luci.dispatcher.context.authsession%>" - ubus_request("packagelist", "list", '{ }', "packagelist") - ubus_request("system", "board", '{ }', "release") - ubus_request("system", "board", '{ }', "board_name") - ubus_request("system", "board", '{ }', "model") - ubus_request("uci", "get", '{"config": "attendedsysupgrade", "section": "@settings[0]", "option": "update_server"}', "update_server") + ubus_request("packagelist", "list", {}, "packagelist"); + ubus_request("system", "board", {}, "release"); + ubus_request("system", "board", {}, "board_name"); + ubus_request("system", "board", {}, "model"); + ubus_request("uci", "get", { "config": "attendedsysupgrade", "section": "updateserver", "option": "url" }, "update_server") } // shows notification if update is available @@ -149,7 +153,7 @@ function update_request_callback(response_object) { } function back_online() { - ubus_request("session", "login", '{ }', back_online_callback) + ubus_request("session", "login", {}, back_online_callback) } function back_online_callback(response_object) { @@ -244,7 +248,7 @@ function upload_image(blob) { // this checksum should be parsed document.getElementById("update_info").innerHTML = "flashing... please wait" // show fancy indicator http://www.ajaxload.info/ - ubus_request("attendedsysupgrade", "sysupgrade", '{ "keep_settings": ' + document.getElementById("keep").checked + ' }', 'done'); + ubus_request("attendedsysupgrade", "sysupgrade", { "keep_settings": document.getElementById("keep").checked }, 'done'); }); upload_request.addEventListener('error', function(event) { |