From 9e55c4b1a151b434cba4b4425514c3aeac6f22d8 Mon Sep 17 00:00:00 2001
From: Noah Loomans <noahloomans@gmail.com>
Date: Wed, 27 Jun 2018 16:45:50 +0200
Subject: Only use history in makeSetUser and makeSetWeek

---
 src/client/react/components/container/Menu.js         |  4 ++--
 src/client/react/components/container/Results.js      |  4 ++--
 src/client/react/components/container/RoomFinder.js   |  4 ++--
 src/client/react/components/container/Search.js       |  4 ++--
 src/client/react/components/container/WeekSelector.js |  4 ++--
 src/client/react/lib/url.js                           | 10 +++++-----
 6 files changed, 15 insertions(+), 15 deletions(-)

(limited to 'src/client')

diff --git a/src/client/react/components/container/Menu.js b/src/client/react/components/container/Menu.js
index 5a18a8d..0e81fde 100644
--- a/src/client/react/components/container/Menu.js
+++ b/src/client/react/components/container/Menu.js
@@ -28,8 +28,8 @@ const mapStateToProps = (state, { match }) => ({
   user: userFromMatch(match),
 });
 
-const mapDispatchToProps = (dispatch, { location, history }) => ({
-  setUser: makeSetUser(location, history),
+const mapDispatchToProps = (dispatch, { history }) => ({
+  setUser: makeSetUser(history),
   showRoomFinder: () => dispatch({ type: 'ROOM_FINDER/SHOW' }),
 });
 
diff --git a/src/client/react/components/container/Results.js b/src/client/react/components/container/Results.js
index 773e469..c20fdc4 100644
--- a/src/client/react/components/container/Results.js
+++ b/src/client/react/components/container/Results.js
@@ -101,8 +101,8 @@ const mapStateToProps = (state, { match }) => ({
   selectedResult: state.search.result,
 });
 
-const mapDispatchToProps = (dispatch, { location, history }) => ({
-  setUser: makeSetUser(location, history),
+const mapDispatchToProps = (dispatch, { history }) => ({
+  setUser: makeSetUser(history),
   dispatch,
 });
 
diff --git a/src/client/react/components/container/RoomFinder.js b/src/client/react/components/container/RoomFinder.js
index c8c506b..8bcccc4 100644
--- a/src/client/react/components/container/RoomFinder.js
+++ b/src/client/react/components/container/RoomFinder.js
@@ -105,8 +105,8 @@ const mapStateToProps = (state, { match }) => ({
   isVisible: state.isRoomFinderVisible,
 });
 
-const mapDispatchToProps = (dispatch, { location, history }) => ({
-  setUser: makeSetUser(location, history),
+const mapDispatchToProps = (dispatch, { history }) => ({
+  setUser: makeSetUser(history),
   onHide: () => dispatch({ type: 'ROOM_FINDER/HIDE' }),
 });
 
diff --git a/src/client/react/components/container/Search.js b/src/client/react/components/container/Search.js
index fc741c6..865a0bc 100644
--- a/src/client/react/components/container/Search.js
+++ b/src/client/react/components/container/Search.js
@@ -189,8 +189,8 @@ const mapStateToProps = state => ({
   selectedResult: state.search.selected,
 });
 
-const mapDispatchToProps = (dispatch, { location, history }) => ({
-  setUser: makeSetUser(location, history),
+const mapDispatchToProps = (dispatch, { history }) => ({
+  setUser: makeSetUser(history),
   dispatch,
 });
 
diff --git a/src/client/react/components/container/WeekSelector.js b/src/client/react/components/container/WeekSelector.js
index f927117..bc428cc 100644
--- a/src/client/react/components/container/WeekSelector.js
+++ b/src/client/react/components/container/WeekSelector.js
@@ -85,8 +85,8 @@ const mapStateToProps = (state, { location }) => ({
   week: weekFromLocation(location),
 });
 
-const mapDispatchToProps = (dispatch, { location, history }) => ({
-  setWeek: makeSetWeek(location, history),
+const mapDispatchToProps = (dispatch, { history }) => ({
+  setWeek: makeSetWeek(history),
 });
 
 export default withRouter(connect(mapStateToProps, mapDispatchToProps)(WeekSelector));
diff --git a/src/client/react/lib/url.js b/src/client/react/lib/url.js
index c94383c..644fd74 100644
--- a/src/client/react/lib/url.js
+++ b/src/client/react/lib/url.js
@@ -43,20 +43,20 @@ export function weekFromLocation(location) {
   return purifyWeek(parseInt(weekStr, 10));
 }
 
-export function makeSetUser(location, history) {
+export function makeSetUser(history) {
   return (userId) => {
-    const query = location.search;
+    const query = history.location.search;
     history.push(`/${userId}${query}`);
   };
 }
 
-export function makeSetWeek(location, history) {
+export function makeSetWeek(history) {
   return (week) => {
     const query = queryString.stringify({
-      ...queryString.parse(location.search),
+      ...queryString.parse(history.location.search),
       week,
     });
 
-    history.push(`${location.pathname}?${query}`);
+    history.push(`${history.location.pathname}?${query}`);
   };
 }
-- 
cgit v1.1