Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I notice in firebug that the views_chart.js file is not added to aggregation.
Comments
Comment #1
Georgii CreditAttribution: Georgii commentedMoreover, it is added to the HTML HEAD section even if no chart is placed there. I.e. it is added to EVERY page (even to 404 error page) which looks like incorrect.
Are there any comments on this or plans to fix?
Comment #2
Georgii CreditAttribution: Georgii commentedAs for aggregation, the reason should be here
The last FALSE in drupal_add_js turns aggregation off, bun unclear why. Was it done on purpose?
And the question about having views_chart.js on every page is still actual.
Comment #3
Leon Kessler CreditAttribution: Leon Kessler commentedCan we get this fixed please? With something like this patch.
We're only using this module for a handful of admin pages, and now we have the js added un-aggregated onto every page request.
There was obviously a reason why the developer choose not to include this on js preprocessing. Although the fact that it's included on every page makes me think it was just for debugging purposes only.
Thanks
Leon
Comment #4
Leon Kessler CreditAttribution: Leon Kessler commentedComment #5
Leon Kessler CreditAttribution: Leon Kessler commentedUrgh, d/o not accepting the patch uploaded. Try once more...