Changeset 690769

Timestamp:

04/02/2013 07:35:35 PM (13 years ago)

Author:

gibrown

Message:

Notifications: add the post types to sync after WP init.

• custom post types need to be registered so we can trigger syncing them
• don't use get_post_types() in a loop since that causes multiple calls.

File:

• jetpack/trunk/modules/notes.php (modified) (2 diffs)

jetpack/trunk/modules/notes.php

@@ -44 +44 @@
 
         add_action( 'init', array( &$this, 'action_init' ) );
-
-        //post types that support comments
-        $filt_post_types = array();
-        foreach ( get_post_types() as $post_type ) {
-            if ( post_type_supports( $post_type, 'comments' ) ) {
-                $filt_post_types[] = $post_type;
-            }
-        }
-
-        Jetpack_Sync::sync_posts( __FILE__, array(
-            'post_types' => $filt_post_types,
-            'post_stati' => array( 'publish' ),
-        ) );
-        Jetpack_Sync::sync_comments( __FILE__, array(
-            'post_types' => $filt_post_types,
-            'post_stati' => array( 'publish' ),
-            'comment_stati' => array( 'approve', 'approved', '1', 'hold', 'unapproved', 'unapprove', '0', 'spam', 'trash' ),
-        ) );
     }
 
@@ -106 +88 @@
 
     function action_init() {
+        //syncing must wait until after init so
+        //post types that support comments
+        $filt_post_types = array();
+        $all_post_types = get_post_types();
+        foreach ( $all_post_types as $post_type ) {
+            if ( post_type_supports( $post_type, 'comments' ) ) {
+                $filt_post_types[] = $post_type;
+            }
+        }
+
+        Jetpack_Sync::sync_posts( __FILE__, array(
+            'post_types' => $filt_post_types,
+            'post_stati' => array( 'publish' ),
+        ) );
+        Jetpack_Sync::sync_comments( __FILE__, array(
+            'post_types' => $filt_post_types,
+            'post_stati' => array( 'publish' ),
+            'comment_stati' => array( 'approve', 'approved', '1', 'hold', 'unapproved', 'unapprove', '0', 'spam', 'trash' ),
+        ) );
+
         if ( defined( 'DOING_AJAX' ) && DOING_AJAX )
             return;