Changeset 3757 for trunk/bp-messages/bp-messages-notifications.php

Timestamp:

01/20/2011 10:53:49 AM (14 years ago)

Author:

johnjamesjacoby

Message:

Second pass at huge component refactor.

Move component files out of root directory and append '-loader' to the file names. Split those files up into smaller pieces.

Abstract 'settings' component out of 'core' and into its own component for future extension, including new template files in bp-default.

Append '-sa' to bbPress bridge file for 'Stand Alone' compatibility, to make room for bbPress plugin compatibility layer.

Various bug fixes through-out all components. Various code clean-up and documentation through-out all components.

Still more refactoring to do.

File:

• trunk/bp-messages/bp-messages-notifications.php (modified) (2 diffs)

trunk/bp-messages/bp-messages-notifications.php

@@ -3 +3 @@
 function messages_notification_new_message( $args ) {
     global $bp;
+
     extract($args);
 
-    $sender_name = bp_core_get_user_displayname( $sender_id );
+    $email_subject = $email_content = $args = '';
+    $sender_name   = bp_core_get_user_displayname( $sender_id );
 
     foreach( $recipients as $recipient ) {
-        if ( $sender_id == $recipient->user_id || 'no' == get_user_meta( $recipient->user_id, 'notification_messages_new_message', true ) ) continue;
+        if ( $sender_id == $recipient->user_id || 'no' == get_user_meta( $recipient->user_id, 'notification_messages_new_message', true ) )
+            continue;
 
-        $ud = get_userdata( $recipient->user_id );
-        $message_link = bp_core_get_user_domain( $recipient->user_id ) . BP_MESSAGES_SLUG .'/';
+        // User data and links
+        $ud            = get_userdata( $recipient->user_id );
+        $message_link  = bp_core_get_user_domain( $recipient->user_id ) . BP_MESSAGES_SLUG .'/';
         $settings_link = bp_core_get_user_domain( $recipient->user_id ) .  BP_SETTINGS_SLUG . '/notifications/';
 
-        $sender_name = stripslashes( $sender_name );
-        $subject = stripslashes( wp_filter_kses( $subject ) );
-        $content = stripslashes( wp_filter_kses( $content ) );
+        // Sender info
+        $sender_name   = stripslashes( $sender_name );
+        $subject       = stripslashes( wp_filter_kses( $subject ) );
+        $content       = stripslashes( wp_filter_kses( $content ) );
 
         // Set up and send the message
@@ -37 +42 @@
         $email_content .= sprintf( __( 'To disable these notifications please log in and go to: %s', 'buddypress' ), $settings_link );
 
-        /* Send the message */
+        // Send the message
         $email_to = apply_filters( 'messages_notification_new_message_to', $email_to );
         $email_subject = apply_filters( 'messages_notification_new_message_subject', $email_subject, $sender_name );