API: Fix return type of empty values in file/history

If the array is not casted to a object, json_encode will encode it as
[] if empty, but {} if it contains data. Always return an object by
casting the array to an object if it is empty.

Fixes #15

Signed-off-by: Florian Pritz <bluewind@xinu.at>

2 files changed, 12 insertions, 2 deletions

diff --git a/application/test/tests/api_v2/common.php b/application/test/tests/api_v2/common.php
index dbef3cc9c..103e156a8 100644
--- a/application/test/tests/api_v2/common.php
+++ b/application/test/tests/api_v2/common.php
@@ -53,8 +53,8 @@ class common extends \test\Test {
 		return $this->createApikey($userid, $access_level);
 	}
 
-	protected function callEndpoint($verb, $endpoint, $data)
+	protected function callEndpoint($verb, $endpoint, $data, $return_json = false)
 	{
-		return $this->CallAPI($verb, "$this->server_url/api/v2.0.0/$endpoint", $data);
+		return $this->CallAPI($verb, "$this->server_url/api/v2.0.0/$endpoint", $data, $return_json);
 	}
 }
diff --git a/application/test/tests/api_v2/test_history.php b/application/test/tests/api_v2/test_history.php
index 3a168cafa..f09aab9bb 100644
--- a/application/test/tests/api_v2/test_history.php
+++ b/application/test/tests/api_v2/test_history.php
@@ -31,6 +31,16 @@ class test_history extends common {
 		$this->t->is($ret["data"]["total_size"], "0", "total_size = 0 since no uploads");
 	}
 
+	public function test_history_empty_json_structure()
+	{
+		$apikey = $this->createUserAndApikey();
+		$ret = $this->CallEndpoint("POST", "file/history", array(
+			"apikey" => $apikey,
+		), true);
+
+		$this->t->is($ret, '{"status":"success","data":{"items":{},"multipaste_items":{},"total_size":"0"}}', "empty lists should be json objects, not arrays");
+	}
+
 	public function test_history_notEmptyAfterUploadSameMD5()
 	{
 		$apikey = $this->createUserAndApikey();