From 0e5be0d68a9daac7d2b04f23bc1e18935cb80145 Mon Sep 17 00:00:00 2001 From: Jo-Philipp Wich Date: Thu, 12 Sep 2019 18:48:58 +0200 Subject: [PATCH] luci-mod-system: reimplent system/crontab as client side view Signed-off-by: Jo-Philipp Wich --- .../root/usr/share/rpcd/acl.d/luci-base.json | 2 + .../resources/view/system/crontab.js | 52 +++++++++++++++++++ .../luasrc/controller/admin/system.lua | 2 +- .../luasrc/model/cbi/admin_system/crontab.lua | 33 ------------ 4 files changed, 55 insertions(+), 34 deletions(-) create mode 100644 modules/luci-mod-system/htdocs/luci-static/resources/view/system/crontab.js delete mode 100644 modules/luci-mod-system/luasrc/model/cbi/admin_system/crontab.lua diff --git a/modules/luci-base/root/usr/share/rpcd/acl.d/luci-base.json b/modules/luci-base/root/usr/share/rpcd/acl.d/luci-base.json index f2ee7ce2c..d364508c2 100644 --- a/modules/luci-base/root/usr/share/rpcd/acl.d/luci-base.json +++ b/modules/luci-base/root/usr/share/rpcd/acl.d/luci-base.json @@ -23,6 +23,7 @@ "file": { "/": [ "list" ], "/*": [ "list" ], + "/etc/crontabs/root": [ "read" ], "/etc/rc.local": [ "read" ], "/proc/sys/kernel/hostname": [ "read" ] }, @@ -40,6 +41,7 @@ "write": { "cgi-io": [ "upload", "/etc/luci-uploads/*" ], "file": { + "/etc/crontabs/root": [ "write" ], "/etc/luci-uploads/*": [ "write" ], "/etc/rc.local": [ "write" ] }, diff --git a/modules/luci-mod-system/htdocs/luci-static/resources/view/system/crontab.js b/modules/luci-mod-system/htdocs/luci-static/resources/view/system/crontab.js new file mode 100644 index 000000000..c8baa4722 --- /dev/null +++ b/modules/luci-mod-system/htdocs/luci-static/resources/view/system/crontab.js @@ -0,0 +1,52 @@ +'use strict'; +'require rpc'; + +return L.view.extend({ + callFileRead: rpc.declare({ + object: 'file', + method: 'read', + params: [ 'path' ], + expect: { data: '' } + }), + + callFileWrite: rpc.declare({ + object: 'file', + method: 'write', + params: [ 'path', 'data' ] + }), + + load: function() { + return this.callFileRead('/etc/crontabs/root'); + }, + + handleSave: function(ev) { + var value = (document.querySelector('textarea').value || '').trim().replace(/\r\n/g, '\n') + '\n'; + + return this.callFileWrite('/etc/crontabs/root', value).then(function(rc) { + if (rc != 0) + throw rpc.getStatusText(rc); + + document.querySelector('textarea').value = value; + L.ui.addNotification(null, E('p', _('Contents have been saved.')), 'info'); + + }).catch(function(e) { + L.ui.addNotification(null, E('p', _('Unable to save contents: %s').format(e))); + }); + }, + + render: function(crontab) { + return E([ + E('h2', _('Scheduled Tasks')), + E('p', {}, + _('This is the system crontab in which scheduled tasks can be defined.') + + _('
Note: you need to manually restart the cron service if the crontab file was empty before editing.')), + E('p', {}, E('textarea', { 'style': 'width:100%', 'rows': 10 }, crontab != null ? crontab : '')), + E('div', { 'class': 'right' }, [ + E('button', { + 'class': 'btn cbi-button-positive important', + 'click': L.ui.createHandlerFn(this, 'handleSave') + }, _('Save')) + ]) + ]); + } +}); diff --git a/modules/luci-mod-system/luasrc/controller/admin/system.lua b/modules/luci-mod-system/luasrc/controller/admin/system.lua index c3e3678fd..b9785994a 100644 --- a/modules/luci-mod-system/luasrc/controller/admin/system.lua +++ b/modules/luci-mod-system/luasrc/controller/admin/system.lua @@ -22,7 +22,7 @@ function index() end entry({"admin", "system", "startup"}, view("system/startup"), _("Startup"), 45) - entry({"admin", "system", "crontab"}, form("admin_system/crontab"), _("Scheduled Tasks"), 46) + entry({"admin", "system", "crontab"}, view("system/crontab"), _("Scheduled Tasks"), 46) if fs.access("/sbin/block") and fs.access("/etc/config/fstab") then entry({"admin", "system", "fstab"}, cbi("admin_system/fstab"), _("Mount Points"), 50) diff --git a/modules/luci-mod-system/luasrc/model/cbi/admin_system/crontab.lua b/modules/luci-mod-system/luasrc/model/cbi/admin_system/crontab.lua deleted file mode 100644 index beb24e7ca..000000000 --- a/modules/luci-mod-system/luasrc/model/cbi/admin_system/crontab.lua +++ /dev/null @@ -1,33 +0,0 @@ --- Copyright 2008 Steven Barth --- Copyright 2008-2013 Jo-Philipp Wich --- Licensed to the public under the Apache License 2.0. - -local fs = require "nixio.fs" -local cronfile = "/etc/crontabs/root" - -f = SimpleForm("crontab", translate("Scheduled Tasks"), - translate("This is the system crontab in which scheduled tasks can be defined.") .. - translate("
Note: you need to manually restart the cron service if the " .. - "crontab file was empty before editing.")) - -t = f:field(TextValue, "crons") -f.forcewrite = true -t.rmempty = true -t.rows = 10 -function t.cfgvalue() - return fs.readfile(cronfile) or "" -end - -function f.handle(self, state, data) - if state == FORM_VALID then - if data.crons then - fs.writefile(cronfile, data.crons:gsub("\r\n", "\n")) - luci.sys.call("/usr/bin/crontab %q" % cronfile) - else - fs.writefile(cronfile, "") - end - end - return true -end - -return f -- 2.25.1