fix(options): Fix handling for falsy data.types

Check when falsy and non-valid chart type value is specified.

Fix #3125

src/ChartInternal/data/convert.ts

@@ -76,6 +76,7 @@ export default {
 	convertDataToTargets(data: {[key:string]: number|null}[], appendXs: boolean): IData[] {
 		const $$ = this;
 		const {axis, config, state} = $$;
+		const chartType = config.data_type;
 		let isCategorized = false;
 		let isTimeSeries = false;
 		let isCustomX = false;
@@ -207,9 +208,11 @@ export default {
 		state.hasPositiveValue = $$.hasPositiveValueInTargets(targets);
 
 		// set target types
-		if (config.data_type) {
-			$$.setTargetType($$.mapToIds(targets)
-				.filter(id => !(id in config.data_types)), config.data_type);
+		if (chartType && $$.isValidChartType(chartType)) {
+			const targetIds = $$.mapToIds(targets)
+				.filter(id => !(id in config.data_types) || !$$.isValidChartType(config.data_types[id]));
+
+			$$.setTargetType(targetIds, chartType);
 		}
 
 		// cache as original id keyed

src/ChartInternal/internals/type.ts

@@ -6,6 +6,16 @@ import {isArray, isNumber, isString} from "../../module/util";
 import {TYPE, TYPE_BY_CATEGORY} from "../../config/const";
 
 export default {
+	/**
+	 * Check if the given chart type is valid
+	 * @param {string} type Chart type string
+	 * @returns {boolean}
+	 * @private
+	 */
+	isValidChartType(type: string): boolean {
+		return !!(type && Object.values(TYPE).indexOf(type) > -1);
+	},
+
 	setTargetType(targetIds: string[], type: string): void {
 		const $$ = this;
 		const {config, state: {withoutFadeIn}} = $$;

test/internals/type-spec.ts

@@ -15,6 +15,41 @@ describe("TYPES", () => {
 		chart = util.generate(args);
 	});
 
+	describe("data.type / data.types", () => {
+		before(() => {
+			args = {
+				data: {
+					type: "bars",
+					columns: [
+						["data1", 30, 20, 50, 40, 60, 50],
+						["data2", 200, 130, 90, 240, 130, 220],
+					]
+				}
+			};
+		});
+
+		it("should generate when wrong data.type is specified.", () => {
+			const lines = chart.$.line.lines.size();
+
+			expect(lines).to.be.equal(2);
+		});
+
+		it("set option: data.types", () => {
+			args.data.type = "bar";
+			args.data.types = {
+				data2: undefined
+			};
+		});
+
+		it("should generate when falsy data.types value is specified.", () => {
+			const {bar: {bars}, line: {lines}} = chart.$;
+
+			expect(lines).to.be.null;
+			expect(bars.size()).to.be.equal(12);
+		});
+	});
+
+
 	describe("internal.hasArcType", () => {
 		describe("with data", () => {
 			before(() => {