[buddypress-trac] [BuddyPress] #3622: bp_get_sitewide_activity_feed_link()

buddypress-trac at lists.automattic.com buddypress-trac at lists.automattic.com
Tue Sep 27 05:26:31 UTC 2011


#3622: bp_get_sitewide_activity_feed_link()
-------------------------------------+--------------------
 Reporter:  webraket                 |       Owner:
     Type:  defect                   |      Status:  new
 Priority:  normal                   |   Milestone:  1.5.1
Component:  Activity                 |     Version:  1.5
 Severity:  major                    |  Resolution:
 Keywords:  has-patch needs-testing  |
-------------------------------------+--------------------

Comment (by cnorris23):

 @webraket bp_get_sitewide_activity_feed_link() is broken, just called to
 early, as was more or less discussed already.

 {{{
 <?php
 class BP_PubSubhubbub {

         public function __construct(){

                 add_action('bp_setup_globals',
 array(&$this,'setup_globals'));
                 add_action('bp_activity_sitewide_feed_head',
 array(&$this,'add_rss_link_tag'));
                 add_action('publish_post', array(&$this,'publish_to_hub'),
 999);

         }

         function setup_globals() {

                 $this->hub_urls[] = "http://pubsubhubbub.appspot.com";
                 $this->feed_urls[] = bp_get_sitewide_activity_feed_link();

         }

         function add_rss_link_tag() {
                 //
         }

         function publish_to_hub($post_id)  {
                 //
         }

     private function http_post($url, $post_string) {
                 //
     }

 }

 function Init_BP_PubSubHubbub(){
         $oBP_PubSubHubbub = new BP_PubSubHubbub;
 }
 add_action('bp_include', 'Init_BP_PubSubHubbub');
 ?>
 }}}

 Based on the above code, give bp_get_sitewide_activity_feed_link() another
 try, without any of the patches from this ticket applied. My guess is that
 bp_get_sitewide_activity_feed_link() worked just fine, but judging from
 boone's patches, it's possible we inadvertently found bug.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/3622#comment:16>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list