Aggregate Graph is being shown more than once, leading to layout/display issues

[DavidLiedke]

Describe the bug
Unable to see Aggregate Graph in Device View

To Reproduce
Steps to reproduce the behavior:

1. Go to Graphs
2. Click on selected device
3. Scroll down to the aggregate graph

Expected behavior
If you click on "Graph Details" or "Time Graph View" you can see the graph.

Screenshots
1.

2.

[cigamit]

@DavidLiedke, is this a Windows box? If not, provide the debug output (rrdtool graph ...). If it is, Windows has problems with long strings, or used to years ago. Not sure if it still persists.

[DavidLiedke]

No it is a linux system (openSUSE 15.0).

I need to correct my error message. The graph is displayed after all. But cacti wants to display it next to the graph again. This only happens in the Device View.

Overview:

Graph 1:

graph.php?action=zoom&local_graph_id=840&rra_id=0&graph_start=1556171100&graph_end=1556257500

graph.php?local_graph_id=840

graph_view.php?page=1&graph_template_id=-1&host_id=-1&filter=&style=selective&action=preview&graph_add=14%2C203%2C815%2C816%2C

Graph 2:

graph.php?action=zoom&local_graph_id=840&rra_id=0&graph_start=1556171100&graph_end=1556257500

graph.php?local_graph_id=840

graph_view.php?page=1&graph_template_id=-1&host_id=-1&filter=&style=selective&action=preview&graph_add=14%2C203%2C815%2C816%2C

[kurowskydb]

What browser are you using? Also try private / incognito tab to see if extensions are affecting. Had similar issue of a particular type of graph not showing in chrome. Disabled some extensions, restarted and the graph appeared... :/

[cigamit]

Need to see the browser console, and the debug output.

[DavidLiedke]

Issue exist in all browsers (Microsoft Edge, Google Chrome and Firefox).
Microsoft Edge has no extensions. In Firefox & Google Chrome the extensions are disabled for this URL.

In the debug log i cant find anything. Also there is no entry in the apache error log.

[netniV]

So the setTimeout warning is that it is above the 50ms which is chromes default warning limit for execution time of animation based events. See https://stackoverflow.com/questions/41218507/violation-long-running-javascript-task-took-xx-ms and https://stackoverflow.com/questions/42218699/javascript-chrome-violation-violation-handler-took-83ms-of-runtime/44824402#44824402

So, it's unlikely that they are what is causing your issue. Since everything appears to be OK with the status result etc, I can only assume that the data coming back isn't a valid image (or has a PHP notification/warning smack in the middle of it corrupting the data).

When you browse those URLs directly, what do you get? Can you use wget to download the data and post it as a data (rather than image file)?

[DavidLiedke]

If i browse this URLs ( graph_json.php?rra_id=0&local_graph_id=840&graph_start=1556440072&graph_end=1556526472&graph_height=120&graph_width=500 ) directly i get the JSON RAW Data.

The aggregate graph works without any problems. But Cacti wants to show this graph two times. The first time (left) is shown. The second time it is not displayed. I can click the buttons (Graph Details, CSV Export, Time Graph View, Display Graphs from this Aggregate) of both graphs without any problems.

[Image Removed]

[netniV]

Oh, so my bad. This is a case of trying to display it more than once... But, if I now understand the issue right, that will likely cause issues because the div's ID that is used to inject the image already exists so it won't be updating both I imagine.

Somewhere, there needs to be a dedupe of the graphs being used, to ensure we don't have multiple entries.

[netniV]

Looking back up, that is the real issue as these are both for graph id 840.

[DavidLiedke]

I had only one aggregate graph in my system. And this problem only happends in the "device view".

[cigamit]

I think @netniV got it right. Likely an ID issue. We will move this fix to 1.2.5. Patch real soon though.

[pautiina]

I also have this bug. It's happening on 1.2.2 release

[cigamit]

This is resolved now. Moved to 1.2.4 milestone due to the delays.

[onnieBLT]

When will the new version be released?

[DavidLiedke]

Thx. Problem is fixed.

[netniV]

We are just trying to test/tweak the installer and once we are confident in that, it will be released. I was going to be this weekend, but I think it'll be Saturday now just to give us a bit more time.