diff --git a/src/client/components/facet_results/ResultTable.js b/src/client/components/facet_results/ResultTable.js index a8c546696a256d97ca828440dc973b37a266edc3..9df3ebffb96ee0a4b4af89be47d330b98db0d388 100644 --- a/src/client/components/facet_results/ResultTable.js +++ b/src/client/components/facet_results/ResultTable.js @@ -173,22 +173,24 @@ class ResultTable extends React.Component { const expanded = this.state.expandedRows.has(row.id); let hasExpandableContent = false; const dataCells = this.props.data.tableColumns.map(column => { - const columnData = row[column.id]; + const columnData = row[column.id] == null ? '-' : row[column.id]; const isArray = Array.isArray(columnData); if (isArray) { hasExpandableContent = true; } if (!isArray + && columnData !== '-' && column.valueType === 'string' && column.collapsedMaxWords - && columnData.split(' ').length > column.collapsedMaxWords) { + && columnData.split(' ').length > column.collapsedMaxWords + ) { hasExpandableContent = true; } return ( <ResultTableCell key={column.id} columnId={column.id} - data={row[column.id] == null ? '-' : row[column.id]} + data={columnData} valueType={column.valueType} makeLink={column.makeLink} externalLink={column.externalLink} diff --git a/src/client/components/facet_results/StringList.js b/src/client/components/facet_results/StringList.js index 7b5b325f3bf0771ca0be197e1e8fccbe789496cf..9e0e98088982d91719854b8586bebb3d94c5442e 100644 --- a/src/client/components/facet_results/StringList.js +++ b/src/client/components/facet_results/StringList.js @@ -31,9 +31,9 @@ const StringList = props => { const createFirstValue = (data, isArray) => { let firstValue = isArray ? data[0] : data; if (props.collapsedMaxWords) { - const wordCount = data.split(' ').length; + const wordCount = firstValue.split(' ').length; if (wordCount > props.collapsedMaxWords) { - firstValue = data.trim().split(' ').splice(0, props.collapsedMaxWords).join(' '); + firstValue = firstValue.trim().split(' ').splice(0, props.collapsedMaxWords).join(' '); firstValue = `${firstValue}...`; } } else if (isArray) {