Changeset 3750

Timestamp:

07/29/2016 07:34:51 PM (9 years ago)

Author:

iandunn

Message:

WordCamp Participation Notifier: Strip trailing whitespace.

File:

• sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-participation-notifier/wordcamp-participation-notifier.php (modified) (18 diffs)

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-participation-notifier/wordcamp-participation-notifier.php

@@ -33 +33 @@
      *       hook when they're ready to send the notification, rather than this plugin having to be aware of (and
      *       coupled to) the internal logic of those plugins.
-     * 
+     *
      * @param string  $new_status
      * @param string  $old_status
@@ -74 +74 @@
      * Updates the activity and associations of a profile when the WordPress.org username on a published speaker
      * or organizer post changes.
-     * 
-     * @todo The handler doesn't support removing activity, but maybe do that here if support is added.
-     * 
+     *
+     * @todo The handler doesn't support removing activity, but maybe do that here if support is added.
+     *
      * @param WP_Post $post
      */
@@ -87 +87 @@
             return;
         }
-        
+
+        // todo change this to use an if/elseif/elseif structure, just to be safe
+
         // A new username was added, so add the activity and association.
         if ( $new_user_id && ! $previous_user_id ) {
@@ -93 +95 @@
             $this->remote_post( self::PROFILES_HANDLER_URL, $this->get_post_association_payload( $post, 'add' ) );
         }
-        
+
         // The username was removed, so remove the association.
         if ( ! $new_user_id && $previous_user_id ) {
@@ -124 +126 @@
             // Get the $user_id from post meta instead of $_POST in case it changed during the unpublish update.
             // This makes sure that the association is removed from the same user that it was originally added to.
-            
+
             $user_id = $this->get_saved_wporg_user_id( $post );
             $this->remote_post( self::PROFILES_HANDLER_URL, $this->get_post_association_payload( $post, 'remove', $user_id ) );
@@ -188 +190 @@
     /**
      * Builds the payload for an activity notification based on a new post
-     * 
+     *
      * @param WP_Post $post
      * @param int     $user_id
@@ -211 +213 @@
                 'wordcamp_id'   => get_current_blog_id(),
                 'wordcamp_name' => get_wordcamp_name(),
-                'wordcamp_date' => empty( $wordcamp->meta['Start Date (YYYY-mm-dd)'][0] ) ? false : date( 'F jS', $wordcamp->meta['Start Date (YYYY-mm-dd)' ][0] ), 
+                'wordcamp_date' => empty( $wordcamp->meta['Start Date (YYYY-mm-dd)'][0] ) ? false : date( 'F jS', $wordcamp->meta['Start Date (YYYY-mm-dd)' ][0] ),
                 'url'           => site_url(),
             );
-    
+
             switch( $post->post_type ) {
                 case 'wcb_speaker':
                     $activity['speaker_id']   = $post->ID;
                 break;
-    
+
                 case 'wcb_organizer':
                     $activity['organizer_id'] = $post->ID;
@@ -243 +245 @@
                     }
                 break;
-    
+
                 default:
                     $activity = false;
@@ -255 +257 @@
     /**
      * Build the payload for an association notification based on a new or updated post
-     * 
+     *
      * @param WP_Post $post
      * @param string  $command 'add' | 'remove'
@@ -263 +265 @@
     protected function get_post_association_payload( $post, $command, $user_id = null ) {
         $association = false;
-        
+
         if ( ! $user_id ) {
             $user_id = $this->get_new_wporg_user_id( $post );
@@ -278 +280 @@
                 'url'           => site_url(),
             );
-            
+
             switch( $post->post_type ) {
                 case 'wcb_speaker':
                     $association['association'] = 'wordcamp-speaker';
                 break;
-    
+
                 case 'wcb_organizer':
                     $association['association'] = 'wordcamp-organizer';
@@ -297 +299 @@
         return apply_filters( 'wpn_post_association_payload', $association, $post, $command, $user_id );
     }
-    
+
     /**
      * Get the current WordPress.org user_id associated with a custom post
@@ -303 +305 @@
      * This is called during the context of a post being updated, so the new username is the one submitted in
      * the $_POST request, or the currently logged in user, as opposed to the user_id saved in the database.
-     * 
+     *
      * @param WP_Post $post
      * @return false|int
@@ -350 +352 @@
      *
      * This reduces the amount of duplicated code in the callers, makes them more readable, and logs errors to aid in debugging
-     * 
+     *
      * @param string $url
      * @param array  $body The value intended to be passed to wp_remote_post() as $args['body']
@@ -360 +362 @@
         if ( $body ) {
             $response = wp_remote_post( $url, array( 'body' => $body ) );
-        
+
             if ( is_wp_error( $response ) ) {
                 $error = sprintf(
@@ -369 +371 @@
             } elseif ( 200 != $response['response']['code'] || 1 != (int) $response['body'] ) {
                 // error_log() has a message limit of 1024 bytes, so we truncate $response['body'] to make sure that $body doesn't get truncated.
-                
+
                 $error = sprintf(
                     'Received HTTP code: %s and body: %s. Request was: %s',
@@ -377 +379 @@
                 );
             }
-            
+
             if ( $error ) {
                 error_log( sprintf( '%s error for %s: %s', __METHOD__, parse_url( site_url(), PHP_URL_HOST ), sanitize_text_field( $error ) ) );
-                
+
                 if ( $to = apply_filters( 'wpn_error_email_addresses', array() ) ) {
                     wp_mail( $to, sprintf( '%s error for %s', __METHOD__, parse_url( site_url(), PHP_URL_HOST ) ), sanitize_text_field( $error ) );
@@ -386 +388 @@
             }
         }
-        
+
         return $response;
     }