fix updating file to return all file keys

client/constants.js

@@ -1,4 +1,4 @@
-export const UPDATE_FILE = 'UPDATE_FILE';
+export const UPDATE_FILE_CONTENT = 'UPDATE_FILE_CONTENT';
 export const TOGGLE_SKETCH = 'TOGGLE_SKETCH';
 
 export const START_SKETCH = 'START_SKETCH';

client/modules/IDE/actions/files.js

@@ -1,8 +1,8 @@
 import * as ActionTypes from '../../../constants';
 
-export function updateFile(name, content) {
+export function updateFileContent(name, content) {
   return {
-    type: ActionTypes.UPDATE_FILE,
+    type: ActionTypes.UPDATE_FILE_CONTENT,
     name,
     content
   };

client/modules/IDE/components/Editor.js

@@ -14,7 +14,7 @@ class Editor extends React.Component {
       mode: 'javascript'
     });
     this._cm.on('change', () => { // eslint-disable-line
-      this.props.updateFile('sketch.js', this._cm.getValue());
+      this.props.updateFileContent('sketch.js', this._cm.getValue());
     });
     this._cm.getWrapperElement().style['font-size'] = `${this.props.fontSize}px`;
   }
@@ -42,7 +42,7 @@ class Editor extends React.Component {
 
 Editor.propTypes = {
   content: PropTypes.string.isRequired,
-  updateFile: PropTypes.func.isRequired,
+  updateFileContent: PropTypes.func.isRequired,
   fontSize: PropTypes.number.isRequired
 };
 

client/modules/IDE/components/Sidebar.js

@@ -5,7 +5,10 @@ function Sidebar(props) {
     <section className="sidebar">
       <ul className="sidebar__file-list">
         {props.files.map(file =>
-          <li className="sidebar__file-item">{file.name}</li>
+          <li
+            className="sidebar__file-item"
+            key={file.id}
+          >{file.name}</li>
         )}
       </ul>
     </section>

client/modules/IDE/pages/IDEView.js

@@ -48,7 +48,7 @@ class IDEView extends React.Component {
         <Sidebar files={this.props.files} />
         <Editor
           content={this.props.files[0].content}
-          updateFile={this.props.updateFile}
+          updateFileContent={this.props.updateFileContent}
           fontSize={this.props.preferences.fontSize}
         />
         <PreviewFrame
@@ -89,7 +89,7 @@ IDEView.propTypes = {
   increaseFont: PropTypes.func.isRequired,
   decreaseFont: PropTypes.func.isRequired,
   files: PropTypes.array.isRequired,
-  updateFile: PropTypes.func.isRequired
+  updateFileContent: PropTypes.func.isRequired
 };
 
 function mapStateToProps(state) {

client/modules/IDE/reducers/files.js

@@ -33,16 +33,13 @@ const initialState = [
 
 const files = (state = initialState, action) => {
   switch (action.type) {
-    case ActionTypes.UPDATE_FILE:
+    case ActionTypes.UPDATE_FILE_CONTENT:
       return state.map(file => {
         if (file.name !== action.name) {
           return file;
         }
 
-        return {
+        return Object.assign({}, file, { content: action.content });
-          name: file.name,
-          content: action.content
-        };
       });
     case ActionTypes.NEW_PROJECT:
       return [...action.files];

server/controllers/project.controller.js

@@ -10,11 +10,7 @@ export function createProject(req, res) {
 
   Project.create(projectValues, (err, newProject) => {
     if (err) { return res.json({ success: false }); }
-    return res.json({
+    return res.json(newProject);
-      id: newProject._id, // eslint-disable-line no-underscore-dangle
-      name: newProject.name,
-      files: newProject.files
-    });
   });
 }
 
@@ -24,11 +20,7 @@ export function updateProject(req, res) {
       $set: req.body
     }, (err, updatedProject) => {
       if (err) { return res.json({ success: false }); }
-      return res.json({
+      return res.json(updatedProject);
-        id: updatedProject._id, // eslint-disable-line no-underscore-dangle
-        name: updatedProject.name,
-        file: updatedProject.files
-      });
     });
 }
 
@@ -38,11 +30,7 @@ export function getProject(req, res) {
       return res.status(404).send({ message: 'Project with that id does not exist' });
     }
 
-    return res.json({
+    return res.json(project);
-      id: project._id, // eslint-disable-line no-underscore-dangle
-      name: project.name,
-      files: project.files
-    });
   });
 }
 

server/models/project.js

@@ -28,6 +28,14 @@ const fileSchema = new Schema({
   content: { type: String, default: defaultSketch }
 }, { timestamps: true, _id: true });
 
+fileSchema.virtual('id').get(function(){
+    return this._id.toHexString();
+});
+
+fileSchema.set('toJSON', {
+    virtuals: true
+});
+
 const projectSchema = new Schema({
   name: { type: String, default: "Hello p5.js, it's the server" },
   user: { type: Schema.Types.ObjectId, ref: 'User' },
@@ -35,4 +43,12 @@ const projectSchema = new Schema({
   _id: { type: String, default: shortid.generate }
 }, { timestamps: true });
 
+projectSchema.virtual('id').get(function(){
+    return this._id;
+});
+
+projectSchema.set('toJSON', {
+    virtuals: true
+});
+
 export default mongoose.model('Project', projectSchema);