-
Notifications
You must be signed in to change notification settings - Fork 170
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
APCu statistics #24
Comments
Hey @amenadiel, sorry for the late reply (real life and stupid stuff like that). I am not against the idea at all to include APCu. If you are willing to experiment with and I am totally fine with that. The only thing that might be annoying (at least for now) is that I am currently in the process of fixing some stuff of this project and going through the open tickets. So perhaps it might be a better idea to wait just a bit till this work is done to prevent you from having to do it again. I am planning to release a new version somewhere this week so if you are still up for it that would be cool. |
Sure, I'll warm up for when the time comes. |
I'm ready whenever you decide to push v1.0.0.dev to release. |
Great! There will a merge today of the new design. I will ping you when this is done. |
Sorry for the late reply. Will merge this PR tonight which contains a lot of fixes and new shiny interface. I think I will release it as v1.0.0-rc1 (meaning it should be stable enough to start the work :-) ) |
Warning: Division by zero in /src/OpCacheGUI/Addons/APCUHelper.php on line 147 Latest commit on amenadiel-apcu_tabs branch |
You are right! There was an edge case for the zero hits scenario. I fixed that one in the last commit. Thanks! |
Merged into a apcu-dev branch @ https://github.com/PeeHaa/OpCacheGUI/tree/apcu. ATM working on some refactoring. |
Not sure, if I should open up a seperate ticket, but just tried the latest versionand "APCU status" gives the following error message: Catchable fatal error: Argument 3 passed to OpCacheGUI\APCu\Status::__construct() must implement interface OpCacheGUI\I18n\Translator, array given, called in /cluster/public_html/OpCacheGUI-apcu/template/apcustatus.phtml on line 3 and defined in /cluster/public_html/OpCacheGUI-apcu/src/OpCacheGUI/APCu/Status.php on line 72 I am using php-5.6.5 on latest commit on apcu |
Yeah working out the last issues in the apcu branch before I can release the feature. Currently the apcu branch is a WIP |
Thanks for the unbelieveable fast feedback and again for the great work on OpenCacheGUI. |
Np. Thanks for the kind words. |
The ACPu piece would be awesome. Any timeline on releasing it? |
@WebEndevSnippets Most of the work is already done in the apcu branch. IIRC only need to clean something up and adds nice looking graphs for fragmentation. Once I have closed the roadmap I will have a better idea of a timeframe for you all |
This one is on the edge of being off topic, but several developers that used APC until php 5.4 are now using a combination of OpCache + APCu for user variables.
@PeeHaa, what do you think about the idea of adding an APCu tab, displayed only if
extension_loaded('apcu')
is true? I could start experimenting with styles to match your GUI look & feel.The text was updated successfully, but these errors were encountered: