Merge pull request #745 from finos/phosphor_bugfix

Fix empty list in phosphor plugin
finos · Sep 29, 2019 · de59d3e · de59d3e
2 parents a5fd196 + 04c6056
commit de59d3e
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/packages/perspective-phosphor/src/ts/widget.ts b/packages/perspective-phosphor/src/ts/widget.ts
@@ -310,11 +310,15 @@ export
                 this.pspNode.load(this.schema as Schema, options as TableOptions);
             }
 
-            if (Array.isArray(data) && data.length > 0) {
-                if (this._wrap) {
-                    this.pspNode.update([data]);
+            if (Array.isArray(data)) {
+                if (data.length > 0) {
+                    if (this._wrap) {
+                        this.pspNode.update([data]);
+                    } else {
+                        this.pspNode.update(data);
+                    }
                 } else {
-                    this.pspNode.update(data);
+                    console.warn('Perspective received length 0 data');
                 }
             } else {
                 if(Object.keys(data)){