Comparison

util.datamanager: Collect errors when deleting all stores of a user, but ignore "no such file"

comparison

equal deleted inserted replaced

-:e646c849d72f
+:dddbcd62183a
 end
 function purge(username, host)
 	local host_dir = format("%s/%s/", data_path, encode(host));
 	local deleted = 0;
+	local errs = {};
 	for file in lfs.dir(host_dir) do
 		if lfs.attributes(host_dir..file, "mode") == "directory" then
 			local store = decode(file);
-			deleted = deleted + (os_remove(getpath(username, host, store)) and 1 or 0);
+			local ok, err = os_remove(getpath(username, host, store));
-			deleted = deleted + (os_remove(getpath(username, host, store, "list")) and 1 or 0);
+			if not ok and not err:lower():match("no such") then errs[#errs+1] = err; end
-			-- We this will generate loads of "No such file or directory", but do we care?
+			local ok, err = os_remove(getpath(username, host, store, "list"));
-		end
+			if not ok and not err:lower():match("no such") then errs[#errs+1] = err; end
-	end
+		end
-	return deleted > 0, deleted;
+	end
+	return #errs == 0, t_concat(errs, ", ");
 end
 return _M;