Skip to content

Commit

Permalink
Change serialize to _serialize
Browse files Browse the repository at this point in the history
Its possible that a developer would use 'serialize' as a legitimate
view variable.  Prefix with an _ to minimize that.
  • Loading branch information
markstory committed Nov 28, 2011
1 parent 04463c4 commit 6e1b1df
Show file tree
Hide file tree
Showing 4 changed files with 24 additions and 24 deletions.
6 changes: 3 additions & 3 deletions lib/Cake/Test/Case/View/JsonViewTest.php
Expand Up @@ -39,7 +39,7 @@ public function testRenderWithoutView() {
$Response = new CakeResponse();
$Controller = new Controller($Request, $Response);
$data = array('user' => 'fake', 'list' => array('item1', 'item2'));
$Controller->set(array('data' => $data, 'serialize' => 'data'));
$Controller->set(array('data' => $data, '_serialize' => 'data'));
$View = new JsonView($Controller);
$output = $View->render(false);

Expand All @@ -48,7 +48,7 @@ public function testRenderWithoutView() {
}

/**
* Test render with an array in serialize
* Test render with an array in _serialize
*
* @return void
*/
Expand All @@ -58,7 +58,7 @@ public function testRenderWithoutViewMultiple() {
$Controller = new Controller($Request, $Response);
$data = array('no' => 'nope', 'user' => 'fake', 'list' => array('item1', 'item2'));
$Controller->set($data);
$Controller->set('serialize', array('no', 'user'));
$Controller->set('_serialize', array('no', 'user'));
$View = new JsonView($Controller);
$output = $View->render(false);

Expand Down
6 changes: 3 additions & 3 deletions lib/Cake/Test/Case/View/XmlViewTest.php
Expand Up @@ -39,7 +39,7 @@ public function testRenderWithoutView() {
$Response = new CakeResponse();
$Controller = new Controller($Request, $Response);
$data = array('users' => array('user' => array('user1', 'user2')));
$Controller->set(array('users' => $data, 'serialize' => 'users'));
$Controller->set(array('users' => $data, '_serialize' => 'users'));
$View = new XmlView($Controller);
$output = $View->render(false);

Expand All @@ -49,7 +49,7 @@ public function testRenderWithoutView() {
}

/**
* Test render with an array in serialize
* Test render with an array in _serialize
*
* @return void
*/
Expand All @@ -59,7 +59,7 @@ public function testRenderWithoutViewMultiple() {
$Controller = new Controller($Request, $Response);
$data = array('no' => 'nope', 'user' => 'fake', 'list' => array('item1', 'item2'));
$Controller->set($data);
$Controller->set('serialize', array('no', 'user'));
$Controller->set('_serialize', array('no', 'user'));
$View = new XmlView($Controller);
$output = $View->render(false);

Expand Down
18 changes: 9 additions & 9 deletions lib/Cake/View/JsonView.php
Expand Up @@ -16,31 +16,31 @@
/**
* A view class that is used for JSON responses.
*
* By setting the 'serialize' key in your controller, you can specify a view variable
* By setting the '_serialize' key in your controller, you can specify a view variable
* that should be serialized to JSON and used as the response for the request.
* This allows you to omit views + layouts, if your just need to emit a single view
* variable as the JSON response.
*
* In your controller, you could do the following:
*
* `$this->set(array('posts' => $posts, 'serialize' => 'posts'));`
* `$this->set(array('posts' => $posts, '_serialize' => 'posts'));`
*
* When the view is rendered, the `$posts` view variable will be serialized
* into JSON.
*
* You can also define `'serialize'` as an array. This will create a top level object containing
* You can also define `'_serialize'` as an array. This will create a top level object containing
* all the named view variables:
*
* {{{
* $this->set(compact('posts', 'users', 'stuff'));
* $this->set('serialize', array('posts', 'users'));
* $this->set('_serialize', array('posts', 'users'));
* }}}
*
* The above would generate a JSON object that looks like:
*
* `{"posts": [...], "users": [...]}`
*
* If you don't use the `serialize` key, you will need a view. You can use extended
* If you don't use the `_serialize` key, you will need a view. You can use extended
* views to provide layout like functionality.
*
* @package Cake.View
Expand Down Expand Up @@ -73,18 +73,18 @@ public function __construct($controller) {
/**
* Render a JSON view.
*
* Uses the special 'serialize' parameter to convert a set of
* Uses the special '_serialize' parameter to convert a set of
* view variables into a JSON response. Makes generating simple
* JSON responses very easy. You can omit the 'serialize' parameter,
* JSON responses very easy. You can omit the '_serialize' parameter,
* and use a normal view + layout as well.
*
* @param string $view The view being rendered.
* @param string $layout The layout being rendered.
* @return string The rendered view.
*/
public function render($view = null, $layout = null) {
if (isset($this->viewVars['serialize'])) {
$serialize = $this->viewVars['serialize'];
if (isset($this->viewVars['_serialize'])) {
$serialize = $this->viewVars['_serialize'];
if (is_array($serialize)) {
$data = array();
foreach ($serialize as $key) {
Expand Down
18 changes: 9 additions & 9 deletions lib/Cake/View/XmlView.php
Expand Up @@ -17,33 +17,33 @@
/**
* A view class that is used for creating XML responses.
*
* By setting the 'serialize' key in your controller, you can specify a view variable
* By setting the '_serialize' key in your controller, you can specify a view variable
* that should be serialized to XML and used as the response for the request.
* This allows you to omit views + layouts, if your just need to emit a single view
* variable as the XML response.
*
* In your controller, you could do the following:
*
* `$this->set(array('posts' => $posts, 'serialize' => 'posts'));`
* `$this->set(array('posts' => $posts, '_serialize' => 'posts'));`
*
* When the view is rendered, the `$posts` view variable will be serialized
* into XML.
*
* **Note** The view variable you specify must be compatible with Xml::fromArray().
*
* You can also define `'serialize'` as an array. This will create an additional
* You can also define `'_serialize'` as an array. This will create an additional
* top level element named `<response>` containing all the named view variables:
*
* {{{
* $this->set(compact('posts', 'users', 'stuff'));
* $this->set('serialize', array('posts', 'users'));
* $this->set('_serialize', array('posts', 'users'));
* }}}
*
* The above would generate a XML object that looks like:
*
* `<response><posts>...</posts><users>...</users></response>`
*
* If you don't use the `serialize` key, you will need a view. You can use extended
* If you don't use the `_serialize` key, you will need a view. You can use extended
* views to provide layout like functionality.
*
* @package Cake.View
Expand Down Expand Up @@ -74,18 +74,18 @@ public function __construct($controller) {
/**
* Render a XML view.
*
* Uses the special 'serialize' parameter to convert a set of
* Uses the special '_serialize' parameter to convert a set of
* view variables into a XML response. Makes generating simple
* XML responses very easy. You can omit the 'serialize' parameter,
* XML responses very easy. You can omit the '_serialize' parameter,
* and use a normal view + layout as well.
*
* @param string $view The view being rendered.
* @param string $layout The layout being rendered.
* @return string The rendered view.
*/
public function render($view = null, $layout = null) {
if (isset($this->viewVars['serialize'])) {
$serialize = $this->viewVars['serialize'];
if (isset($this->viewVars['_serialize'])) {
$serialize = $this->viewVars['_serialize'];
if (is_array($serialize)) {
$data = array('response' => array());
foreach ($serialize as $key) {
Expand Down

0 comments on commit 6e1b1df

Please sign in to comment.