Fix pie chart xaxis labels extraction for chartjs

Created on 11 February 2025, about 2 months ago

Problem/Motivation

I have a bar chart that displays correctly, using Chart.js.
When I switch the chart to pie, I'm getting a JS error:

Uncaught TypeError: n.reduce is not a function
    o plugin.legend.js:564
    <anonymous> plugin.legend.js:556
    _fitCols plugin.legend.js:205
    _fitCols plugin.legend.js:204
    fit plugin.legend.js:150
    update plugin.legend.js:89
    ss core.layouts.js:187
    update core.layouts.js:417
    _updateLayout core.controller.js:634
    update core.controller.js:528
    An core.controller.js:211
    attach charts_chartjs.js:39
    attach charts_chartjs.js:21
    attachBehaviors drupal.js:166
    attachBehaviors drupal.js:162
    <anonymous> drupal.init.js:32
    listener drupal.init.js:20

And it seems the same issue happened with HighCharts: #3239687: Fix pie chart xaxis labels extraction for highcharts →

I'll provide a fix similar to that issue.

🐛 Bug report

Status

Active

Version

5.1

Component

Chart.js integration

Created by

🇧🇪Belgium ludo.r Brussels

Live updates comments and jobs are added and updated live.

Merge Requests

!133Fix pie chart xaxis labels extraction for chartjs
Merged
🇺🇸United States andileco
updated about 1 month ago

Comments & Activities

Issue created by @ludo.r
Comment about 2 months ago →
🇧🇪Belgium ludo.r Brussels
And here's the patch that works for me, needs to be tested or validated as I don't have deep knowledge of the module.
First commit to issue fork.
Merge request !133Edit Chartjs.php → (Merged) created by andileco
Pipeline finished with Skipped
about 1 month ago
#442976
Comment about 1 month ago →
System Message

andileco → committed cab27230 on 5.1.x
Issue #3505816: Fix pie chart xaxis labels extraction for chartjs
Comment about 1 month ago →
🇺🇸United States andileco
Thank you @ludo.r - I made a couple adjustments to include donut as well. I tested with Chart Field, Charts API Example, and Views.
Comment 20 days ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024

Fix pie chart xaxis labels extraction for chartjs

Problem/Motivation

Merge Requests

!133Fix pie chart xaxis labels extraction for chartjsMerged

Comments & Activities

!133Fix pie chart xaxis labels extraction for chartjs
Merged