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

v160417: PHP Fatal error: Undefined class constant 'CACHE_PATH_NO_PATH_INDEX' #752

Closed
raamdev opened this issue Apr 26, 2016 · 3 comments
Closed
Milestone

Comments

@raamdev
Copy link
Contributor

@raamdev raamdev commented Apr 26, 2016

Reported in an private ticket, this appears to be a similar problem as the one that was fixed in v160417:

[Mon Apr 25 17:31:25.095675 2016] [proxy_fcgi:error] [pid 22938] [client 130.223.32.47:17867] AH01071: Got error 'PHP message: PHP Fatal error: Undefined class constant 'CACHE_PATH_NO_PATH_INDEX' in /var/www/html/example/wordpress/wp-content/plugins/comet-cache-pro/src/includes/classes/DirStats.php on line 574\n', referer: http://example.com/wp-admin/network/admin.php?page=comet_cache-stats

It would appear that we need to add an implements Interfaces\Shared\CachePathConsts to the end of this line.

I have not yet reproduced the issue myself, but reviewing DirStats.php and its usage of Cache Path Constants tells me there is a bug there.

@raamdev raamdev added this to the Next Release milestone Apr 26, 2016
@jaswrks
Copy link

@jaswrks jaswrks commented Apr 26, 2016

I concur. My suggestion would be to implement those constants at the base class layer.

On this line add:

 implements Interfaces\Shared\NcDebugConsts, Interfaces\Shared\CachePathConsts

That way those constants are available to all classes that extend the base, which I believe is all of the classes that we have. It looks like we did that here for AbsBaseApp, but we should remove them from that class and implement them in the absolute base class, which is AbsBase.

@raamdev
Copy link
Contributor Author

@raamdev raamdev commented May 11, 2016

Next Release Changelog:

  • Bug Fix: Fixed a bug that, in some scenarios, resulted in "PHP Fatal error: Undefined class constant 'CACHE_PATH_NO_PATH_INDEX'". This also affected the Cache Statistics feature, resulting in a blank panel when hovering over the Cache Stats button in the menu bar. See Issue #752.
@raamdev
Copy link
Contributor Author

@raamdev raamdev commented May 21, 2016

Comet Cache v160521 has been released and includes changes from this GitHub Issue. See the v160521 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 (#752).

@wpsharks wpsharks locked and limited conversation to collaborators May 21, 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
2 participants