Skip to content

Commit c204d46

Browse files
author
Florian Krämer
committed
Simplifying the CookieCollections
1 parent 685eb90 commit c204d46

File tree

2 files changed

+5
-59
lines changed

2 files changed

+5
-59
lines changed

src/Http/Cookie/CookieCollection.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,9 @@
1313
*/
1414
namespace Cake\Http\Cookie;
1515

16-
class CookieCollection
16+
use \ArrayIterator ;
17+
18+
class CookieCollection extends ArrayIterator
1719
{
1820
/**
1921
* @var Cookie[]

src/Http/Cookie/RequestCookies.php

Lines changed: 2 additions & 58 deletions
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,13 @@
1717
use Iterator;
1818
use Psr\Http\Message\ServerRequestInterface;
1919

20-
class RequestCookies extends CookieCollection implements Iterator
20+
class RequestCookies extends CookieCollection
2121
{
2222
/**
2323
* Create instance from a server request.
2424
*
2525
* @param \Psr\Http\Message\ServerRequestInterface $request Request object
26-
* @return \Cake\Http\Client\Cookie\RequestCookies
26+
* @return \Cake\Http\Client\RequestCookies
2727
*/
2828
public static function createFromRequest(ServerRequestInterface $request)
2929
{
@@ -66,60 +66,4 @@ public function get($name)
6666

6767
return $this->cookies[$key];
6868
}
69-
70-
/**
71-
* Current
72-
*
73-
* @return \Cake\Http\Client\Cookie\Cookie $cookie
74-
*/
75-
public function current()
76-
{
77-
return current($this->cookies);
78-
}
79-
80-
/**
81-
* Key
82-
*
83-
* @return string
84-
*/
85-
public function key()
86-
{
87-
$key = key($this->cookies);
88-
if ($key === null) {
89-
return $key;
90-
}
91-
$cookie = $this->cookies[$key];
92-
93-
return $cookie->getName();
94-
}
95-
96-
/**
97-
* Next
98-
*
99-
* @return void
100-
*/
101-
public function next()
102-
{
103-
next($this->cookies);
104-
}
105-
106-
/**
107-
* Valid
108-
*
109-
* @return bool
110-
*/
111-
public function valid()
112-
{
113-
return key($this->cookies) !== null;
114-
}
115-
116-
/**
117-
* Rewind
118-
*
119-
* @return void
120-
*/
121-
public function rewind()
122-
{
123-
reset($this->cookies);
124-
}
12569
}

0 commit comments

Comments
 (0)