Merge pull request #7683 from owncloud/proper-content-type-on-ocs-exceptions

set content-type on ocs exceptions
This commit is contained in:
Vincent Petry 2014-03-12 10:19:40 +01:00
commit ce790119ae
2 changed files with 32 additions and 3 deletions

View File

@ -116,9 +116,7 @@ class OC_API {
);
}
$response = self::mergeResponses($responses);
$formats = array('json', 'xml');
$format = !empty($_GET['format']) && in_array($_GET['format'], $formats) ? $_GET['format'] : 'xml';
$format = self::requestedFormat();
if (self::$logoutRequired) {
OC_User::logout();
}
@ -350,4 +348,33 @@ class OC_API {
}
}
/**
* @return string
*/
public static function requestedFormat() {
$formats = array('json', 'xml');
$format = !empty($_GET['format']) && in_array($_GET['format'], $formats) ? $_GET['format'] : 'xml';
return $format;
}
/**
* Based on the requested format the response content type is set
*/
public static function setContentType() {
$format = \OC_API::requestedFormat();
if ($format === 'xml') {
header('Content-type: text/xml; charset=UTF-8');
return;
}
if ($format === 'json') {
header('Content-Type: application/json; charset=utf-8');
return;
}
header('Content-Type: application/octet-stream; charset=utf-8');
}
}

View File

@ -28,8 +28,10 @@ use Symfony\Component\Routing\Exception\MethodNotAllowedException;
try {
OC::getRouter()->match('/ocs'.OC_Request::getRawPathInfo());
} catch (ResourceNotFoundException $e) {
OC_API::setContentType();
OC_OCS::notFound();
} catch (MethodNotAllowedException $e) {
OC_API::setContentType();
OC_Response::setStatus(405);
}