This is a combined patch for the issue queues
#1250720: Line Markers and Circle Markers integration for making Composite Charts through Views
#1206306: Setting Chart Axes, Gridlines and Axis Labels in Chart using Views Plugin
#1203716: Data Value Markers in Chart Views
#1202740: Appending Legends to Chart using Chart Views
It combines all the features of Axes, Gridlines, Markers and Legends for the Chart Views Integration.
A Demo can be found at http://maulik-kamdar.com/testbed/drupal/country

This patch can only be applied to a new temporary branch named 7.x-1.x-mk.
To clone:
git clone --branch 7.x-1.x-mk http://git.drupal.org/project/chart.git

CommentFileSizeAuthor
chart-views.patch27.06 KBmaulik.kamdar
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

13rac1’s picture

13rac1 CreditAttribution: 13rac1 commented

Applied this patch to 7.x-1.x-mk and automatically/manually merged in new commits from 7.x-1.x. Please test.

Ranko’s picture

Ranko CreditAttribution: Ranko commented

I did a few quick tests.

#1250720: Line Markers and Circle Markers integration for making Composite Charts through Views - whenever I set the line markers they appear not to be saved (on editing the settings nothing is selected). It does work though. But I do not see any markers on the chart.

#1206306: Setting Chart Axes, Gridlines and Axis Labels in Chart using Views Plugin - results are strange. Setting the X works, but nothing is displayed in the range labels (using custom field labels). Setting the Y works (also using custom field labels), but the labels are shown (although not the values, rather the nid is shown as seen is #1256050: Charts showing nid as value instead of value)

#1203716: Data Value Markers in Chart Views - didn't work in line and scatter chart, actually made the line disappear; worked properly on bar and radar.

#1202740: Appending Legends to Chart using Chart Views - works as advertised.

To be on the same page you can take a look at a quick stage I have built up. The chart should be showing Date field values o, Integer field values on Y, grouped on a text list field type excluded from the display; all for a custom content type. Demo is here.

Ranko’s picture

Ranko CreditAttribution: Ranko commented

My initial report might be wrong. I am not sure why but now I am getting different results. By tomorrow I'll nuke my test environment and redo the testing on as clean as it gets install to confirm.

maulik.kamdar’s picture

maulik.kamdar CreditAttribution: maulik.kamdar commented

Hello Ranko,

So did this patch function after you had decided to test it again in a clean environment.

#1206306: Setting Chart Axes, Gridlines and Axis Labels in Chart using Views Plugin - There might be a problem related to setting the bottom X-Axis labels. Somehow they are not being displayed. (Maybe an error in Google Chart API ??)

And the values are not definitely not saved in the Chart Settings form for the Select Lists.
I will try to look into this.

I have not checked the remaining errors, but the data markers did appear for the Line Charts when I had tested it.

Thanks.

Pierre.Vriens’s picture

Pierre.Vriens CreditAttribution: Pierre.Vriens commented
Issue summary: View changes
Parent issue: » #2371075: Chart 7.x-2.x Release
Pierre.Vriens’s picture

Pierre.Vriens CreditAttribution: Pierre.Vriens commented

Yes I realise we're over 3 years later now ... But still I've added a parent link in this issue and the 4 other issues mentioned here. And I changed 2 of those issue to status = needs review.

The demo link cause this error however:

Error messagePDOException: SQLSTATE[42000] [1044] Access denied for user 'maulikk1_root'@'localhost' to database 'maulikk1_drupalchart' in lock_may_be_available() (line 167 of /home/maulikk1/public_html/testbed/drupal/includes/lock.inc).

It would help the move forward with these 5 issues (to status fixed) if the above error can be resolved, or some other link is provided as a replacement for the link causing the problems.

Better still: anybody out there who can confirm that the status of one of more of these issues should be considered as "tested by community"? If so please update the status of such issues accordingly.

Pierre.Vriens’s picture

Pierre.Vriens CreditAttribution: Pierre.Vriens commented

Unless this issue becomes "tested by community" ASAP, it won't make it into the 7.x-1.2 Release (to not put that release at risk). same for the related issues. If it still gets tested "on time", it should be included I think. But "on time" now means "in the coming hours/days" ...

Also, what is the current status of the -mk branch? Why was it actually needed?