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

Bug: Version Salt Filter #522

Closed
jaswrks opened this issue Jul 9, 2015 · 2 comments
Closed

Bug: Version Salt Filter #522

jaswrks opened this issue Jul 9, 2015 · 2 comments
Labels
bug
Milestone

Comments

@jaswrks
Copy link

@jaswrks jaswrks commented Jul 9, 2015

[09-Jul-2015 21:32:45 UTC] PHP Notice:  Undefined variable: this in /home4/wsksdev/public_html/jason/wp-content/plugins/zencache-pro/src/includes/closures/Ac/ObUtils.php on line 145
[09-Jul-2015 21:32:45 UTC] PHP Warning:  get_class() called without object from outside a class in /home4/wsksdev/public_html/jason/wp-content/plugins/zencache-pro/src/includes/closures/Ac/ObUtils.php on line 145

Should be $self instead of $this:
https://github.com/websharks/zencache-pro/blob/feature/59-jaswsinc/src/includes/closures/Ac/ObUtils.php#L145

@jaswrks jaswrks added the bug label Jul 9, 2015
jaswrks pushed a commit to wpsharks/comet-cache-pro that referenced this issue Jul 9, 2015
@raamdev raamdev modified the milestone: Next Release Jul 10, 2015
@raamdev
Copy link
Contributor

@raamdev raamdev commented Jul 10, 2015

Next Pro Release Changelog:

  • Bug Fix: Fixed a bug in the Dynamic Version Salt filter that was generating PHP notices and warnings. See Issue #522.
@raamdev raamdev closed this Jul 10, 2015
@raamdev
Copy link
Contributor

@raamdev raamdev commented Jul 17, 2015

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 (#522).

@wpsharks wpsharks locked and limited conversation to collaborators Jul 17, 2015
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
2 participants