[#16606] - feat: added events and events manager to storage and cache classes

phalcon / cphalcon

High performance, full-stack PHP framework delivered as a C extension.

https://phalcon.io

BSD 3-Clause "New" or "Revised" License

10.79k stars 1.96k forks source link

[#16606] - feat: added events and events manager to storage and cache classes #16607

Closed noone-silent closed 5 months ago

noone-silent commented 5 months ago

Hello!

Type: new feature
Link to issue: https://github.com/phalcon/cphalcon/issues/16606

In raising this pull request, I confirm the following:

[X] I have read and understood the Contributing Guidelines
[X] I have checked that another pull request for this purpose does not exist
[X] I wrote some tests for this PR
[X] I have updated the relevant CHANGELOG
[X] I have created a PR for the documentation about this change

Small description of change: See title

Thanks

noone-silent commented 5 months ago

This is the first implementation. Currently we have to add it to every class. I will refactor it later so that Phalcon\Storage\AbstractAdapter holds all the methods (decrement, increment, etc) and calls the doXXX() method of the class, like it is currently with the get() method.

New implementations of a Cache class would so have automatically all events.

niden commented 5 months ago

Thank you @noone-silent