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
[Cache] remove deprecated PSR-16 implementations et al. #31760
Conversation
9eed8d6
to
38a67f8
Compare
*/ | ||
public function reset() | ||
{ | ||
$this->clear(); |
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.
isn't just doing the same thing as clear without the return better ?
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.
We must implement both: clear() for psr-6, reset() for service-contracts.
This is copy/paste from current implementations that exist in traits.
* | ||
* @return \Memcached | ||
* | ||
* @throws \ErrorException When invalid options or servers are provided |
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.
throws \ErrorException or InvalidArgumentException ?
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.
If you found a mismatch, please submit a fix on branch 4.2. this PR is only moving code around.
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.
Ok, thanks for the explanation.
…(nicolas-grekas) This PR was merged into the 5.0-dev branch. Discussion ---------- [Cache] remove deprecated PSR-16 implementations et al. | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | no | BC breaks? | yes | Deprecations? | no | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - \o/ (use `Psr16Cache` instead) Commits ------- 38a67f8 [Cache] remove deprecated PSR-16 implementations et al.
This PR was merged into the 5.2 branch. Discussion ---------- [Cache] Remove MemcachedTrait | Q | A | ------------- | --- | Branch? | 5.2 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | N/A | License | MIT | Doc PR | N/A This trait is not used anywhere. It had been removed once, with #31760. But it resurfaced, presumably because of a bad merge. It is flagged as ``@internal``, so I think we can safely delete it. Commits ------- 5fb4fd4 [Cache] Remove MemcachedTrait
\o/
(use
Psr16Cache
instead)