From 6c46430cdb6edf411142d0a73fb888391ea86fd7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thomas=20M=C3=BCller?= <thomas.mueller@tmit.eu>
Date: Wed, 5 Aug 2015 11:23:29 +0200
Subject: [PATCH] Return 200 not null - we should return consistent types

---
 lib/private/api.php    | 3 ++-
 tests/ocs/response.php | 2 ++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/lib/private/api.php b/lib/private/api.php
index 6f44401576b..86d7558526b 100644
--- a/lib/private/api.php
+++ b/lib/private/api.php
@@ -423,6 +423,7 @@ class OC_API {
 
 	/**
 	 * @param integer $sc
+	 * @return int
 	 */
 	public static function mapStatusCodes($sc) {
 		switch ($sc) {
@@ -447,7 +448,7 @@ class OC_API {
 		if ($sc > 100 && $sc < 200) {
 			return Http::STATUS_BAD_REQUEST;
 		}
-		return null;
+		return Http::STATUS_OK;
 	}
 
 	/**
diff --git a/tests/ocs/response.php b/tests/ocs/response.php
index 71397bcf2c9..0d5883fa4cc 100644
--- a/tests/ocs/response.php
+++ b/tests/ocs/response.php
@@ -35,6 +35,8 @@ class OcsResponseTest extends \Test\TestCase {
 		return [
 			[Http::STATUS_OK, 100],
 			[Http::STATUS_BAD_REQUEST, 104],
+			[Http::STATUS_OK, 1000],
+			[201, 201],
 		];
 	}
 }