Merge pull request #6 from ananyasen/FileIO

Open file fixes and adding opening in code view.

4 files changed, 73 insertions, 39 deletions

diff --git a/js/io/document/document-controller.js b/js/io/document/document-controller.js
index 15f026a3..c2a2dab6 100755
--- a/js/io/document/document-controller.js
+++ b/js/io/document/document-controller.js
@@ -87,8 +87,9 @@ var DocumentController = exports.DocumentController = Montage.create(Component,
     handleExecuteFileOpen: {
         value: function(event) {
             var pickerSettings = event._event.settings || {};
-            pickerSettings.callback = this.openFileWithURI;
-            pickerSettings.callbackScope = this;
+            pickerSettings.callback = this.openFileWithURI.bind(this);
+            pickerSettings.pickerMode = "read";
+            pickerSettings.inFileMode = true;
             this.application.ninja.filePickerController.showFilePicker(pickerSettings);
         }
     },
@@ -96,8 +97,7 @@ var DocumentController = exports.DocumentController = Montage.create(Component,
     handleExecuteNewFile: {
             value: function(event) {
                 var newFileSettings = event._event.settings || {};
-                newFileSettings.callback = this.createNewFile;
-                newFileSettings.callbackScope = this;
+                newFileSettings.callback = this.createNewFile.bind(this);
                 this.application.ninja.newFileController.showNewFileDialog(newFileSettings);
             }
     },
@@ -132,9 +132,9 @@ var DocumentController = exports.DocumentController = Montage.create(Component,
      */
     openNewFileCallback:{
         value:function(doc){
-            var response = doc || {"uri":"/Users/xhdq84/Documents/test.js", "success":true};//default just for testing
+            var response = doc || null;//default just for testing
             if(!!response && response.success && !!response.uri){
-                this.application.ninja.ioMediator.fileOpen({"uri":response.uri}, this.openFileCallback.bind(this));
+                this.application.ninja.ioMediator.fileOpen(response.uri, this.openFileCallback.bind(this));
             }
         }
     },
@@ -146,25 +146,56 @@ var DocumentController = exports.DocumentController = Montage.create(Component,
                 uri = uriArrayObj.uri[0];
             }
             //console.log("URI is: ", uri);
-
-            this.application.ninja.ioMediator.fileOpen({"uri":uri}, this.openFileCallback.bind(this));
+            if(!!uri){
+                this.application.ninja.ioMediator.fileOpen(uri, this.openFileCallback.bind(this));
+            }
         }
     },
 
     /**
      * Public method
-     * doc contains:
-     *      type : file type, like js, css, etc
-     *      name : file name
-     *      source : file content
-     *      uri : file uri
      */
     openFileCallback:{
-        value:function(doc){
-            this.openDocument(doc);
+        value:function(response){
+            //Return Object Description
+//            Object.status (Always presents for handling)
+//            204: File exists (Success)
+//            404: File does not exists (Failure)
+//            500: Unknown (Probably cloud API not running)
+//
+//            (Below only present if succesfull 204)
+//
+//            Object.content
+//            Object.extension
+//            Object.name
+//            Object.uri
+//            Object.creationDate
+//            Object.modifiedDate
+//            Object.readOnly
+//            Object.size
+
+            var fileDetails = {};
+            if(!!response && (response.status === 204)){
+                /**
+                 * fileDetails format:
+                 * {"type": "js", "name": "test", "source": fileContent, "uri": uri}
+                 */
+                fileDetails.type = response.extension;
+                fileDetails.source = response.content;
+                fileDetails.name = response.name;
+                fileDetails.uri = response.uri;
+
+                this.openDocument(fileDetails);
+            }else if(!!response && (response.status === 404)){
+                alert("Unable to open file.\n [Error: File does not exist]");
+            }else if(!!response && (response.status === 500)){
+                alert("Unable to open file.\n Check if Ninja Local Cloud is running.");
+            }else{
+                alert("Unable to open file.");
             }
-    },
 
+        }
+    },
 
     openProjectWithURI: {
         value: function(uri) {
@@ -371,7 +402,10 @@ var DocumentController = exports.DocumentController = Montage.create(Component,
                     this.application.ninja.stage.restoreScroll();
                     this.application.ninja.stage.hideCanvas(false);
                     this.application.ninja.stage.stageView.showRulers();
-            }
+                }else{
+                    //hide the iframe when switching to code view
+                    document.getElementById("iframeContainer").style.display="none";
+                }
         }
         }
     },
diff --git a/js/io/templates/descriptor.json b/js/io/templates/descriptor.json
index 20870934..acc03979 100644
--- a/js/io/templates/descriptor.json
+++ b/js/io/templates/descriptor.json
@@ -6,60 +6,60 @@
                 "name":"Blank File",
                 "uri":"/",
                 "type":"directory",
-        "children":["/js/io/templates/files/html.txt", "/js/io/templates/files/files/js.txt", "/js/io/templates/files/css.txt", "/js/io/templates/files/json.txt", "/js/io/templates/files/php.txt", "/js/io/templates/files/pl.txt", "/js/io/templates/files/py.txt", "/js/io/templates/files/rb.txt"]
+        "children":["js/io/templates/files/html.txt", "js/io/templates/files/js.txt", "js/io/templates/files/css.txt", "js/io/templates/files/json.txt", "js/io/templates/files/php.txt", "js/io/templates/files/pl.txt", "js/io/templates/files/py.txt", "js/io/templates/files/rb.txt"]
         },
-        "/js/io/templates/files/html.txt":{
+        "js/io/templates/files/html.txt":{
                 "name":"HTML",
-                "uri":"/js/io/templates/files/html.txt",
+                "uri":"js/io/templates/files/html.txt",
                 "type":"file",
         "fileExtension":".html",
         "children":["defaultTemplate"]
         },
-        "/js/io/templates/files/files/js.txt":{
+        "js/io/templates/files/js.txt":{
                 "name":"JavaScript",
-                "uri":"/js/io/templates/files/files/js.txt",
+                "uri":"js/io/templates/files/js.txt",
                 "type":"file",
         "fileExtension":".js",
         "children":["defaultTemplate"]
         },
-        "/js/io/templates/files/css.txt":{
+        "js/io/templates/files/css.txt":{
                 "name":"Cascading Style Sheets",
-                "uri":"/js/io/templates/files/css.txt",
+                "uri":"js/io/templates/files/css.txt",
                 "type":"file",
         "fileExtension":".css",
         "children":["defaultTemplate"]
         },
-        "/js/io/templates/files/json.txt":{
+        "js/io/templates/files/json.txt":{
                 "name":"JSON",
-                "uri":"/js/io/templates/files/json.txt",
+                "uri":"js/io/templates/files/json.txt",
                 "type":"file",
         "fileExtension":".json",
         "children":["defaultTemplate"]
         },
-        "/js/io/templates/files/php.txt":{
+        "js/io/templates/files/php.txt":{
                 "name":"PHP",
-                "uri":"/js/io/templates/files/php.txt",
+                "uri":"js/io/templates/files/php.txt",
                 "type":"file",
         "fileExtension":".php",
         "children":["defaultTemplate"]
         },
-        "/js/io/templates/files/pl.txt":{
+        "js/io/templates/files/pl.txt":{
                 "name":"Perl",
-                "uri":"/js/io/templates/files/pl.txt",
+                "uri":"js/io/templates/files/pl.txt",
                 "type":"file",
         "fileExtension":".pl",
         "children":["defaultTemplate"]
         },
-        "/js/io/templates/files/py.txt":{
+        "js/io/templates/files/py.txt":{
                 "name":"Python",
-                "uri":"/js/io/templates/files/py.txt",
+                "uri":"js/io/templates/files/py.txt",
                 "type":"file",
         "fileExtension":".py",
         "children":["defaultTemplate"]
         },
-        "/js/io/templates/files/rb.txt":{
+        "js/io/templates/files/rb.txt":{