Ticket #41112: 41112.2.diff

src/wp-admin/includes/class-wp-community-events.php

@@ -416 +416 @@
         }
 
         /**
-         * Discards expired events, and reduces the remaining list.
+         * Prepares the event list for presentation.
          *
+         * Discards expired events, and sticks a WordCamp to the final list if it
+         * doesn't naturally appear.
+         *
+         * @access protected
          * @since 4.8.0
+         * @since 5.0.0 Stick a WordCamp to the final list.
          *
          * @param  array $response_body The response body which contains the events.
          * @return array The response body with events trimmed.
          */
         protected function trim_events( $response_body ) {
                 if ( isset( $response_body['events'] ) ) {
+                        $wordcamps = array();
                         $current_timestamp = current_time( 'timestamp' );
 
                         foreach ( $response_body['events'] as $key => $event ) {
-                                // Skip WordCamps, because they might be multi-day events.
-                                if ( 'meetup' !== $event['type'] ) {
-                                        continue;
-                                }
+                                if ( 'wordcamp' === $event['type'] ) {
+                                        // Store WordCamps so that one can be pinned later.
+                                        $wordcamps[] = $event;
 
-                                $event_timestamp = strtotime( $event['date'] );
+                                        // Don't unset WordCamps that are past their start date, because they might be multi-day events.
+                                } else {
+                                        $event_timestamp = strtotime( $event['date'] );
 
-                                if ( $current_timestamp > $event_timestamp && ( $current_timestamp - $event_timestamp ) > DAY_IN_SECONDS ) {
-                                        unset( $response_body['events'][ $key ] );
+                                        if ( $current_timestamp > $event_timestamp && ( $current_timestamp - $event_timestamp ) > DAY_IN_SECONDS ) {
+                                                unset( $response_body['events'][ $key ] );
+                                        }
                                 }
                         }
 
                         $response_body['events'] = array_slice( $response_body['events'], 0, 3 );
+                        $trimmed_event_types     = array_column( $response_body['events'], 'type' );
+
+                        // Make sure the soonest upcoming WordCamps is pinned in the list.
+                        if ( ! in_array( 'wordcamp', $trimmed_event_types ) && $wordcamps ) {
+                                array_pop( $response_body['events'] );
+                                array_push( $response_body['events'], $wordcamps[0] );
+                        }
                 }
 
                 return $response_body;

tests/phpunit/tests/admin/includesCommunityEvents.php

@@ -258 +258 @@
                 );
         }
 
+
+        /**
+         * Test: get_events() should return the events with the WordCamp pinned in the prepared list.
+         *
+         * @since 5.0.0
+         */
+        public function test_get_events_pin_wordcamp() {
+                add_filter( 'pre_http_request', array( $this, '_http_request_valid_response_unpinned_wordcamp' ) );
+
+                $response_body = $this->instance->get_events();
+
+                /*
+                 * San Diego was at position 3 in the mock API response, but pinning puts it at position 2,
+                 * so that it remains in the list.
+                 */
+                $this->assertEquals( $response_body['events'][2]['title'], 'WordCamp San Diego' );
+
+                remove_filter( 'pre_http_request', array( $this, '_http_request_valid_response_unpinned_wordcamp' ) );
+        }
+
+        /**
+         * Simulates a valid HTTP response where a WordCamp needs to be pinned higher than it's default position.
+         *
+         * @since 5.0.0
+         *
+         * @return array A mock HTTP response.
+         */
+        public function _http_request_valid_response_unpinned_wordcamp() {
+                return array(
+                        'headers'  => '',
+                        'response' => array( 'code' => 200 ),
+                        'cookies'  => '',
+                        'filename' => '',
+                        'body'     => wp_json_encode(
+                                array(
+                                        'location' => $this->get_user_location(),
+                                        'events'   => array(
+                                                array(
+                                                        'type'       => 'meetup',
+                                                        'title'      => 'Flexbox + CSS Grid: Magic for Responsive Layouts',
+                                                        'url'        => 'https://www.meetup.com/Eastbay-WordPress-Meetup/events/236031233/',
+                                                        'meetup'     => 'The East Bay WordPress Meetup Group',
+                                                        'meetup_url' => 'https://www.meetup.com/Eastbay-WordPress-Meetup/',
+                                                        'date'       => date( 'Y-m-d H:i:s', strtotime( 'next Sunday 1pm' ) ),
+                                                        'location'   => array(
+                                                                'location'  => 'Oakland, CA, USA',
+                                                                'country'   => 'us',
+                                                                'latitude'  => 37.808453,
+                                                                'longitude' => -122.26593,
+                                                        ),
+                                                ),
+                                                array(
+                                                        'type'       => 'meetup',
+                                                        'title'      => 'Part 3- Site Maintenance - Tools to Make It Easy',
+                                                        'url'        => 'https://www.meetup.com/Wordpress-Bay-Area-CA-Foothills/events/237706839/',
+                                                        'meetup'     => 'WordPress Bay Area Foothills Group',
+                                                        'meetup_url' => 'https://www.meetup.com/Wordpress-Bay-Area-CA-Foothills/',
+                                                        'date'       => date( 'Y-m-d H:i:s', strtotime( 'next Wednesday 1:30pm' ) ),
+                                                        'location'   => array(
+                                                                'location'  => 'Milpitas, CA, USA',
+                                                                'country'   => 'us',
+                                                                'latitude'  => 37.432813,
+                                                                'longitude' => -121.907095,
+                                                        ),
+                                                ),
+                                                array(
+                                                        'type'       => 'meetup',
+                                                        'title'      => 'WordPress Q&A',
+                                                        'url'        => 'https://www.meetup.com/sanjosewp/events/245419844/',
+                                                        'meetup'     => 'The San Jose WordPress Meetup',
+                                                        'meetup_url' => 'https://www.meetup.com/sanjosewp/',
+                                                        'date'       => date( 'Y-m-d H:i:s', strtotime( 'next Thursday 5:30pm' ) ),
+                                                        'location'   => array(
+                                                                'location'  => 'Milpitas, CA, USA',
+                                                                'country'   => 'us',
+                                                                'latitude'  => 37.244194,
+                                                                'longitude' => -121.889313,
+                                                        ),
+                                                ),
+                                                array(
+                                                        'type'       => 'wordcamp',
+                                                        'title'      => 'WordCamp San Diego',
+                                                        'url'        => 'https://2018.sandiego.wordcamp.org',
+                                                        'meetup'     => null,
+                                                        'meetup_url' => null,
+                                                        'date'       => date( 'Y-m-d H:i:s', strtotime( 'next Saturday' ) ),
+                                                        'location'   => array(
+                                                                'location'  => 'San Diego, CA',
+                                                                'country'   => 'US',
+                                                                'latitude'  =>  32.7220419,
+                                                                'longitude' => -117.1534513,
+                                                        ),
+                                                ),
+                                        ),
+                                )
+                        ),
+                );
+        }
+
         /**
          * Test that get_unsafe_client_ip() properly anonymizes all possible address formats
          *