The Global "Highcharts.charts" JS array seems to be still populated with old charts after view ajax refresh adds new chart.
This might be result in a memory leak.

Consider destroying old chrats object on Drupal deattach.

Couldn't find where/if google charts has same issue.

Patch upcoming.

CommentFileSizeAuthor
#2 old_highcharts_js_object_not_detroyed_on_views_ajax-2942355.patch660 bytessilverham
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

silverham created an issue. See original summary.

silverham’s picture

silverham CreditAttribution: silverham at EY Digital commented
FileSize
old_highcharts_js_object_not_detroyed_on_views_ajax-2942355.patch660 bytes

Patched attached.

silverham’s picture

silverham CreditAttribution: silverham at EY Digital commented
Status: Active » Needs review
joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented

a very smart improvement, thanks!

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Status: Needs review » Reviewed & tested by the community

  • andileco committed 8499a6d on 7.x-2.x 
    Issue #2942355 by silverham, joseph.olstad, nikathone, andileco: Old...
andileco’s picture

andileco CreditAttribution: andileco at JSI Research & Training Institute, Inc. (JSI) commented
Version: 7.x-2.x-dev » 8.x-4.x-dev
Status: Reviewed & tested by the community » Needs work

Thanks for the patch. Moving to 8.x-4.x.

  • andileco committed 234615d on 8.x-4.x authored by silverham 
    Issue #2942355 by silverham, joseph.olstad, andileco, nikathone: Old...
andileco’s picture

andileco CreditAttribution: andileco at JSI Research & Training Institute, Inc. (JSI) commented
Status: Needs work » Fixed
joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented

Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.