The labels for a line graph aren't always useful, so I added support to use tokens as chart labels by passing the label through tokenize_value() at chart render.

With the patch applied, I added the field that had the correct corresponding labels to an existing chart field, and excluded it from display. Then, I used the token for the hidden field as the label for the chart field. I also sorted the view based on the hidden field's value.

CommentFileSizeAuthor
#1 chart_views-1431456.patch1.1 KBpaulgendek
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

paulgendek’s picture

Status: Active » Needs review
FileSize
1.1 KB
Pierre.Vriens’s picture

Issue summary: View changes
Parent issue: » #2371075: Chart 7.x-2.x Release

@paulgendek: sorry we're 2,5 years later, but still I want to give it a try to move forward with this issue. Any help you can provide to get its status updated to "tested by the community" would be appreciated. E.g. if you get back with something like "I'm using this already for quite some time in X production sites".

Anybody else against my proposal? If so what's the alternative to decide about the status of this issue?

To not forget about this issue, for now I'm adding the parent link ... which includes the details what such link means (at least to me, for now).

Pierre.Vriens’s picture

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). If it still gets tested "on time", it should be included I think. But "on time" now means "in the coming hours/days" ...