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.

Sign up for GitHub

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

Jump to bottom

[NFR]: Events for Storage and Cache classes #16606

Closed
noone-silent opened this issue Jun 7, 2024 · 1 comment
Closed

[NFR]: Events for Storage and Cache classes #16606

noone-silent opened this issue Jun 7, 2024 · 1 comment
Labels
5.0 The issues we want to solve in the 5.0 release new feature request Planned Feature or New Feature Request

Comments

@noone-silent
Copy link
Contributor

Describe the solution you'd like
We have different events for example in db, views, dispatcher, etc. For storage or cache these events are missing.

Describe alternatives you've considered
There is no alternative.
@noone-silent noone-silent added the new feature request Planned Feature or New Feature Request label Jun 7, 2024
noone-silent added a commit to noone-silent/cphalcon that referenced this issue Jun 7, 2024
@noone-silent
[phalcon#16606] - feat: added events and events manager to storage an…
3c2d147 
…d cache classes.
This was referenced Jun 7, 2024
Merged
Open
noone-silent added a commit to noone-silent/cphalcon that referenced this issue Jun 7, 2024
@noone-silent
[phalcon#16606] - fix: added missing EventsAwareInterface
566c600
noone-silent added a commit to noone-silent/cphalcon that referenced this issue Jun 8, 2024
@noone-silent
[phalcon#16606] - refactor: changed fire method to request key/keys.
09a6657
niden added a commit that referenced this issue Jun 8, 2024
@niden
Merge pull request #16607 from noone-silent/T16606-add-events-to-stor…
fbb8318 
…age-and-cache

[#16606] - feat: added events and events manager to storage and cache classes
@niden
Copy link
Member

niden commented Jun 8, 2024

This has been resolved with #16607

Thank you again @noone-silent for the work!

@niden niden added the 5.0 The issues we want to solve in the 5.0 release label Jun 8, 2024
@niden niden added this to Phalcon v5 Jun 8, 2024
@niden niden moved this to Implemented in Phalcon v5 Jun 8, 2024
@niden niden closed this as completed Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
5.0 The issues we want to solve in the 5.0 release new feature request Planned Feature or New Feature Request
Projects
Phalcon v5
Status: Implemented
Milestone
No milestone
Development

No branches or pull requests
2 participants
@niden @noone-silent