Check that a chart exists before calling destroy. about ember-highcharts

[ahmadsoe]

@jscadden how did you catch this issue?

[jscadden]

Good question. I have an odd case where my application caused a reload of the view before the component had finished rendering (The didInsertElement hadn't ran yet). So it called the willDestroyElement before a chart could be created. This issue may be a sign that something else is a problem with the flow of rendering in my application.

[jscadden]

I have forked the repo and made a couple changes to get around the issue for now. I am also working on upgrading to Ember 1.13. I believe the new component lifecycle hooks will help in my situation as well.

[20v100]

I'm having the same error. I'm on ember 1.13.3. Seem that destroy was ran before the component finished updating. This solved the issue:

_destroyChart: on("willDestroyElement", function () {
  this._super();
  if (!!this.get("chart")) {
    get(this, "chart").destroy();
  }
})

[ahmadsoe]

I have some application using this component on ember 1.13.3 too and its working fine without problems.

If willDestroyElement was ran before didInsertElement, then there must be something wrong with your application or ember itself.

[ahmadsoe]

@jscadden I just merged PR #29. Let me know if you had any news on this.