From 9ed3ecc7bb0c47ded7d5c2f3425cd851452fde56 Mon Sep 17 00:00:00 2001
From: Paul Kaplan <pkaplan@media.mit.edu>
Date: Thu, 8 Nov 2018 13:19:12 -0500
Subject: [PATCH] Split up the backpack options and tie visibility to
 isLoggedIn

---
 src/views/preview/presentation.jsx | 12 ++++++------
 src/views/preview/preview.jsx      | 19 +++++++++----------
 2 files changed, 15 insertions(+), 16 deletions(-)

diff --git a/src/views/preview/presentation.jsx b/src/views/preview/presentation.jsx
index 1b8b69bb9..74416cd6a 100644
--- a/src/views/preview/presentation.jsx
+++ b/src/views/preview/presentation.jsx
@@ -44,12 +44,13 @@ const onKeyPress = e => {
 
 const PreviewPresentation = ({
     assetHost,
-    backpackOptions,
+    backpackHost,
     canAddToStudio,
     canDeleteComments,
     canReport,
     canRestoreComments,
     canShare,
+    canUseBackpack,
     cloudHost,
     comments,
     editable,
@@ -173,7 +174,8 @@ const PreviewPresentation = ({
                                 <IntlGUI
                                     isPlayerOnly
                                     assetHost={assetHost}
-                                    backpackOptions={backpackOptions}
+                                    backpackHost={backpackHost}
+                                    backpackVisible={canUseBackpack}
                                     basePath="/"
                                     className="guiPlayer"
                                     cloudHost={cloudHost}
@@ -444,15 +446,13 @@ const PreviewPresentation = ({
 PreviewPresentation.propTypes = {
     addToStudioOpen: PropTypes.bool,
     assetHost: PropTypes.string,
-    backpackOptions: PropTypes.shape({
-        host: PropTypes.string,
-        visible: PropTypes.bool
-    }),
+    backpackHost: PropTypes.string,
     canAddToStudio: PropTypes.bool,
     canDeleteComments: PropTypes.bool,
     canReport: PropTypes.bool,
     canRestoreComments: PropTypes.bool,
     canShare: PropTypes.bool,
+    canUseBackpack: PropTypes.bool,
     cloudHost: PropTypes.string,
     comments: PropTypes.arrayOf(PropTypes.object),
     editable: PropTypes.bool,
diff --git a/src/views/preview/preview.jsx b/src/views/preview/preview.jsx
index 7767648db..edf1fc444 100644
--- a/src/views/preview/preview.jsx
+++ b/src/views/preview/preview.jsx
@@ -400,12 +400,13 @@ class Preview extends React.Component {
                     <PreviewPresentation
                         addToStudioOpen={this.state.addToStudioOpen}
                         assetHost={this.props.assetHost}
-                        backpackOptions={this.props.backpackOptions}
+                        backpackHost={this.props.backpackHost}
                         canAddToStudio={this.props.canAddToStudio}
                         canDeleteComments={this.props.isAdmin || this.props.userOwnsProject}
                         canReport={this.props.canReport}
                         canRestoreComments={this.props.isAdmin}
                         canShare={this.props.canShare}
+                        canUseBackpack={this.props.canUseBackpack}
                         cloudHost={this.props.cloudHost}
                         comments={this.props.comments}
                         editable={this.props.isEditable}
@@ -453,7 +454,8 @@ class Preview extends React.Component {
                     <IntlGUI
                         hideIntro
                         assetHost={this.props.assetHost}
-                        backpackOptions={this.props.backpackOptions}
+                        backpackHost={this.props.backpackHost}
+                        backpackVisible={this.props.canUseBackpack}
                         basePath="/"
                         canCreateCopy={this.props.canCreateCopy}
                         canCreateNew={this.props.canCreateNew}
@@ -486,10 +488,7 @@ class Preview extends React.Component {
 
 Preview.propTypes = {
     assetHost: PropTypes.string.isRequired,
-    backpackOptions: PropTypes.shape({
-        host: PropTypes.string,
-        visible: PropTypes.bool
-    }),
+    backpackHost: PropTypes.string,
     canAddToStudio: PropTypes.bool,
     canCreateCopy: PropTypes.bool,
     canCreateNew: PropTypes.bool,
@@ -497,6 +496,7 @@ Preview.propTypes = {
     canReport: PropTypes.bool,
     canSave: PropTypes.bool,
     canShare: PropTypes.bool,
+    canUseBackpack: PropTypes.bool,
     cloudHost: PropTypes.string,
     comments: PropTypes.arrayOf(PropTypes.object),
     enableCommunity: PropTypes.bool,
@@ -560,10 +560,8 @@ Preview.propTypes = {
 
 Preview.defaultProps = {
     assetHost: process.env.ASSET_HOST,
-    backpackOptions: {
-        host: process.env.BACKPACK_HOST,
-        visible: true
-    },
+    backpackHost: process.env.BACKPACK_HOST,
+    canUseBackpack: false,
     cloudHost: process.env.CLOUDDATA_HOST,
     projectHost: process.env.PROJECT_HOST,
     sessionStatus: sessionActions.Status.NOT_FETCHED,
@@ -593,6 +591,7 @@ const mapStateToProps = state => {
         canReport: isLoggedIn && !userOwnsProject,
         canSave: isLoggedIn && userOwnsProject,
         canShare: userOwnsProject && state.permissions.social,
+        canUseBackpack: isLoggedIn,
         comments: state.preview.comments,
         enableCommunity: projectInfoPresent,
         faved: state.preview.faved,