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

Undefined Variable Notice - framework.php line 876 #36

Closed
corradomatt opened this issue Sep 16, 2013 · 3 comments
Closed

Undefined Variable Notice - framework.php line 876 #36

corradomatt opened this issue Sep 16, 2013 · 3 comments

Comments

@corradomatt
Copy link
Contributor

The th variable is setting an undefined notice when in WP debug mode.

This is due to the initial variable define being placed inside of an if statement that isn't being called all the time. If you move the following on line 824...

$th = '';

...to above and outside the current if statement scope, the error is resolved (move to around line 819). There is probably a better way to write all of this but I'm not sure exactly how it would be done.

@dovy
Copy link
Contributor

dovy commented Sep 16, 2013

Can you give me your config file via gist? I don't get that error normally. I only build in debug mode. I'd like to resolve this case asap. ;)

@corradomatt
Copy link
Contributor Author

hmmm...then it may be my issue alone. I'm in the process of rebuilding my config file which started as a Redux config from before the merger. I'll finish my merge and let you know if the problem still persists. Thanks!

@dovy
Copy link
Contributor

dovy commented Sep 16, 2013

No problem. Please re-open this if it still exists. Sorry, things did have to change a little from the old redux. ;) I know you'll be happy with the improvements.

@dovy dovy closed this as completed Sep 16, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants