From e94daacc1d9fb3870a74256de3b94be0dbaf05a1 Mon Sep 17 00:00:00 2001
From: Georgios Andreadis <info@gandreadis.com>
Date: Mon, 11 May 2020 14:41:44 +0200
Subject: Upgrade and fix regressions

---
 src/components/app/map/MapStageComponent.js        | 26 +++++++++++++---------
 .../app/sidebars/topology/machine/UnitComponent.js |  8 +++----
 src/components/modals/Modal.js                     |  4 ++--
 src/containers/auth/Login.js                       |  9 ++++----
 src/sagas/experiments.js                           |  3 +--
 src/store/configure-store.js                       |  7 ++++--
 6 files changed, 31 insertions(+), 26 deletions(-)

(limited to 'src')

diff --git a/src/components/app/map/MapStageComponent.js b/src/components/app/map/MapStageComponent.js
index ce6a60f6..4e4972f4 100644
--- a/src/components/app/map/MapStageComponent.js
+++ b/src/components/app/map/MapStageComponent.js
@@ -7,6 +7,8 @@ import RoomHoverLayer from "../../../containers/app/map/layers/RoomHoverLayer";
 import jQuery from "../../../util/jquery";
 import { NAVBAR_HEIGHT } from "../../navigation/Navbar";
 import { MAP_MOVE_PIXELS_PER_EVENT } from "./MapConstants";
+import {Provider} from "react-redux";
+import {store} from "../../../store/configure-store";
 
 class MapStageComponent extends React.Component {
   state = {
@@ -14,8 +16,8 @@ class MapStageComponent extends React.Component {
     mouseY: 0
   };
 
-  constructor() {
-    super();
+  constructor(props) {
+    super(props);
 
     this.updateDimensions = this.updateDimensions.bind(this);
     this.updateScale = this.updateScale.bind(this);
@@ -104,15 +106,17 @@ class MapStageComponent extends React.Component {
           height={this.props.mapDimensions.height}
           onMouseMove={this.updateMousePosition.bind(this)}
         >
-          <MapLayer />
-          <RoomHoverLayer
-            mouseX={this.state.mouseX}
-            mouseY={this.state.mouseY}
-          />
-          <ObjectHoverLayer
-            mouseX={this.state.mouseX}
-            mouseY={this.state.mouseY}
-          />
+          <Provider store={store}>
+            <MapLayer />
+            <RoomHoverLayer
+              mouseX={this.state.mouseX}
+              mouseY={this.state.mouseY}
+            />
+            <ObjectHoverLayer
+              mouseX={this.state.mouseX}
+              mouseY={this.state.mouseY}
+            />
+          </Provider>
         </Stage>
       </Shortcuts>
     );
diff --git a/src/components/app/sidebars/topology/machine/UnitComponent.js b/src/components/app/sidebars/topology/machine/UnitComponent.js
index 10a310e8..7c27043d 100644
--- a/src/components/app/sidebars/topology/machine/UnitComponent.js
+++ b/src/components/app/sidebars/topology/machine/UnitComponent.js
@@ -49,18 +49,16 @@ class UnitComponent extends React.Component {
             this.props.unit.generation}
         </span>
         <span>
-          <a
+          <span
             tabIndex="0"
-            className="unit-info-popover btn btn-outline-info mr-1"
+            className="unit-info-popover btn btn-outline-info mr-1 fa fa-info-circle"
             role="button"
             data-toggle="popover"
             data-trigger="focus"
             title="Unit information"
             data-content={unitInfo}
             data-html="true"
-          >
-            <span className="fa fa-info-circle" />
-          </a>
+          />
           {this.props.inSimulation ? (
             undefined
           ) : (
diff --git a/src/components/modals/Modal.js b/src/components/modals/Modal.js
index 367cf2da..19337db8 100644
--- a/src/components/modals/Modal.js
+++ b/src/components/modals/Modal.js
@@ -22,8 +22,8 @@ class Modal extends React.Component {
   // 'close' triggers)
   visible = false;
 
-  constructor() {
-    super();
+  constructor(props) {
+    super(props);
     this.id = "modal-" + Modal.idCounter++;
   }
 
diff --git a/src/containers/auth/Login.js b/src/containers/auth/Login.js
index 28e95963..4afb393c 100644
--- a/src/containers/auth/Login.js
+++ b/src/containers/auth/Login.js
@@ -31,11 +31,12 @@ class LoginContainer extends React.Component {
         clientId={process.env.REACT_APP_OAUTH_CLIENT_ID}
         onSuccess={this.onAuthResponse.bind(this)}
         onFailure={this.onAuthResponse.bind(this)}
-        className="login btn btn-primary"
+        render={renderProps => (
+          <span onClick={renderProps.onClick} className="login btn btn-primary">
+            <span className="fa fa-google" /> Login with Google
+          </span>
+        )}
       >
-        <span>
-          <span className="fa fa-google" /> Login with Google
-        </span>
       </GoogleLogin>
     );
   }
diff --git a/src/sagas/experiments.js b/src/sagas/experiments.js
index 9b8c1a8f..d9c410f7 100644
--- a/src/sagas/experiments.js
+++ b/src/sagas/experiments.js
@@ -1,5 +1,4 @@
-import { delay } from "redux-saga";
-import { call, put, select } from "redux-saga/effects";
+import { call, put, select, delay } from "redux-saga/effects";
 import { addPropToStoreObject, addToStore } from "../actions/objects";
 import { setLastSimulatedTick } from "../actions/simulation/tick";
 import { addBatchToStates } from "../actions/states";
diff --git a/src/store/configure-store.js b/src/store/configure-store.js
index 2c5f46fb..02faf073 100644
--- a/src/store/configure-store.js
+++ b/src/store/configure-store.js
@@ -24,12 +24,15 @@ const middlewares = [
   viewportAdjustmentMiddleware
 ];
 
+export let store = undefined;
+
 export default function configureStore() {
-  const store = createStore(
+  const configuredStore = createStore(
     rootReducer,
     compose(persistState("auth"), applyMiddleware(...middlewares))
   );
   sagaMiddleware.run(rootSaga);
+  store = configuredStore;
 
-  return store;
+  return configuredStore;
 }
-- 
cgit v1.2.3