[pbs-devel] [PATCH proxmox-backup] fix: docs: prune sim: show "keep" entries in backup list
Friedrich Weber
f.weber at proxmox.com
Wed Jan 22 14:38:09 CET 2025
Currently, the list of backups only shows removed backups and is
missing backups that are kept, though they are shown correctly in the
calendar view.
The reason is that a refactor (see Fixes tag) moved the definition of
a custom field renderer referencing `me` to a scope where `me` is not
defined. This causes the renderer to error out for "kept" backups,
which apparently causes the grid to skip the rows altogether (without
any messages in the console).
Fix this by replacing the broken `me` reference.
Fixes: bb044304 ("prune sim: move PruneList to more static declaration")
Signed-off-by: Friedrich Weber <f.weber at proxmox.com>
---
docs/prune-simulator/prune-simulator_source.js | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/docs/prune-simulator/prune-simulator_source.js b/docs/prune-simulator/prune-simulator_source.js
index 4bd31acd..0fb779e6 100644
--- a/docs/prune-simulator/prune-simulator_source.js
+++ b/docs/prune-simulator/prune-simulator_source.js
@@ -126,7 +126,8 @@ Ext.onReady(function() {
if (data.mark !== 'keep') {
return `<div style="text-decoration: line-through;">${text}</div>`;
}
- if (me.useColors) {
+ let pruneList = this.up('prunesimulatorPruneList');
+ if (pruneList.useColors) {
let bgColor = COLORS[data.keepName];
let textColor = TEXT_COLORS[data.keepName];
return `<div style="background-color: ${bgColor};color: ${textColor};">${text}</div>`;
--
2.39.5
More information about the pbs-devel
mailing list