Fixed issue with hidden pie slices affecting the positions of data labels. Closes #2510.

TorsteinHonsi · TorsteinHonsi · commit d5e176b5c01b · 2013-12-09T13:36:46.000+01:00
diff --git a/js/highcharts.src.js b/js/highcharts.src.js
@@ -16812,7 +16812,7 @@ var PieSeries = {
 
 		// arrange points for detection collision
 		each(data, function (point) {
-			if (point.dataLabel) { // it may have been cancelled in the base method (#407)
+			if (point.dataLabel && point.visible) { // #407, #2510
 				halves[point.half].push(point);
 			}
 		});
diff --git a/js/highstock.src.js b/js/highstock.src.js
@@ -16512,7 +16512,7 @@ var PieSeries = {
 
 		// arrange points for detection collision
 		each(data, function (point) {
-			if (point.dataLabel) { // it may have been cancelled in the base method (#407)
+			if (point.dataLabel && point.visible) { // #407, #2510
 				halves[point.half].push(point);
 			}
 		});
diff --git a/js/parts/PieSeries.js b/js/parts/PieSeries.js
@@ -482,7 +482,7 @@ var PieSeries = {
 
 		// arrange points for detection collision
 		each(data, function (point) {
-			if (point.dataLabel) { // it may have been cancelled in the base method (#407)
+			if (point.dataLabel && point.visible) { // #407, #2510
 				halves[point.half].push(point);
 			}
 		});

Original file line number	Diff line number	Diff line change
`@@ -16812,7 +16812,7 @@ var PieSeries = {`
`16812`	`16812`
`16813`	`16813`	`// arrange points for detection collision`
`16814`	`16814`	`each(data, function (point) {`
`16815`		`- if (point.dataLabel) { // it may have been cancelled in the base method (#407)`
	`16815`	`+ if (point.dataLabel && point.visible) { // #407, #2510`
`16816`	`16816`	`halves[point.half].push(point);`
`16817`	`16817`	`}`
`16818`	`16818`	`});`
Original file line number	Diff line number	Diff line change
`@@ -16512,7 +16512,7 @@ var PieSeries = {`
`16512`	`16512`
`16513`	`16513`	`// arrange points for detection collision`
`16514`	`16514`	`each(data, function (point) {`
`16515`		`- if (point.dataLabel) { // it may have been cancelled in the base method (#407)`
	`16515`	`+ if (point.dataLabel && point.visible) { // #407, #2510`
`16516`	`16516`	`halves[point.half].push(point);`
`16517`	`16517`	`}`
`16518`	`16518`	`});`
Original file line number	Diff line number	Diff line change
`@@ -482,7 +482,7 @@ var PieSeries = {`
`482`	`482`
`483`	`483`	`// arrange points for detection collision`
`484`	`484`	`each(data, function (point) {`
`485`		`- if (point.dataLabel) { // it may have been cancelled in the base method (#407)`
	`485`	`+ if (point.dataLabel && point.visible) { // #407, #2510`
`486`	`486`	`halves[point.half].push(point);`
`487`	`487`	`}`
`488`	`488`	`});`