luci-mod-system: fix SimpleForm usage on file editing pages
authorJo-Philipp Wich <jo@mein.io>
Wed, 5 Jun 2019 14:01:16 +0000 (16:01 +0200)
committerJo-Philipp Wich <jo@mein.io>
Wed, 5 Jun 2019 14:16:48 +0000 (16:16 +0200)
commit9fc05682f603d4b47270ae4217558b4381b5cc06
treeeaa71a70197e5d04066eefbc654b4306cbbdbdb8
parent0dd8878837464901bdb7a0e61afb907b84b11c53
luci-mod-system: fix SimpleForm usage on file editing pages

When a value identical to the stored one is submitted, the CBI framework
will not emit an option write event and therfore not store the value in
the form data dictionary passed to SimpleForm.handle().

This usage pattern usally works be accident for file editor views such
as admin_system/crontab because \r\n windows style line endings are
substituted with unix \n ones before writing the data, defeating the
equality check in CBI.

When a single line without trailing newline is submitted however, the
CBI will not see a difference to the data stored in the file and clear
out the value on subsequent saves.

This commit alignes the logic used by various SimpleForm views to
behave identically and predictable:

 - File data is handled in the SimpleForm.handle() callback
 - The forcewrite property is used to disable equality checks
 - Submission of an empty string empties the backing file

Fixes: #2737
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
(cherry picked from commit 1c09ee5e42550d6339bffa58d4cba3461948e19c)
modules/luci-mod-admin-full/luasrc/model/cbi/admin_system/backupfiles.lua
modules/luci-mod-admin-full/luasrc/model/cbi/admin_system/crontab.lua
modules/luci-mod-admin-full/luasrc/model/cbi/admin_system/startup.lua