# HG changeset patch
# User Kim Alvefur <zash@zash.se>
# Date 1449861217 -3600
# Node ID 507301531cf5fcff187bdedaf1cc95c3464807c5
# Parent  dc0c6b8dc638c7cd9a7c2ba218c9d6b60d24cb95
util.datamanager: In append() collect status when closing file handle as it may fail (eg the implied flush)

diff -r dc0c6b8dc638 -r 507301531cf5 util/datamanager.lua
--- a/util/datamanager.lua	Fri Dec 11 20:11:48 2015 +0100
+++ b/util/datamanager.lua	Fri Dec 11 20:13:37 2015 +0100
@@ -210,9 +210,10 @@
 end
 
 local function append(username, host, datastore, ext, data)
-	local f, msg = io_open(getpath(username, host, datastore, ext, true), "r+");
+	local filename = getpath(username, host, datastore, ext, true);
+	local f, msg = io_open(filename, "r+");
 	if not f then
-		f, msg = io_open(getpath(username, host, datastore, ext, true), "w");
+		f, msg = io_open(filename, "w");
 	end
 	if not f then
 		return nil, msg;
@@ -225,7 +226,12 @@
 	else
 		return ok, msg;
 	end
-	f:close();
+
+	ok, msg = f:close();
+	if not ok then
+		return ok, msg;
+	end
+
 	return true;
 end