Merge pull request #857 from sharetribe/topbar-search-value

Fix Topbar search initial value

src/components/SectionHero/SectionHero.js

@@ -7,6 +7,7 @@ import { createResourceLocatorString } from '../../util/routes';
 import routeConfiguration from '../../routeConfiguration';
 import { IconSearch, Button } from '../../components';
 import { LocationSearchForm } from '../../forms';
+import config from '../../config';
 
 import css from './SectionHero.css';
 
@@ -22,7 +23,8 @@ const SectionHero = props => {
   const handleSearchSubmit = values => {
     const { search, selectedPlace } = values.location;
     const { origin, bounds, country } = selectedPlace;
-    const searchParams = { address: search, origin, bounds, country };
+    const originMaybe = config.sortSearchByDistance ? { origin } : {};
+    const searchParams = { ...originMaybe, address: search, bounds, country };
     history.push(createResourceLocatorString('SearchPage', routeConfiguration(), {}, searchParams));
   };
 

src/components/Topbar/Topbar.js

@@ -174,7 +174,9 @@ class TopbarComponent extends Component {
     );
 
     // Only render current search if full place object is available in the URL params
-    const locationFieldsPresent = address && origin && bounds && country;
+    const locationFieldsPresent = config.sortSearchByDistance
+      ? address && origin && bounds && country
+      : address && bounds && country;
     const initialSearchFormValues = {
       location: locationFieldsPresent
         ? {

src/util/types.js

@@ -70,7 +70,7 @@ propTypes.route = shape({
 // Place object from LocationAutocompleteInput
 propTypes.place = shape({
   address: string.isRequired,
-  origin: propTypes.latlng.isRequired,
+  origin: propTypes.latlng,
   bounds: propTypes.latlngBounds, // optional viewport bounds
   country: string, // country code, e.g. FI, US
 });