The output of the command "vnstat --dbiflist 1" is empty when there are
no interfaces in the database. Add a check to avoid a "result.stdout is
undefined" error in that case.
Ref: https://forum.openwrt.org/t/error-in-luci-vnstat2-configuration-menu/179700
Fixes: 3ac4f567df ("luci-app-vnstat2: use more efficient way to get database interfaces")
Signed-off-by: Jan Hoffmann <jan@3e8.eu>
The currently used command "vnstat --json f 1" can be very slow when the
database contains many entries. This can result in an XHR timeout error
on the configuration page.
Use the "--dbiflist" option instead, which was added in vnStat 2.7.
Fixes: #6702
Signed-off-by: Jan Hoffmann <jan@3e8.eu>
This is a huge change to the already existing vnstat2 LuCI module with some
improvements and some new features.
"Graphs Page":
- Implementation of refreshing the graphs.
- Only in the UCI configuration existing interfaces will be shown in any
case. Before this change, all interfaces existing in the database were
shown.
- Introduced a button to clear the statistics for all interfaces (in fact
this is removing and adding again the interfaces from/to the database).
Before clearing the data a confirmation dialog is shown.
- Show user hint if service is not running, so no updating of graphs.
- "Error image" for a graph which can not be loaded
General:
- Updated translations, added missing translations for ./po/de
- Renamed the menu entry from "vnStat Traffic Monitor" to "Traffic Monitor" only
Signed-off-by: Helge Mader <ma@dev.tdt.de>
Modern browsers allow decomposing table markup equally well as nested div
constructs, therefor migrate our <div> table markup to actual <table> tags
but keep the old table/tr/th/td CSS classes for now to allow for a smooth
theme transition.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
This converts the graph rendering to client side JavaScript and replaces
the route registration with declarative JSON.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
