Needs work
Project:
Highcharts
Version:
7.x-2.x-dev
Component:
Views integration
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Anonymous (not verified)
Created:
10 May 2013 at 20:50 UTC
Updated:
4 Feb 2021 at 17:28 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
SteveK commentedgetting the same issue. Not sure where to start with configuring this module.
Comment #2
Anonymous (not verified) commentedThis appears to be preventing Highcharts from displaying any data in all chart types (i.e., from working at all).
Comment #3
scottrigby@rhclayto and @SteveK - Views integration has yet to be finished (the project page clarifies this under Versions > 2.x Views integration).
The status is, 7.x-2.x highcharts.module (API) is very stable. However 7.x-2.x highcharts_views.module (Views integration) is still in development toward an initial release.
Thanks for reporting this notice (it's a trivial fix), but this notice is not causing your chart not displaying field content.
I can tell you that the charts do display data properly in the dev branch currently, but it must be done differently depending on your use case. For an explanation of where Views UI configurations for various use cases are at see point 4 in comment 3 on #1674928-3: Allow full use of Highcharts library Options in Views. This is something we are identifying - and when generally useful cases are identified - if we can find a good way to abstract them, these can be added to the the
settingsForm()for each template class. So this is to say, if you can give feedback on your use case(s) in that issue, it could help get your needs into the appropriate chart template settings form :)A note about expectations: The dev branch is for people who want to help test, but I would not use highcharts_views.module for anything other than that for now. You probably want to follow #1674928: Allow full use of Highcharts library Options in Views to stay up to date with progress.
Regarding where to start with configuring highcharts_views.module, you probably want to follow #1992670: Document Highcharts Views integration, which is postponed until #1674928 is ready.
Comment #4
scottrigbyDevelopment note for tracking down this notice:
The
ViewsHighchartsOptionsextension classes should currently only be called byhighcharts_views_plugin_style_highcharts::loadChartTemplate, which is only called byhighcharts_views_plugin_style_highcharts::render. By that time, the settingsForm for the selected chart template should have been saved byhighcharts_views_plugin_style_highcharts::options_formviagetChartTemplateSettingsFormto$view->style_plugin->options['format'][TEMPLATE_CLASS].ViewsHighchartsOptions::settingsFormis also an abstract (required) method, so even if a template'ssettingsFormis empty, the index in that array should be defined during__construct().This was all a kind of gymnastics in order to get dynamically loaded settings per template. Will look into the notice more to see why it might be showing up.
Comment #5
scottrigbyComment #6
scottrigbyActually this is also called when initially retrieving each template's settings form in
highcharts_views_plugin_style_highcharts::getChartTemplateSettingsForm. This should probably be refactored. Something like addingViewsHighchartsOptions::setFormDefaultsrather than doing this on__construct().Comment #7
scottrigbyOut of curiosity, does this solve your problem? I can make the patch but can't test this weekend.
Also, if you can list steps you took to get this notice that would be helpful.
Comment #8
scottrigbyNope. Let me think more on this one.
Comment #9
scottrigbyComment #10
MickC commentedWould be great to see this working - I just tried Charts and Visualisation, but only basic usage of Highcharts.
This module has the potential to specialise and get more flexibility and charting options.
What's the outlook on views integration issues?