From 8effaad2307af3c112a69f453b23e78ae4024e2d Mon Sep 17 00:00:00 2001
From: Julien Castelain <julien.castelain@liferay.com>
Date: Mon, 8 Jan 2018 17:30:08 +0100
Subject: [PATCH] fix(colors): Fix color method

Only charts with background should use tiles

Close #233
---
 spec/color-spec.js     | 49 ++++++++++++++++++++++++++++++++++++++++++
 src/internals/color.js | 12 ++++++++---
 2 files changed, 58 insertions(+), 3 deletions(-)

diff --git a/spec/color-spec.js b/spec/color-spec.js
index 52b22f6d7..27f8c507f 100644
--- a/spec/color-spec.js
+++ b/spec/color-spec.js
@@ -131,5 +131,54 @@ describe("COLOR", () => {
 				});
 			});
 		});
+
+		describe("pattern shouldn't be applying for line types", () => {
+			const checkStroke = (path=[false, false]) => {
+				const internal = chart.internal;
+				const rx = /#bb-\d+-pattern-/;
+
+				chart.data().forEach(v => {
+					const id = v.id;
+					const isLine = internal.isTypeOf(id, ["line", "spline", "step"]) || !internal.config.data_types[id];
+					const stroke = internal.main.select(`.bb-shapes-${id} path`).style("stroke");
+
+					expect(rx.test(stroke)).to.be[!isLine];
+				})
+			};
+
+			it("set options data.type=undefined", () => {
+				args.data.type = undefined;
+			});
+
+			it("check for stroke", checkStroke);
+
+			it("set options data.type=line", () => {
+				args.data.type = "line";
+			});
+
+			it("check for stroke", checkStroke);
+
+			it("set options data.type=spline", () => {
+				args.data.type = "spline";
+			});
+
+			it("check for stroke", checkStroke);
+
+			it("set options data.type=step", () => {
+				args.data.type = "step";
+			});
+
+			it("check for stroke", checkStroke);
+
+			it("set options data.type", () => {
+				args.data.type = undefined;
+				args.data.types = {
+					data1: "bar",
+					data2: "line"
+				};
+			});
+
+			it("check for stroke", checkStroke);
+		});
 	});
 });
diff --git a/src/internals/color.js b/src/internals/color.js
index 5be1fa1ad..fabd5847a 100644
--- a/src/internals/color.js
+++ b/src/internals/color.js
@@ -44,6 +44,8 @@ extend(ChartInternal.prototype, {
 		const ids = [];
 		let pattern = notEmpty(config.color_pattern) ?
 			config.color_pattern : d3ScaleOrdinal(d3SchemeCategory10).range();
+		const originalColorPattern = pattern;
+
 
 		if (isFunction(config.color_tiles)) {
 			const tiles = config.color_tiles();
@@ -62,7 +64,8 @@ extend(ChartInternal.prototype, {
 
 		return function(d) {
 			const id = d.id || (d.data && d.data.id) || d;
-			const isLine = $$.isTypeOf(id, "line");
+			const isLine = $$.isTypeOf(id, ["line", "spline", "step"]) || !$$.config.data_types[id];
+
 			let color;
 
 
@@ -75,9 +78,12 @@ extend(ChartInternal.prototype, {
 				color = colors[id];
 
 			// if not specified, choose from pattern
-			} else if (!isLine) {
+			} else {
 				if (ids.indexOf(id) < 0) { ids.push(id); }
-				color = pattern[ids.indexOf(id) % pattern.length];
+
+				color = isLine ? originalColorPattern[ids.indexOf(id) % originalColorPattern.length] :
+					pattern[ids.indexOf(id) % pattern.length];
+
 				colors[id] = color;
 			}