Skip to content

Commit dd2dfa9

Browse files
rueckstiesskangas
authored andcommitted
fixed bug that assumes arrayFields always present
1 parent 8f951e8 commit dd2dfa9

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

src/minicharts/array-root.jade

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,9 @@ div(data-hook='viz-container')
66
ul.list-inline
77
each name, idx in fieldNames
88
li= name + (idx < fieldNames.length-1 ? ',' : '')
9-
dt array lengths
10-
dd
11-
ul.list-inline
12-
li min: #{min_length},
13-
li average: #{average_length},
14-
li max: #{max_length}
9+
dt array lengths
10+
dd
11+
ul.list-inline
12+
li min: #{min_length},
13+
li average: #{average_length},
14+
li max: #{max_length}

src/minicharts/array-root.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ module.exports = VizView.extend({
1212
min_length: _.min(this.model.lengths),
1313
max_length: _.max(this.model.lengths)
1414
};
15-
if (this.model.parent.arrayFields.length > 0) {
15+
if (this.model.parent.arrayFields) {
1616
parsed.fieldNames = this.model.parent.arrayFields.pluck('name');
1717
parsed.fieldsPluralized = pluralize('nested field', parsed.fieldNames.length, true);
1818
}

0 commit comments

Comments
 (0)