Component ngOnDestroy check suggestion

[kenger]

Hi,

its possibile to change

  ngOnDestroy() {
    if (this.instance) {
      setTimeout(() => {
        this.instance.removeAllListeners();
        CKEDITOR.instances[this.instance.name].destroy();
        this.instance.destroy();
        this.instance = null;
      });
    }
  }

into

  ngOnDestroy() {
    if (this.instance) {
      setTimeout(() => {
        if (this.instance) {
          this.instance.removeAllListeners();
          CKEDITOR.instances[this.instance.name].destroy();
          this.instance.destroy();
          this.instance = null;
        }
      });
    }
  }

?

I'm destroying the instances myself in a wrapper component but now the code die because this.instance in the original component is null. Never check before a setTimeout without a recheck inside the setTimeout :)

[kzimny]

Your suggestion looks good. Thank you. Would you make a pull request?