Context Navigation

← Previous Change
Next Change →

rss.php

Timestamp:

03/17/2016 05:39:02 AM (10 years ago)

Author:

dd32

Message:

Themes Directory: Open-source the pre_get_posts logic used by the Themes Directory & add logic to handle the domain not matching the home_url.

This allows us to remove the 'placeholder' sites we use for the themes directory on internationalized sites (xx.wordpress.org/themes), and makes the query logic more transparent.

See #1630 (which this is the first part for)
See #1618 (Some work will still be needed to correctly trim the description)

File:

: 1 edited

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-themes/rss.php (modified) (2 diffs)

Legend:

: Unmodified
: Added
: Removed

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-themes/rss.php

-              r1489
+              r2755
         <?php
+        $themes = wporg_themes_get_themes_for_query();
+        foreach ( $themes['themes'] as $theme ) :
+        while ( have_posts() ) :
+                the_post();
+                $theme = wporg_themes_theme_information( $post->post_name );
         ?>
         <item>
 …
                 <content:encoded><![CDATA[<?php echo esc_html( $theme->description ); ?>]]></content:encoded>
         </item>
         <?php endforeach; ?>
+        <?php endwhile; ?>
 </channel>
 </rss>

Note: See TracChangeset for help on using the changeset viewer.

Making WordPress.org

Context Navigation

Changeset 2755 for sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-themes/rss.php

Legend:

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-themes/rss.php

Download in other formats: