Having trouble with newest release R4-5
Reported by Jenna | July 18th, 2008 @ 01:38 PM
Once I installed I'm back to having the old chron job error before I used a patch from someone else (see other post) for the earlier release.
The log is stuck on "running cron job" and the latest processed campaigns are all stuck on "next". (I replaced the feed names with ********.)
Latest Log Entries...
- July 18, 12:25 pm — Running cron job
- July 18, 11:25 am — Running cron job
- July 18, 10:28 am — Running cron job
Latest Processed Campaigns...
1. Next! ********
2. Next! ********
3. Next! ********
4. Next! ********
5. Next! ********
Help!
Comments and changes to this ticket
-
Guillermo Rauch July 18th, 2008 @ 01:48 PM
Question: are your date settings ok ? In Wordpress configuration, does UTC match actual UTC ?
-
Jenna July 18th, 2008 @ 01:56 PM
Right now, I'm making the same patch change to the code (I guess it wasn't updated in R4-5?) that I had to for an earlier release.
Not sure if that will fix it. Will keep you posted.
FILE TO MODIFY...
wpomatic.php
LOOK FOR...
Approximately line 966
REPLACE...
if($unparsed)
$where .= " AND active = 1 AND (frequency + lastactive) < ". current_time('timestamp', true) . " ";
WITH...
if($unparsed)
$where .= " AND active = 1 AND (frequency + UNIX_TIMESTAMP(lastactive)) < ". (current_time('timestamp',1) - ($gmt ? 0 : (get_option('gmt_offset') * 3600))) . " ";
//$where .= " AND active = 1 AND (frequency + lastactive) < ". current_time('timestamp', true) . " ";
-
-
Guillermo Rauch July 18th, 2008 @ 01:59 PM
That doesn't make a lot of sense.
The variable $gmt is not set in that function.
-
Guillermo Rauch July 18th, 2008 @ 02:06 PM
- Milestone cleared.
- State changed from “new” to “resolved”
Sorry, I did not test the change when I made it. My fault.
Solution in RC4-6
-
Jenna July 18th, 2008 @ 02:37 PM
So far, the patch I listed above seems to have worked. It may be worth adding it to the next release.
-
Guillermo Rauch July 18th, 2008 @ 02:57 PM
Yes, it works because the $gmt variable doesn't exist, so it simply substracts the gmt offset in seconds.
Which RC4-6 does directly. Without generating a notice for the undefined $gmt.
Thanks Jenna for your valuable feedback
-
Ricardo Rauch April 15th, 2010 @ 08:50 AM
- Assigned user changed from “Guillermo Rauch” to “Ricardo Rauch”
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create new ticket
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
WP-o-Matic project.
Guillermo Rauch