Changeset 788 for sites/trunk/wordcamp.org/public_html/wp-content/plugins/wc-post-types/wc-post-types.php

Timestamp:

08/13/2014 08:14:50 PM (12 years ago)

Author:

iandunn

Message:

WordCamp Post Types: Only specify speaker IDs if track(s) requested.

r779 added support for the track parameter, but introduced a bug where speakers would not show up in the list
if all tracks were requested and the speaker wasn't assigned to any session yet.

File:

• sites/trunk/wordcamp.org/public_html/wp-content/plugins/wc-post-types/wc-post-types.php (modified) (2 diffs)

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wc-post-types/wc-post-types.php

@@ -318 +318 @@
             'post_type' => 'wcb_speaker',
             'posts_per_page' => intval( $attr['posts_per_page'] ),
-            'post__in' => $speaker_ids,
             'orderby' => $attr['orderby'],
             'order' => $attr['order'],
         );
 
+        if ( 'all' != $attr['track'] ) {
+            $speaker_args['post__in'] = $speaker_ids;
+        }
+
         $speakers = new WP_Query( $speaker_args );
 
@@ -338 +341 @@
                 <?php
                     $speaker_classes = array( 'wcorg-speaker', 'wcorg-speaker-' . sanitize_html_class( $post->post_name ) );
-                    foreach ( $speakers_tracks[ get_the_ID() ] as $track ) {
-                        $speaker_classes[] = sanitize_html_class( 'wcorg-track-' . $track );
+
+                    if ( isset( $speakers_tracks[ get_the_ID() ] ) ) {
+                        foreach ( $speakers_tracks[ get_the_ID() ] as $track ) {
+                            $speaker_classes[] = sanitize_html_class( 'wcorg-track-' . $track );
+                        }
                     }
                 ?>