Jonathan Goldford

I’m glad to hear the code was helpful for everyone. @kilozwo, let me know if there is anything else I can do to help.

As far as building out the plugin, the only other big suggestion I’d have is adding a very simple settings page that allows people to adjust the content of the email, as well as the from name and email address. It would also be awesome if you could send a test email.

Thanks. I noticed that too. If you look at my post now I linked to a gist, which is much easier to read.

Awesome. Thanks a ton for the help. This should keep us from showing old events that have already passed.

The other issue we were concerned about was if someone updates the title of an event, which happens regularly. This isn’t then reflected in the stored feed item. This might also be an issue if the date is updated as well. To deal with this we added the following code in functions.php:


<?php
add_action( 'wprss_fetch_all_feeds_hook', 'wi_delete_all_feed_items', 1 ); //Priority 1 so feed items are all deleted before they're fetched.
function wi_delete_all_feed_items(){
$args = array(
'post_type' => 'wprss_feed_item',
'post_status' => array( 'publish', 'future' ),
'cache_results' => false, // Disable caching, used for one-off queries
'fields' => 'ids', // Returns post IDs only
'posts_per_page' => -1,
);
$feed_item_ids = get_posts( $args );
foreach( $feed_item_ids as $feed_item_id ) {
$purge = wp_delete_post( $feed_item_id, true ); // delete the feed item, skipping trash
}
wp_reset_postdata();
wprss_log( sprintf( 'All feed items deleted: %1$d', count($feed_item_ids) ), __FUNCTION__, WPRSS_LOG_LEVEL_INFO );
}
?>


We adapted this from a function within the plugin and it deletes all of the feed items before every fetch, ensuring the events are always accurate.

Aside from the concern of running this too frequently, do you see any issues with this approach? I think we’re almost there so thanks again.

Jonathan

Hi, I wanted to follow up on my last response to see if you all have had a chance to look at it.

Also, I talked with our client and they’re happy to pay for support if that would allow us to get a faster response.

Thanks a ton and I hope you’re having a good week.

Haha. Sorry about the guise. That’s our company account. And I appreciate your explanation of how the plugin saves feed items.

Basically, when wp_query is set to include a ‘post_status’ of both ‘future’ and ‘publish’ posts, the plugin doesn’t actually remove old events (those that are in the past) even though they are no longer contained in the actual RSS feed itself.

It seems like what we need is way to force the plugin to check the feed more frequently and remove posts that no longer exist in the feed. Is that possible? Was my explanation a little more clear this time?

Thanks again for the help.

Thanks for getting back to me Miguel. Here is the URL of the feed source we’re trying to pull from:

http://jewishinstlouis.org/calendar/category/jfed/feed/

I set the option to delete feed items older than one day, but unfortunately it’s still showing events in the past.

Do you have any thoughts on why that would be?

Thanks for getting back to me Miguel. You’re exactly right, we simply need the order reversed.