From 286a5dd6c62adb07f364f59ed58cbf21e5a3331e Mon Sep 17 00:00:00 2001
From: Cassie Tarakajian <ctarakajian@gmail.com>
Date: Fri, 13 Jan 2017 11:55:13 -0500
Subject: [PATCH] fixes #5, turns autosave off when a user logs off, hide timer
 component when user is not owner of sketch

---
 client/modules/IDE/components/Editor.jsx              |  4 +++-
 client/modules/IDE/components/{Timer.js => Timer.jsx} | 10 ++++++++--
 client/modules/IDE/pages/IDEView.jsx                  |  3 ++-
 client/styles/components/_timer.scss                  |  3 +++
 4 files changed, 16 insertions(+), 4 deletions(-)
 rename client/modules/IDE/components/{Timer.js => Timer.jsx} (79%)

diff --git a/client/modules/IDE/components/Editor.jsx b/client/modules/IDE/components/Editor.jsx
index 1d51ea9f..e4aafd19 100644
--- a/client/modules/IDE/components/Editor.jsx
+++ b/client/modules/IDE/components/Editor.jsx
@@ -223,6 +223,7 @@ class Editor extends React.Component {
             </span>
             <Timer
               projectSavedTime={this.props.projectSavedTime}
+              isUserOwner={this.props.isUserOwner}
             />
           </div>
           <button
@@ -287,7 +288,8 @@ Editor.propTypes = {
   files: PropTypes.array.isRequired,
   isExpanded: PropTypes.bool.isRequired,
   collapseSidebar: PropTypes.func.isRequired,
-  expandSidebar: PropTypes.func.isRequired
+  expandSidebar: PropTypes.func.isRequired,
+  isUserOwner: PropTypes.bool
 };
 
 export default Editor;
diff --git a/client/modules/IDE/components/Timer.js b/client/modules/IDE/components/Timer.jsx
similarity index 79%
rename from client/modules/IDE/components/Timer.js
rename to client/modules/IDE/components/Timer.jsx
index 9d980212..e82873e9 100644
--- a/client/modules/IDE/components/Timer.js
+++ b/client/modules/IDE/components/Timer.jsx
@@ -1,5 +1,6 @@
 import React, { PropTypes } from 'react';
 import moment from 'moment';
+import classNames from 'classnames';
 
 class Timer extends React.Component {
   constructor(props) {
@@ -31,8 +32,12 @@ class Timer extends React.Component {
   }
 
   render() {
+    const timerClass = classNames({
+      'timer__saved-time': true,
+      'timer__saved-time--notOwner': !this.props.isUserOwner
+    });
     return (
-      <span className="timer__saved-time">
+      <span className={timerClass}>
         {this.props.projectSavedTime !== '' ? this.showSavedTime() : null}
       </span>
     );
@@ -40,7 +45,8 @@ class Timer extends React.Component {
 }
 
 Timer.propTypes = {
-  projectSavedTime: PropTypes.string.isRequired
+  projectSavedTime: PropTypes.string.isRequired,
+  isUserOwner: PropTypes.bool
 };
 
 export default Timer;
diff --git a/client/modules/IDE/pages/IDEView.jsx b/client/modules/IDE/pages/IDEView.jsx
index d2c91d8a..fc552603 100644
--- a/client/modules/IDE/pages/IDEView.jsx
+++ b/client/modules/IDE/pages/IDEView.jsx
@@ -110,7 +110,7 @@ class IDEView extends React.Component {
       }
     }
 
-    if (this.autosaveInterval && !this.props.project.id) {
+    if (this.autosaveInterval && (!this.props.project.id || !this.isUserOwner())) {
       clearInterval(this.autosaveInterval);
       this.autosaveInterval = null;
     }
@@ -322,6 +322,7 @@ class IDEView extends React.Component {
                   isExpanded={this.props.ide.sidebarIsExpanded}
                   expandSidebar={this.props.expandSidebar}
                   collapseSidebar={this.props.collapseSidebar}
+                  isUserOwner={this.isUserOwner()}
                 />
                 <Console
                   consoleEvents={this.props.console}
diff --git a/client/styles/components/_timer.scss b/client/styles/components/_timer.scss
index abf28e21..1c377e12 100644
--- a/client/styles/components/_timer.scss
+++ b/client/styles/components/_timer.scss
@@ -4,4 +4,7 @@
   }
   font-size: #{12 / $base-font-size}rem;
   padding-right: #{30 / $base-font-size}rem;
+  &.timer__saved-time--notOwner {
+    display: none;
+  }
 }