Merge pull request #234 from toolstack/fix-sym-links

Rewrite logic for saving notes to replace contents...

Author: toolstack

lib/util/file_utils.vala

@@ -3,30 +3,25 @@ namespace FileUtils {
 	public void save_to (File file, string text) {
 		try {
 			if (file.query_exists ()) {
+				// If the file already exists check to see if anything has changed, if not, bail.
 				string etag_out;
 				uint8[] text_data = {};
 				file.load_contents (null, out text_data, out etag_out);
 				if (text == (string) text_data) {
 					return;
 				}
-				file.delete ();
-			}
-			// Let's make sure the directory exists we're trying to save to.
-			// If it doesn't, create it.
-			var path = file.get_parent ().get_parse_name ();
-			var dir = File.new_for_path (path);
-			if (!dir.query_exists ()) {
-				dir.make_directory_with_parents ();
-			}
-			var data_stream = new DataOutputStream (
-				file.create (FileCreateFlags.REPLACE_DESTINATION)
-			);
-			uint8[] data = text.data;
-			var l = data.length;
-			long written = 0;
-			while (written < l) {
-				written += data_stream.write (data[written:data.length]);
+			} else {
+				// Let's make sure the directory exists we're trying to save to.
+				// If it doesn't, create it.
+				var path = file.get_parent ().get_parse_name ();
+				var dir = File.new_for_path (path);
+				if (!dir.query_exists ()) {
+					dir.make_directory_with_parents ();
+				}
 			}
+
+			// Write the text out to the file.
+			file.replace_contents (text.data, null, false, FileCreateFlags.NONE, null);
 		} catch (Error e) {
 			error (e.message);
 		}