Fix mapToTargetIds #1220

da665bc7 · Masayuki Tanaka · e21a8f85 · da665bc7 · da665bc7 · da665bc7
Commit da665bc7 authored Aug 07, 2015 by Masayuki Tanaka
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

c3.js c3.js +1 -1

c3.min.js c3.min.js +0 -0

data.js src/data.js +1 -1

No files found.
--- a/c3.js
+++ b/c3.js
@@ -1789,7 +1789,7 @@
    };
    c3_chart_internal_fn.mapToTargetIds = function (ids) {
        var $$ = this;
-        return ids ? (isString(ids) ? [ids] : ids) : $$.mapToIds($$.data.targets);
+        return ids ? [].concat(ids) : $$.mapToIds($$.data.targets);
    };
    c3_chart_internal_fn.hasTarget = function (targets, id) {
        var ids = this.mapToIds(targets), i;

--- a/c3.min.js
+++ b/c3.min.js
--- a/src/data.js
+++ b/src/data.js
@@ -147,7 +147,7 @@ c3_chart_internal_fn.mapToIds = function (targets) {
 };
 c3_chart_internal_fn.mapToTargetIds = function (ids) {
    var $$ = this;
-    return ids ? (isString(ids) ? [ids] : ids) : $$.mapToIds($$.data.targets);
+    return ids ? [].concat(ids) : $$.mapToIds($$.data.targets);
 };
 c3_chart_internal_fn.hasTarget = function (targets, id) {
    var ids = this.mapToIds(targets), i;