From 61f20d1d4c8255e2196b90ffa8110fa101a77fc2 Mon Sep 17 00:00:00 2001
From: Oliver Wright <meiamsome@meiamso.me>
Date: Mon, 25 Feb 2019 20:11:07 +0000
Subject: [PATCH] Refactor Nav Props - See #824 (#846)

---
 client/components/Nav.jsx            | 27 +++++++++++++++++----------
 client/modules/IDE/pages/IDEView.jsx | 19 -------------------
 2 files changed, 17 insertions(+), 29 deletions(-)

diff --git a/client/components/Nav.jsx b/client/components/Nav.jsx
index 23a6cc0a..b729cca9 100644
--- a/client/components/Nav.jsx
+++ b/client/components/Nav.jsx
@@ -1,12 +1,14 @@
 import PropTypes from 'prop-types';
 import React from 'react';
-import { bindActionCreators } from 'redux';
 import { connect } from 'react-redux';
 import { withRouter } from 'react-router';
 import { Link } from 'react-router';
 import InlineSVG from 'react-inlinesvg';
 import classNames from 'classnames';
 import * as IDEActions from '../modules/IDE/actions/ide';
+import * as projectActions from '../modules/IDE/actions/project';
+import { setAllAccessibleOutput } from '../modules/IDE/actions/preferences';
+import { logoutUser } from '../modules/User/actions';
 
 import { metaKeyName, } from '../utils/metaKey';
 
@@ -600,15 +602,20 @@ Nav.defaultProps = {
   cmController: {}
 };
 
-function mapDispatchToProps(dispatch) {
-  return bindActionCreators(
-    Object.assign(
-      {},
-      IDEActions
-    ),
-    dispatch
-  );
+function mapStateToProps(state) {
+  return {
+    project: state.project,
+    user: state.user,
+    unsavedChanges: state.ide.unsavedChanges
+  };
 }
 
-export default withRouter(connect(() => ({}), mapDispatchToProps)(Nav));
+const mapDispatchToProps = {
+  ...IDEActions,
+  ...projectActions,
+  logoutUser,
+  setAllAccessibleOutput
+};
+
+export default withRouter(connect(mapStateToProps, mapDispatchToProps)(Nav));
 export { Nav as NavComponent };
diff --git a/client/modules/IDE/pages/IDEView.jsx b/client/modules/IDE/pages/IDEView.jsx
index ec69229d..0485fa0a 100644
--- a/client/modules/IDE/pages/IDEView.jsx
+++ b/client/modules/IDE/pages/IDEView.jsx
@@ -209,23 +209,8 @@ class IDEView extends React.Component {
         </Helmet>
         {this.props.toast.isVisible && <Toast />}
         <Nav
-          user={this.props.user}
-          newProject={this.props.newProject}
-          saveProject={this.props.saveProject}
-          autosaveProject={this.props.autosaveProject}
-          exportProjectAsZip={this.props.exportProjectAsZip}
-          cloneProject={this.props.cloneProject}
-          project={this.props.project}
-          logoutUser={this.props.logoutUser}
-          startSketch={this.props.startSketch}
-          stopSketch={this.props.stopSketch}
-          showShareModal={this.props.showShareModal}
-          showErrorModal={this.props.showErrorModal}
-          unsavedChanges={this.props.ide.unsavedChanges}
           warnIfUnsavedChanges={this.warnIfUnsavedChanges}
-          showKeyboardShortcutModal={this.props.showKeyboardShortcutModal}
           cmController={this.cmController}
-          setAllAccessibleOutput={this.props.setAllAccessibleOutput}
         />
         <Toolbar
           className="Toolbar"
@@ -527,7 +512,6 @@ IDEView.propTypes = {
     id: PropTypes.string,
     username: PropTypes.string
   }).isRequired,
-  newProject: PropTypes.func.isRequired,
   saveProject: PropTypes.func.isRequired,
   ide: PropTypes.shape({
     isPlaying: PropTypes.bool.isRequired,
@@ -615,7 +599,6 @@ IDEView.propTypes = {
   closeNewFileModal: PropTypes.func.isRequired,
   expandSidebar: PropTypes.func.isRequired,
   collapseSidebar: PropTypes.func.isRequired,
-  exportProjectAsZip: PropTypes.func.isRequired,
   cloneProject: PropTypes.func.isRequired,
   expandConsole: PropTypes.func.isRequired,
   collapseConsole: PropTypes.func.isRequired,
@@ -623,14 +606,12 @@ IDEView.propTypes = {
   updateFileName: PropTypes.func.isRequired,
   showEditProjectName: PropTypes.func.isRequired,
   hideEditProjectName: PropTypes.func.isRequired,
-  logoutUser: PropTypes.func.isRequired,
   openProjectOptions: PropTypes.func.isRequired,
   closeProjectOptions: PropTypes.func.isRequired,
   newFolder: PropTypes.func.isRequired,
   closeNewFolderModal: PropTypes.func.isRequired,
   createFolder: PropTypes.func.isRequired,
   createFile: PropTypes.func.isRequired,
-  showShareModal: PropTypes.func.isRequired,
   closeShareModal: PropTypes.func.isRequired,
   showEditorOptions: PropTypes.func.isRequired,
   closeEditorOptions: PropTypes.func.isRequired,