Skip to content
This repository has been archived by the owner on Feb 23, 2019. It is now read-only.

Debug Mode Not Working #405

Merged
merged 1 commit into from
Feb 21, 2017
Merged

Debug Mode Not Working #405

merged 1 commit into from
Feb 21, 2017

Conversation

amiga-500
Copy link
Collaborator

The Debug Mode feature is a great way to help get more information when trying to figure out why a page isn't working. However, there were a few issues:

  1. When using CDN or Minify, and enabling their respective debugging option, those specific debug information summary outputs did not display in the page footer.

  2. If a user had opted to support W3TC (via the admin popup, a string can get assigned to common.support or a boolean of true can be assigned to common.tweeted) and then decided to use the debug mode, no debug information would show in the footer for any caching feature.

    When supporting W3TC the intention is to only hide the: Performance optimized by W3 Total Cache. Learn more: http://www.w3-edge.com/products/ text display line that appears in page footers for users who have not opted into supporting W3TC, but by doing so it incorrectly also took away the debug information entirely (if debug mode was used) for those who did opt-in.

Both problems outlined above are now fixed in this patch. :octocat:

@amiga-500 amiga-500 merged commit 9eb5250 into v0.9.5.x Feb 21, 2017
@amiga-500 amiga-500 deleted the debug branch February 21, 2017 00:39
amiga-500 added a commit that referenced this pull request Feb 21, 2017
This is related to #405 .. somehow i forgot to include this changed file
too when pushing the others so here it is now.
amiga-500 added a commit that referenced this pull request Feb 21, 2017
This is related to #405 .. somehow i forgot to include this changed file too when pushing the others so here it is now.
szepeviktor pushed a commit that referenced this pull request May 5, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant