Future cards – again

Thanks for this hint. I fixed it in the next release which will be published today.

Cheers

Merry Christmas ??

Last minute I noticed that correct future cards were no longer sending either. So I installed WP crontrol and saw no wp greet entries.

Updated to v4.6 and future card created a cron event and sent the card ??

I do have a number of cards in the DB that failed to send, how do I get them sent? I suspect setting a future_send date/time in the DB is not enough and I must create cron events for them? And do I need to include each recipient email in the cron event?

Yes, you are wright … mhm… well, there is no such function at the moment.

What do you think?
Will a reschedule function solve the problem?
Clicking on it would delte all wp-cron events and add new events for every card not sent. Maybe this would be a nice feature to handle situations where for some reason the events missed to send the card.

I just developed a beta of the reschedule feature.

Now, I need testers. ??
Would you join in?

Sure, why not. How do I get it?

btw, I assume it will only delete wp greet cron events and leave the others alone..

Well, I can send it to you. I think I have got your email, haven’t I?
Yes, the reschedule routine will only delete the wpgreet events.

Ok, please send it and I’ll check it out.

on its way

I tried it on my localhost, using a DB entry with card_sent, card_fetched and future_send values = all zeroes, and fetchuntil being a future date. I edited future_send to some hours ago, hoping this would be correct values for a card that failed to send.

Then I ran the rescheduler, and it changed the confirmuntil field. But it doesn’t send the email. Sending the mail is what we want for ecards that failed to send, right?

Localhost is connecting to my live smtp server and I confirmed it working by sending another card.

btw, I was hoping it would create a cron entry for each (failed) card an hour or so in the future, allowing me to delete those entries I do not want rescheduled.

the reschedule feature should
– delete all cron entries from wpgreet
– add new entries for cards with future_send not 0 and card_sent = 0

so you do not have to delete entries by yourself

you have to call a page after you ran the rescheduler to take effect of the cron entries.

ahhhhhhhhhhhh…sorry there is a typo in wpg_admin_reschedule.php

change

$sql = "select * from " . $wpdb->prefix . "wpgreet_cards where future_send<>'0000-00-00 00:00:00'
and card_sent<>'0000-00-00 00:00:00';";

to

$sql = "select * from " . $wpdb->prefix . "wpgreet_cards where future_send<>'0000-00-00 00:00:00'
and card_sent='0000-00-00 00:00:00';";

btw. if the futuresend date is in the past and a cron event is added it should be executed immediately..this should bring what you want

Ok understood.

Re deleting I meant preventing specific failed cards from resending… like a 2 month old happy birthday card ?? But I see I can prevent those by setting a card_sent date for those cards.

Anyway, I updated the code… and it’s still not sending. The array for the cron event looks different, maybe that’s a bug?

V4.5 the first string is an email address. With 4.7 the first string contains a number. There’s also a “null” string in there.

ouch… should have tested the code before releasing it ??