Changeset 6985

Timestamp:

02/22/2008 07:59:12 PM (17 years ago)

Author:

ryan

Message:

Make sure post has future status before publishing from cron. Props hailin. fixes #5801

Location:

trunk/wp-includes

Files:

• default-filters.php (modified) (1 diff)
• post.php (modified) (1 diff)

trunk/wp-includes/default-filters.php

@@ -161 +161 @@
 add_action('wp_head', 'wlwmanifest_link');
 add_action('wp_head', 'locale_stylesheet');
-add_action('publish_future_post', 'wp_publish_post', 10, 1);
+add_action('publish_future_post', 'check_and_publish_future_post', 10, 1);
 add_action('wp_head', 'noindex', 1);
 add_action('wp_head', 'wp_print_scripts');

trunk/wp-includes/post.php

@@ -1394 +1394 @@
 }
 
+/**
+ * check_and_publish_future_post() - check to make sure post has correct status before
+ * passing it on to be published. Invoked by cron 'publish_future_post' event
+ * This safeguard prevents cron from publishing drafts, etc. 
+ * 
+ * {@internal Missing Long Description}}
+ *
+ * @package WordPress
+ * @subpackage Post
+ * @since 2.5
+ * @uses $wpdb
+ *
+ * @param int $post_id Post ID
+ * @return int|null {@internal Missing Description}}
+ */
+function check_and_publish_future_post($post_id) {
+    
+    $post = get_post($post_id);
+
+    if ( empty($post) )
+        return;
+
+    if ( 'future' != $post->post_status )
+        return;
+
+    return wp_publish_post($post_id); 
+}
+
 function wp_add_post_tags($post_id = 0, $tags = '') {
     return wp_set_post_tags($post_id, $tags, true);