Thread Starter
staze
(@staze)
I attempted adding “$url[‘mod’] = ”” to the initial blog stuff (under “//Add the homepage first”) in xml-sitemap-base.php, but that seems to make no difference.
If I could figure out where the initial date comes from, I might be able to better debug this… could you fill me in on where the blog “last modified date” comes from?
Also, I’ve noticed that wordpress-seo doesn’t give an error if it can’t generate the sitemap due to a permissions error. it just acts like normal, but doesn’t actually write out the new sitemap.xml(.gz).
Thanks!
Thread Starter
staze
(@staze)
AH HA! Typos be damned!
On line 96 of xml-sitemap-base-class.php you have:
$time = time();
This is incorrect, as mysql2date is expecting a mysql statement. It SHOULD read:
$time=”time()”;
What I can’t figure out is how that typo got in there, since the svn change logs don’t seem to indicate you did anything with that line. And if it’s been that way for a while, then what changed? time() is a php function… so really, this never should have worked.
Anyway… fixed!
Thread Starter
staze
(@staze)
looks like the change was made in rev 315637. switched from using the php date() function to using mysql2date, but didn’t change what $time was equal to.
What I still can’t figure out is why I only hit this bug today… I’ve updated my site between this change (and my update of wordpress-seo) and now, yet I only hit it today.
Weird weird weird. Anyway, thanks for the awesome product. I’m just happy I can contribute back a bit. =)
thanks for the investigation =) your solution worked for me, except instead of using the full quotes (“) i had to use the single apostrophe ‘
$time=’time()’;
lord love a duck.
-MB
Thread Starter
staze
(@staze)
mbartizmo,
Good to know. Though it _shouldn’t_ matter in this case, it’s probably safer for the real fix by Yoast to be single quotes.
Also glad to know I’m not the only one that saw this issue. If it weren’t for my site being in Google’s webmaster tools, I would have never known the issue existed (Webmaster Tools was nice enough to warn me that there was a bad date in my sitemap).
staze – mbartizmo,
I had the same problem. $time=’time()’; wonderfully solved the problem. Thanks to both!
Thanks staze & mbartizmo!
I found the issue too after checking with Google’s Webmaster tools.
It’s fixed now.
Thanks for your time staze!
Fixed this in 0.2.3.3, thanks 🙂
Thread Starter
staze
(@staze)
Sweet. Just installed 0.2.3.3.
Thanks!
