add readonly param to openDocuments

microfins · Feb 18, 2016 · 5f46218 · 5f46218
1 parent 6cf0e1b
commit 5f46218
Show file tree

Hide file tree

Showing 4 changed files with 28 additions and 4 deletions.
diff --git a/qml/CodeEditorView.qml b/qml/CodeEditorView.qml
@@ -33,6 +33,7 @@ Item {
 	signal breakpointsChanged(string documentId)
 	signal isCleanChanged(var isClean, var document)
 	signal loadComplete
+    signal changeDocument(var document)
 	signal documentClosed(string path)
 
 	onDocumentClosed:
@@ -134,6 +135,9 @@ Item {
 			editor.onLoadComplete.connect(function() {
 				codeEditorView.loadComplete();
 			});
+            editor.onChangeDocument.connect(function() {
+                codeEditorView.changeDocument(editor.document);
+            });
 			editor.onEditorTextChanged.connect(function() {
 				documentEdit(editor.document.documentId);
 				if (editor.document.isContract)
@@ -321,7 +325,7 @@ Item {
 			for (var i = 0; i < editorListModel.count; i++)
 			{
 				var doc = editorListModel.get(i);
-				if (doc.path === document.path)
+                if (doc.path === document.path && !document.readOnly)
 				{
 					fileIo.writeFile(document.path, editors.itemAt(i).item.getText());
 					break;
@@ -384,6 +388,20 @@ Item {
 				loadComplete()
 			}
 
+            Connections
+            {
+                target: mainContent.codeEditor
+                onChangeDocument: {
+                    var docs = mainContent.codeEditor.openedDocuments();
+                    for (var d = 0; d < editorListModel.count; d++)
+                    {
+                        if (editorListModel.get(d).path === document.path)
+                            editorListModel.get(d).readOnly = document.readOnly;
+                    }
+                    console.log("signal " + editorListModel.count);
+                }
+            }
+
 			Connections
 			{
 				target: projectModel

diff --git a/qml/WebCodeEditor.qml b/qml/WebCodeEditor.qml
@@ -31,6 +31,7 @@ Item {
 	signal breakpointsChanged
 	signal editorTextChanged
 	signal loadComplete
+    signal changeDocument(var document)
 	property bool isClean: true
 	property string currentText: ""
 	property string currentMode: ""
@@ -172,18 +173,21 @@ Item {
 		{
 			if (!loading && editorBrowser) {
 				initialized = true;
-				setFontSize(fontSize);
-
+                setFontSize(fontSize);
 				var size = fileIo.getFileSize(document.path);
 				if (size > c_max_open_filesize)
 				{
 					setText("File size is too large!", currentMode);
 					setReadOnly(true);
+                    document.readOnly = true;
+                    changeDocument(document);
 				}
 				else if (!fileIo.isFileText(document.path))
 				{
 					setText("Can't read binary file!", currentMode);
 					setReadOnly(true);
+                    document.readOnly = true;
+                    changeDocument(document);
 				}
 				else 
 					setText(currentText, currentMode);

diff --git a/qml/js/ProjectModel.js b/qml/js/ProjectModel.js
@@ -253,6 +253,7 @@ function file(docData)
 	var groupName = isContract ? qsTr("Contracts") : isJs ? qsTr("Javascript") : isHtml ? qsTr("Web Pages") : isCss ? qsTr("Styles") : isImg ? qsTr("Images") : qsTr("Misc");
 	var docData = {
 		contract: false,
+        readOnly: false,
 		path: path,
 		fileName: fileName,
 		name: fileName,

diff --git a/src/FileIo.cpp b/src/FileIo.cpp
@@ -222,7 +222,8 @@ bool FileIo::isFileText(QString const& _url)
 					return true;
 			}
 
-			for (int i = 0; i < buf.size(); i++)
+			int checkLength = std::min(buf.size(), 1024000);
+			for (int i = 0; i < checkLength; i++)
 			{
 				if (!isascii(buf[i]) ||
 					(iscntrl(buf[i]) && !isspace(buf[i]) &&