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

Notice: Undefined variable: qp in feedwordpress/syndicatedlink.class.php on line 747 #32

Closed
zoul0813 opened this issue Aug 22, 2014 · 5 comments

Comments

@zoul0813
Copy link
Contributor

When using "update checked" manually, I receive the following error repeated numerous times:

Notice: Undefined variable: qp in .../feedwordpress/syndicatedlink.class.php on line 747

I checked the code, and it appears the $gp is only defined inside of the conditional check

// line 726
if (!is_null($uri) and strlen($uri) > 0 and $params['add_params']) :

$gp should be declared outside of the block, with a default value (null?) and then set inside this conditional to prevent this error.

This happens on one of the sites I use FWP on, but not another.

@drdrstudio
Copy link

Thanks ! Am getting the same error. What would that code look like?

Just placing a

$qp = NULL;

Before the line 747 that is looking for that var?

@zoul0813
Copy link
Contributor Author

before line 726

@hughiemolloy
Copy link

Same problem. I put
$qp = null;
before the if loop on line 726.

Can this be fixed in the next update?

@McBoof
Copy link

McBoof commented May 13, 2015

Oh pretty please include this in a release soon. FeedWordPress is the sweetest plugin ever, but this tiny buglet causes me and clients major upgrade headaches :-)

@radgeek
Copy link
Owner

radgeek commented May 14, 2015

Hey y'all,

Thanks for the heads-up on this issue. I have just pushed a fix for this to the main branch (7d3aeae), which you can get with a git pull. I will be putting out a release through the Subversion repository that encompasses this change probably later today.

Cheers,
-C

@radgeek radgeek closed this as completed May 14, 2015
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

5 participants