Added optionsError output to give control over error handling in .set…

…Option() (#240) Co-authored-by: trajnisz <TomaszRajnisz@DESKTOP-7SFS8I0.com>
xieziyu · Jul 24, 2020 · 535bb6c · 535bb6c
1 parent 05f3e86
commit 535bb6c
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/projects/ngx-echarts/src/lib/ngx-echarts.directive.ts b/projects/ngx-echarts/src/lib/ngx-echarts.directive.ts
@@ -44,6 +44,7 @@ export class NgxEchartsDirective implements OnChanges, OnDestroy, OnInit, DoChec
 
   // ngx-echarts events
   @Output() chartInit = new EventEmitter<any>();
+  @Output() optionsError = new EventEmitter<Error>();
 
   // echarts mouse events
   @Output() chartClick = this.createLazyEvent('click');
@@ -166,7 +167,13 @@ export class NgxEchartsDirective implements OnChanges, OnDestroy, OnInit, DoChec
 
   private setOption(option: any, opts?: any) {
     if (this.chart) {
-      this.chart.setOption(option, opts);
+      try {
+        this.chart.setOption(option, opts);
+      }
+      catch (e){
+        console.error(e);
+        this.optionsError.emit(e);
+      }
     }
   }
 
@@ -206,7 +213,7 @@ export class NgxEchartsDirective implements OnChanges, OnDestroy, OnInit, DoChec
         this.chartInit.emit(this.chart);
       }
 
-      this.chart.setOption(this.options, true);
+      this.setOption(this.options, true);
     }
   }