Skip to content

feat: add remember() to CacheInterface#9875

Merged
michalsn merged 6 commits intocodeigniter4:4.7from
datamweb:feat/cache-remember-interface
Jan 19, 2026
Merged

feat: add remember() to CacheInterface#9875
michalsn merged 6 commits intocodeigniter4:4.7from
datamweb:feat/cache-remember-interface

Conversation

@datamweb
Copy link
Contributor

@datamweb datamweb commented Jan 8, 2026 •
edited
Loading

Description
The method has already been implemented in BaseHandler and is therefore
available on all built-in cache handlers. Exposing it on the interface
improves IDE support and allows consumers to safely rely on the method
when type-hinting against CacheInterface.

Screenshot 2026-01-08 111152

See: #4107 (comment)

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value (without duplication)
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@paulbalandan paulbalandan added breaking change Pull requests that may break existing functionalities 4.7 labels Jan 8, 2026
michalsn
michalsn reviewed Jan 9, 2026
View reviewed changes
paulbalandan
paulbalandan reviewed Jan 9, 2026
View reviewed changes
@paulbalandan paulbalandan force-pushed the feat/cache-remember-interface branch from 28c56cf to 6349e0c Compare January 19, 2026 07:45
@paulbalandan paulbalandan requested a review from michalsn January 19, 2026 08:18
@michalsn michalsn merged commit 21fbcf6 into codeigniter4:4.7 Jan 19, 2026
50 checks passed
@michalsn
Copy link
Member

michalsn commented Jan 19, 2026

Thank you @datamweb and @paulbalandan

@datamweb datamweb deleted the feat/cache-remember-interface branch January 22, 2026 23:34
@datamweb
Copy link
Contributor Author

datamweb commented Jan 22, 2026

Thanks everyone for pushing this PR forward.
I was offline for about 15 days due to connectivity limitations, so I couldn’t follow up—sorry about that.
Really appreciate the team for moving it ahead and getting it done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@michalsn michalsn michalsn approved these changes

@paulbalandan paulbalandan paulbalandan approved these changes

+1 more reviewer

@neznaika0 neznaika0 neznaika0 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

breaking change Pull requests that may break existing functionalities

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@datamweb @michalsn @neznaika0 @paulbalandan