Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ErrorException: setcookie() expects at most 7 parameters, 8 given #14643

Closed
mvanvu opened this issue Dec 23, 2019 · 1 comment
Closed

ErrorException: setcookie() expects at most 7 parameters, 8 given #14643

mvanvu opened this issue Dec 23, 2019 · 1 comment

Comments

@mvanvu
Copy link
Contributor

@mvanvu mvanvu commented Dec 23, 2019

The issue when I use (new Phalcon\Debug)->listen(true, true) and delete a cookie name from CONTROLLER like below

if ($this->cookies->has('cms.site.language'))
{
          $this->cookies->delete('cms.site.language');
}

It works fine when $lowServerity === false

@niden niden added 4.0 and removed Bug - Unverified labels Dec 23, 2019
niden added a commit to niden/cphalcon that referenced this issue Dec 23, 2019
niden added a commit to niden/cphalcon that referenced this issue Dec 23, 2019
@niden niden mentioned this issue Dec 23, 2019
3 of 4 tasks complete
niden added a commit to niden/cphalcon that referenced this issue Dec 23, 2019
niden added a commit to niden/cphalcon that referenced this issue Dec 23, 2019
niden added a commit that referenced this issue Dec 23, 2019
@niden

This comment has been minimized.

Copy link
Member

@niden niden commented Dec 23, 2019

Resolved in #14647

Thank you @mvanvu

@niden niden closed this Dec 23, 2019
niden added a commit to niden/cphalcon that referenced this issue Dec 23, 2019
…om/niden/cphalcon into niden-T2893-logger-date-format

* 'T2893-logger-date-format' of https://github.com/niden/cphalcon:
  [phalcon#14643] - Updated changelog
  [phalcon#14643] - Corrected the setcookie parameters
  [phalcon#2893] - Updated changelog
  [phalcon#2893] - Updated changelog
  [phalcon#2893] - Corrected typo
  [phalcon#2893] - Minor correction
  [phalcon#2893] - Added tests
  [phalcon#2893] - Restructured logger to use DateTimeImmutable to handle microseconds
  add mysql column definitions tests
  Fixed Mysql column definition
niden added a commit to niden/cphalcon that referenced this issue Dec 23, 2019
* niden-T2893-logger-date-format:
  [phalcon#14643] - Updated changelog
  [phalcon#14643] - Corrected the setcookie parameters
  [phalcon#2893] - Updated changelog
  [phalcon#2893] - Updated changelog
  [phalcon#2893] - Corrected typo
  [phalcon#2893] - Minor correction
  [phalcon#2893] - Added tests
  [phalcon#2893] - Restructured logger to use DateTimeImmutable to handle microseconds
  add mysql column definitions tests
  Fixed Mysql column definition
@niden niden added bug status: low and removed Bug - Low labels Dec 23, 2019
@niden niden added this to Implemented in Phalcon Roadmap Dec 26, 2019
@niden niden moved this from Implemented to Released in Phalcon Roadmap Jan 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Phalcon Roadmap
  
Released
2 participants
You can’t perform that action at this time.