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.

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

Backwards compatibility for $GLOBALS['zencache'] is missing #689

Closed
raamdev opened this issue Feb 26, 2016 · 2 comments
Closed

Backwards compatibility for $GLOBALS['zencache'] is missing #689

raamdev opened this issue Feb 26, 2016 · 2 comments
Milestone

Comments

@raamdev
Copy link
Contributor

@raamdev raamdev commented Feb 26, 2016

Comet Cache does not contain proper backwards compatibility for $GLOBALS['zencache'], which was often recommended for making various API calls.

@jaswsinc writes...

it looks like we don't have any BC routines that fill the $GLOBALS['zencache'] global. I think this was overlooked because that global did not exist in Quick Cache if I recall, so we didn't need that BC routine in ZenCache before. Now we do. For instance, in this file it would help to point $GLOBALS['zencache'] to $GLOBALS[GLOBAL_NS] to improve BC. https://github.com/websharks/comet-cache-pro/blob/000000-dev/src/includes/plugin.php#L15

@raamdev
Copy link
Contributor Author

@raamdev raamdev commented Feb 27, 2016

Next Release Changelog:

  • Bug Fix: Fixed a ZenCache Backwards Compatibility bug that was preventing calls to $GLOBALS['zencache'] from working properly with Comet Cache. See Issue #689
@raamdev raamdev closed this Feb 27, 2016
@raamdev
Copy link
Contributor Author

@raamdev raamdev commented Feb 27, 2016

Comet Cache v160227 has been released and includes changes from this GitHub Issue. See the v160227 announcement for further details.


This issue will now be locked to further updates. If you have something to add related to this GitHub Issue, please open a new GitHub Issue and reference this one (#689).

@wpsharks wpsharks locked and limited conversation to collaborators Feb 27, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant