Skip to content

Commit

Permalink
luci-app-commands: avoid deprecated XHR() usage
Browse files Browse the repository at this point in the history
Replace direct usage of deprecated XHR() wrapper with L.request class calls.

Signed-off-by: Jo-Philipp Wich <[email protected]>
  • Loading branch information
jow- committed Jul 27, 2024
1 parent 0f9c168 commit 702c007
Showing 1 changed file with 18 additions and 31 deletions.
49 changes: 18 additions & 31 deletions applications/luci-app-commands/ucode/template/commands.ut
Original file line number Diff line number Diff line change
Expand Up @@ -29,15 +29,9 @@
-%}

<script type="text/javascript">//<![CDATA[
var stxhr = new XHR();

function command_run(ev, id)
{
var args;
var field = document.getElementById(id);
if (field)
args = encodeURIComponent(field.value);

var legend = document.getElementById('command-rc-legend');
var output = document.getElementById('command-rc-output');

Expand All @@ -51,31 +45,24 @@
legend.parentNode.style.display = 'block';
legend.style.display = 'inline';

stxhr.get(L.url('admin/system/commands/run', id) + (args ? '?args=' + args : ''), null,
function(x, st)
{
if (st)
{
if (st.binary)
st.stdout = '[' + _('Binary data not displayed, download instead.') + ']';

legend.style.display = 'none';
output.innerHTML = String.format(
'<pre><strong># %h\n</strong>%h<span style="color:red">%h</span></pre>' +
'<div class="alert-message warning">%h (%h %d)</div>',
st.command, st.stdout, st.stderr,
(st.exitcode == 0) ? _('Command successful') : _('Command failed'),
_('Code:'), st.exitcode);
}
else
{
legend.style.display = 'none';
output.innerHTML = '<span class="error">%h</span>'.format(_('Failed to execute command!'));
}

location.hash = '#output';
}
);
L.Request.get(L.url('admin/system/commands/run', id), field ? { args: field.value } : null).then(function(reply) {
var st = reply.json();

if (st.binary)
st.stdout = '[' + _('Binary data not displayed, download instead.') + ']';

output.innerHTML = String.format(
'<pre><strong># %h\n</strong>%h<span style="color:red">%h</span></pre>' +
'<div class="alert-message warning">%h (%h %d)</div>',
st.command, st.stdout, st.stderr,
(st.exitcode == 0) ? _('Command successful') : _('Command failed'),
_('Code:'), st.exitcode);
}).catch(function() {
output.innerHTML = '<span class="error">%h</span>'.format(_('Failed to execute command!'));
}).finally(function() {
legend.style.display = 'none';
location.hash = '#output';
});
}

ev.preventDefault();
Expand Down

0 comments on commit 702c007

Please sign in to comment.