After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 779411 - jqplot code leaks to other reports in multi-column report
jqplot code leaks to other reports in multi-column report
Status: RESOLVED FIXED
Product: GnuCash
Classification: Other
Component: Reports
git-maint
Other All
: Normal normal
: ---
Assigned To: gnucash-reports-maint
gnucash-reports-maint
Depends on:
Blocks:
 
 
Reported: 2017-03-01 11:44 UTC by Geert Janssens
Modified: 2018-06-29 23:54 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Piechart changes and syntax change (1.99 KB, patch)
2017-03-11 11:53 UTC, Bob
committed Details | Review

Description Geert Janssens 2017-03-01 11:44:55 UTC
Noticed this today by chance:
I've set up a multi-column report with 2 charts: a bar chart and a pie chart.
Hovering over the bar chart will pop up a tool tip with more detail on the bar being hovered over.
When hovering the pie chart this doesn't happen (which shouldn't as it's not implemented). However the trace fill will now receive lots of lines like this:
WARN <gnc.html> [webkit_console_msg_cb()] JS: file:////kobaltnet/janssege-development/installs/kdevelop/gnucash/master/share/gnucash/jqplot/jqplot.cursor.js (488): TypeError: undefined is not an object (evaluating 'plot.axes[g[j]]._ticks[0].formatter')

and

WARN <gnc.html> [webkit_console_msg_cb()] JS: file:////kobaltnet/janssege-development/installs/kdevelop/gnucash/master/share/gnucash/jqplot/jqplot.highlighter.js (240): TypeError: undefined is not an object (evaluating 'series._xaxis._ticks[0].formatter')

Opening the report also started with lines like:
WARN <gnc.html> [webkit_console_msg_cb()] JS: file:////kobaltnet/janssege-development/installs/kdevelop/gnucash/master/share/gnucash/jqplot/ (1): SyntaxError: Unexpected token '<'
However I guess this is a separate issue. Just mentioning it for completeness.

Note none of this happens when the same piechart is opened as an independent report (that is not as part of a multi-column report).
Comment 1 Bob 2017-03-11 11:53:16 UTC
Created attachment 347699 [details] [review]
Piechart changes and syntax change

This patch should fix the errors you are seeing by setting a couple of pie chart options to false and changing the way the extra js lines are removed.

Bob
Comment 2 Geert Janssens 2017-03-16 20:31:53 UTC
Lovely. Thanks for fixing this !

The fix will be released in the next stable gnucash version at the end of this month.
Comment 3 John Ralls 2018-06-29 23:54:52 UTC
GnuCash bug tracking has moved to a new Bugzilla host. This bug has been copied to https://bugs.gnucash.org/show_bug.cgi?id=779411. Please update any external references or bookmarks.