Permalink
Browse files

Adding Todo about moving Cookie setting to CakeResponse.

  • Loading branch information...
1 parent 7276f33 commit 58888399f17842b005cd0c5e0ebb1671283372d9 @markstory markstory committed Aug 20, 2011
Showing with 3 additions and 0 deletions.
  1. +3 −0 lib/Cake/Controller/Component/CookieComponent.php
@@ -398,6 +398,9 @@ protected function _delete($name) {
/**
* Object wrapper for setcookie() so it can be mocked in unit tests.
*
+ * @todo Re-factor setting cookies into CakeResponse. Cookies are part
+ * of the HTTP response, and should be handled there.
+ *
* @param string $name Name of the cookie
* @param integer $expire Time the cookie expires in
* @param string $path Path the cookie applies to

0 comments on commit 5888839

Please sign in to comment.