only construct an element when getFirst does not, closes #879.

appnexus · Jan 22, 2018 · dfcaa0c · dfcaa0c
1 parent 3486f6e
commit dfcaa0c
Show file tree

Hide file tree

Showing 9 changed files with 23 additions and 37 deletions.
diff --git a/src/components/BarChart/BarChart.jsx b/src/components/BarChart/BarChart.jsx
@@ -335,11 +335,9 @@ const BarChart = createClass({
 			: yAxisFinalFormatter;
 
 		if (_.isEmpty(data) || width < 1 || height < 1 || isLoading) {
-			const emptyStateWrapper = getFirst(
-				this.props,
-				BarChart.EmptyStateWrapper,
-				<BarChart.EmptyStateWrapper Title="You have no data." />
-			);
+			const emptyStateWrapper =
+				getFirst(this.props, BarChart.EmptyStateWrapper) ||
+				<BarChart.EmptyStateWrapper Title="You have no data." />;
 
 			return (
 				<EmptyStateWrapper

diff --git a/src/components/DataTable/DataTable.jsx b/src/components/DataTable/DataTable.jsx
@@ -226,14 +226,12 @@ const DataTable = createClass({
 				childComponentElement.type === DataTable.ColumnGroup
 		);
 
-		const emptyStateWrapper = getFirst(
-			this.props,
-			DataTable.EmptyStateWrapper,
+		const emptyStateWrapper =
+			getFirst(this.props, DataTable.EmptyStateWrapper) ||
 			<DataTable.EmptyStateWrapper
 				Title="No items found."
 				Body="Try creating a new object or removing a filter."
-			/>
-		);
+			/>;
 
 		const fillerRowCount = _.clamp(minRows - _.size(data), 0, Infinity);
 

diff --git a/src/components/DateSelect/DateSelect.jsx b/src/components/DateSelect/DateSelect.jsx
@@ -306,11 +306,9 @@ const DateSelect = createClass({
 		const { cursor } = this.state;
 		const isRangeSameDay = DateUtils.isSameDay(from, to);
 
-		const calendarMonth = getFirst(
-			this.props,
-			DateSelect.CalendarMonth,
-			<DateSelect.CalendarMonth />
-		);
+		const calendarMonth =
+			getFirst(this.props, DateSelect.CalendarMonth) ||
+			<DateSelect.CalendarMonth />;
 		const monthsShown = clampMonthsShown(monthsShownRaw);
 
 		/* istanbul ignore next */

diff --git a/src/components/InfiniteSlidePanel/InfiniteSlidePanel.jsx b/src/components/InfiniteSlidePanel/InfiniteSlidePanel.jsx
@@ -91,11 +91,9 @@ const InfiniteSlidePanel = createClass({
 			...passThroughs
 		} = this.props;
 
-		const slide = getFirst(
-			this.props,
-			InfiniteSlidePanel.Slide,
-			<InfiniteSlidePanel.Slide>{children}</InfiniteSlidePanel.Slide>
-		);
+		const slide =
+			getFirst(this.props, InfiniteSlidePanel.Slide) ||
+			<InfiniteSlidePanel.Slide>{children}</InfiniteSlidePanel.Slide>;
 		const slideChildRenderFunction = slide.props.children;
 		if (!_.isFunction(slideChildRenderFunction)) {
 			throw new Error(

diff --git a/src/components/LineChart/LineChart.jsx b/src/components/LineChart/LineChart.jsx
@@ -500,11 +500,9 @@ const LineChart = createClass({
 		const xPoints = _.map(_.keys(xPointMap), _.toNumber);
 
 		if (_.isEmpty(data) || width < 1 || height < 1 || isLoading) {
-			const emptyStateWrapper = getFirst(
-				this.props,
-				LineChart.EmptyStateWrapper,
-				<LineChart.EmptyStateWrapper Title="You have no data." />
-			);
+			const emptyStateWrapper =
+				getFirst(this.props, LineChart.EmptyStateWrapper) ||
+				<LineChart.EmptyStateWrapper Title="You have no data." />;
 
 			return (
 				<EmptyStateWrapper

diff --git a/src/components/LoadingIndicator/LoadingIndicator.jsx b/src/components/LoadingIndicator/LoadingIndicator.jsx
@@ -58,7 +58,8 @@ const LoadingIndicator = createClass({
 
 		const { LoadingMessage } = LoadingIndicator;
 
-		const messageElement = getFirst(props, LoadingMessage, <LoadingMessage />);
+		const messageElement =
+			getFirst(props, LoadingMessage) || <LoadingMessage />;
 		const otherChildren = rejectTypes(children, LoadingMessage);
 
 		return (

diff --git a/src/components/SearchField/SearchField.jsx b/src/components/SearchField/SearchField.jsx
@@ -135,11 +135,8 @@ const SearchField = createClass({
 			value,
 		};
 
-		const textFieldElement = getFirst(
-			props,
-			TextField,
-			<TextField {...textFieldProps} />
-		);
+		const textFieldElement =
+			getFirst(props, TextField) || <TextField {...textFieldProps} />;
 		const isIconActive = _.isUndefined(isValid)
 			? !_.isEmpty(_.get(textFieldElement, 'props.value'))
 			: isValid;

diff --git a/src/components/SearchableMultiSelect/SearchableMultiSelect.jsx b/src/components/SearchableMultiSelect/SearchableMultiSelect.jsx
@@ -470,13 +470,11 @@ const SearchableMultiSelect = createClass({
 			'props',
 			{}
 		);
-		const selectionLabel = getFirst(
-			props,
-			SearchableMultiSelect.SelectionLabel,
+		const selectionLabel =
+			getFirst(props, SearchableMultiSelect.SelectionLabel) ||
 			<SearchableMultiSelect.SelectionLabel>
 				Selected
-			</SearchableMultiSelect.SelectionLabel>
-		);
+			</SearchableMultiSelect.SelectionLabel>;
 		const isSmall = responsiveMode === 'small';
 
 		return (

diff --git a/src/components/SearchableSelect/SearchableSelect.jsx b/src/components/SearchableSelect/SearchableSelect.jsx
@@ -321,7 +321,7 @@ const SearchableSelect = createClass({
 		const { flattenedOptionsData } = this.state;
 
 		const searchFieldProps = _.get(
-			getFirst(props, SearchField, <SearchField />),
+			getFirst(props, SearchField) || <SearchField />,
 			'props'
 		);
 		const placeholderProps = _.first(