Project

General

Profile

Actions

Bug #12774

open

Picture widget image is not saved in backup

Added by Viktor Gurov almost 3 years ago. Updated 11 months ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Backup / Restore
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Plus Target Version:
Release Notes:
Default
Affected Version:
2.5.2
Affected Architecture:

Description

After restoring from a backup, the dashboard "picture widget" image is blank


Related issues

Related to Feature #8371: Reduce config.xml size by removing picture widget images to file systemResolved03/16/2018

Actions
Has duplicate Bug #13021: Image data of dashboard image widget does not get backed upDuplicate

Actions
Actions #1

Updated by Jim Pingle almost 3 years ago

That was an intentional change. See 1f0bbb13abd34ad06aa9272516b13a5c17a1dc08

Maybe we could suppress the picture widget if the file is not present, but we should not change it back to storing image data in the configuration.

Actions #2

Updated by Jim Pingle almost 3 years ago

  • Related to Feature #8371: Reduce config.xml size by removing picture widget images to file system added
Actions #3

Updated by Viktor Gurov almost 3 years ago

But we can only backup image data if the "Include extra data" option is checked.

Actions #4

Updated by Viktor Gurov over 2 years ago

  • Has duplicate Bug #13021: Image data of dashboard image widget does not get backed up added
Actions #5

Updated by Ronald Antony over 2 years ago

Where the picture data is stored while the system is operating is IMO of no consequence regarding as to whether or not it should be backed up.

If for the sake of robustness/performance the configuration xml file is reduced in size by removing said picture data from the config file and storing it on the file system, is utterly orthogonal to the question as to whether or not the picture is part of the configuration, which IMO it undeniably is.

This isn’t some data related to a third party packaging or some data that found it’s way into the system by some custom tweaks: it a widget that’s part of the base system, and which provides important visual cues.

Actions #6

Updated by Ronald Antony 11 months ago

Viktor Gurov wrote in #note-3:

But we can only backup image data if the "Include extra data" option is checked.

Exactly!
If data intensive things like RRD can be backed up, I don't see why the picture data shouldn't be optionally backed up, too!

Actions #7

Updated by Kris Phillips 11 months ago

Ronald Antony wrote in #note-6:

Viktor Gurov wrote in #note-3:

But we can only backup image data if the "Include extra data" option is checked.

Exactly!
If data intensive things like RRD can be backed up, I don't see why the picture data shouldn't be optionally backed up, too!

The image would have to be a separate file, since the backup function exports just the config.xml file. However, this would also require creating a compressed archive that contains both the image file and the config.xml file. Backing up the image data into the XML file is a terrible idea given there is no good way to limit the file size of the image data.

Exporting the image file separately in an archive or omitting the image widget if the image file is non-existent are the better options from a security and functionality standpoint.

Actions

Also available in: Atom PDF