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
[8.0] Convert static property to constant #2463
base: 8.0
Are you sure you want to change the base?
Conversation
I think this is probably a good idea. But it's not that important for 7.0. I'd like to concentrate on the release. |
Yes I had it on 8.0.0 milestone. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 2 weeks if no further activity occurs. Thank you for your contributions. |
This needs a rebase. Also please dont change the phpstan-baseline. |
@@ -30,7 +30,7 @@ class SetCookie | |||
public static function fromString(string $cookie): self | |||
{ | |||
// Create the default return array | |||
$data = self::$defaults; | |||
$data = self::DEFAULTS; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what was the reason changing this?
@gmponos Are you able to resolve the merge conflicts please? |
By converting the static property to constant I guess this will disallow overwritting.
Again I am not sure if this is something we want for v7 or v8 or even if we want it at all.