diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

feature #9963 [HttpFoundation] JsonResponse::setEncodingOptions accepts also integer (stloyd)

Browse Source 
This PR was merged into the 2.5-dev branch.

Discussion
----------

[HttpFoundation] JsonResponse::setEncodingOptions accepts also integer

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Tests pass?   | yes
| License       | MIT

Now you can also set encoding options like:
```php
$response->setEncodingOptions(JSON_UNESCAPED_UNICODE | $response->getEncodingOptions());

Commits
-------

f8bc3b2 [HttpFoundation] JsonResponse::setEncodingOptions accepts also integer
This commit is contained in:
Fabien Potencier 2014-01-07 14:23:52 +01:00
commit ef12af9948
2 changed files with 4 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/Symfony/Component/HttpFoundation/JsonResponse.php
View File

@ -117,18 +117,13 @@ class JsonResponse extends Response
/**
* Sets options used while encoding data to JSON.
*
* @param array $encodingOptions
* @param integer $encodingOptions
*
* @return JsonResponse
*/
public function setEncodingOptions(array $encodingOptions)
public function setEncodingOptions($encodingOptions)
{
$this->encodingOptions = 0;
foreach ($encodingOptions as $encodingOption) {
if (($this->encodingOptions & $encodingOption) != $encodingOption) {
$this->encodingOptions |= $encodingOption;
}
}
$this->encodingOptions = (integer) $encodingOptions;
return $this->setData(json_decode($this->data));
}

2
src/Symfony/Component/HttpFoundation/Tests/JsonResponseTest.php
View File

@ -180,7 +180,7 @@ class JsonResponseTest extends \PHPUnit_Framework_TestCase
$this->assertEquals('[[1,2,3]]', $response->getContent());
$response->setEncodingOptions(array(JSON_FORCE_OBJECT));
$response->setEncodingOptions(JSON_FORCE_OBJECT);
$this->assertEquals('{"0":{"0":1,"1":2,"2":3}}', $response->getContent());
}