Opened 12 years ago

Closed 12 years ago

#4774 closed defect (fixed)

[CLA -- fix provided] dojox.charting.Chart2D -- cant "update" series by calling addSeries() with same name

Reported by: tk Owned by: Tom Trenka
Priority: high Milestone: 1.0
Component: Charting Version: 0.9
Keywords: charting chart2d Cc: Eugene Lazutkin
Blocked By: Blocking:

Description

I cant create a patch, but I've modified a local copy and tested this... it does work Basically I created a seriesCache object to check against before pushing a new series object into the series array... works great :D

		addSeries: function(name, data, kwArgs){
			var run = {name: name, data: data};
			if(kwArgs){ dojo.mixin(run, kwArgs); }
			if(typeof run.plot != "string"){ run.plot = "default"; }
			if(typeof(this.seriesCache[name])=='undefined'){
				this.series.push(run);
				this.seriesCache[name] = this.series.length-1;
			} else {
				this.series[this.seriesCache[name]] = run;
			}
			return this;
		},

Change History (5)

comment:1 Changed 12 years ago by tk

Owner: uhop deleted

comment:2 Changed 12 years ago by tk

Component: ChartingDojoX GFX

Just resetting to default owner of Charting...

comment:3 Changed 12 years ago by tk

Component: DojoX GFXCharting
Priority: normalhigh

or not... apparently it wont do it that way...

comment:4 Changed 12 years ago by tk

Cc: Eugene Lazutkin added
Owner: set to Tom Trenka

Just going to set to Tom and CC Eugene.

comment:5 Changed 12 years ago by Eugene Lazutkin

Resolution: fixed
Status: newclosed

(In [10928]) charting: added replacement facilities for plots and series, fixes #4774, thx tk.

Note: See TracTickets for help on using tickets.