-
Notifications
You must be signed in to change notification settings - Fork 17
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
Last version of comet cache generates an error #755
Comments
After I renamed the comet cache plugin directory back to the original name and refreshed wp-admin I could activate the plugin again. It recompiled and seems to work again. |
@abiere Thank you for the report. Do you remember which version of Comet Cache you were upgrading from? Was it v160417, v160416, or v160227? |
I'm sorry, but I don't remember. I updated everything every month or so. Is there some way I can check this for you? |
Hmm, I'm afraid not. However, do you have an idea roughly how long ago you did the last update? v160417 and v160416 were released within the past 2 weeks, so if the last time you updated was earlier than that, you must've been running v160227. |
It is a month or more since I've updated, so it must be v160227. There is no harm done. Everything works again. Scared me as hell, but I survived ;) |
When it crashed, were you updating using the built-in Pro Plugin Updater, or were you upgrading in some other way? |
This looks like an issue with the OPcache to me. This exception had to have been thrown by the old version of the software as seen here on line 57 in v160227: https://github.com/websharks/comet-cache/blob/160227/src/includes/classes/AbsBaseAp.php#L57 Now look at these lines where the closures are loaded in the old version of the software. If those files no longer exist (e.g., if you upgraded to the latest release that doesn't come with those files), you will see the exception about a missing closure. This can happen as a result of updating the plugin, but with a stale opcode cache; e.g., the old version of files that existed in both versions of the software are still in memory, but with missing dependencies that those old versions had, because the filesystem itself was updated, and no longer contains the |
Related: #740 |
I was updating with the built in updater. |
Yeah, this is not your fault. Sorry for any confusion. I'm just seeing how we can avoid this in the future and I think Raam is too. The OPcache is something that works internally in a lot of PHP installations and if not properly flushed after an upgrade it can lead to this problem. Thanks for your patience. |
No worries guys :) I was glad I was able to fix it with my nub skills ;) I will patiently await your magic to avoid panic with nubs like me ;) Keep up the good work! |
The last version of Comet Cache generates the following error:
I was only able to fix it by renaming the comet cache plugin directory. I have no clue how github etc works. Please let me know how to fix this.
The text was updated successfully, but these errors were encountered: