Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

SetCookie: let it accept DateTime object #6673

Closed
ThaDafinser opened this issue Sep 17, 2014 · 1 comment
Closed

SetCookie: let it accept DateTime object #6673

ThaDafinser opened this issue Sep 17, 2014 · 1 comment
Assignees
@Ocramius
Labels
enhancement Http
Milestone
2.3.4

Comments

@ThaDafinser
Copy link
Contributor

Improve SetCookie to accept DateTime. In general i think everywhere, where you work with date/time/datetime, there should be possible to use DateTime object

https://github.com/zendframework/zf2/blob/master/library/Zend/Http/Header/SetCookie.php#L180
@dkemper dkemper mentioned this issue Nov 24, 2014
Merged
Ocramius added a commit that referenced this issue Nov 25, 2014
@Ocramius
#6673 - importing used DateTime class
63d28f2
Ocramius added a commit that referenced this issue Nov 25, 2014
@Ocramius
#6923 #6673 - minor docblock CS fixes
4d4a913
Ocramius added a commit that referenced this issue Nov 25, 2014
@Ocramius
#6923 #6673 - adding missing @group annotations for newly introduce…
38c663f 
…d tests
@Ocramius Ocramius self-assigned this Nov 25, 2014
@Ocramius Ocramius added this to the 2.3.4 milestone Nov 25, 2014
@Ocramius
Copy link
Member

Handled in #6923

gianarb pushed a commit to zendframework/zend-http that referenced this issue May 15, 2015
@dkemper
allow to set DateTimeObject to the expires SetCookie expire ref zendf…
e6c88a3 
…ramework/zendframework#6673
gianarb pushed a commit to zendframework/zend-http that referenced this issue May 15, 2015
@Ocramius
zendframework/zendframework#6673 - importing used DateTime class
dfa54c2
gianarb pushed a commit to zendframework/zend-http that referenced this issue May 15, 2015
@Ocramius
zendframework/zendframework#6923 zendframework/zendframework#6673 - m…
2ac9cda 
…inor docblock CS fixes
gianarb pushed a commit to zendframework/zend-http that referenced this issue May 15, 2015
@Ocramius
zendframework/zendframework#6923 zendframework/zendframework#6673 - a…
b720fdf 
…dding missing `@group` annotations for newly introduced tests
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Assignees

@Ocramius Ocramius

Labels
enhancement Http
Projects
None yet
Milestone
2.3.4
Development

No branches or pull requests
2 participants
@Ocramius @ThaDafinser