Changeset 1337

Timestamp:

02/26/2015 10:05:12 PM (10 years ago)

Author:

obenland

Message:

WP.org Themes: Display parent theme information on the PHP side.

With the switch away from using the API, we have to manually set the required
fields for our requests. Since wp_parse_args() only parses one level deep,
this is a bit more verbose than usual.

See #745.

Location:

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

Files:

• content-single.php (modified) (1 diff)
• functions.php (modified) (2 diffs)

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

@@ -23 +23 @@
             </div>
 
-            <?php if ( ! empty( get_post()->parent ) ) : ?>
+            <?php
+                if ( ! empty( get_post()->post_parent ) ) :
+                    $parent = get_post( get_post()->post_parent );
+            ?>
             <div class="theme-notice notice notice-info">
-                <p class="parent-theme"><?php printf( __( 'This is a child theme of %s.' ), sprintf( '<a href="/%1$s">%2$s</a>', $theme->parent->slug, $theme->parent->name ) ); ?></p>
+                <p class="parent-theme"><?php printf( __( 'This is a child theme of %s.' ), sprintf( '<a href="%1$s">%2$s</a>', get_permalink( $parent->ID ), $parent->post_title ) ); ?></p>
             </div>
             <?php endif; ?>

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

@@ -142 +142 @@
  */
 function wporg_themes_query_themes() {
-    $args = wp_parse_args( wp_unslash( $_REQUEST['request'] ), array(
+    $request = wp_unslash( $_REQUEST['request'] );
+    $request['fields'] = wp_parse_args( $request['fields'], array(
+        'description'  => true,
+        'sections'     => false,
+        'tested'       => true,
+        'requires'     => true,
+        'rating'       => true,
+        'ratings'      => true,
+        'downloaded'   => true,
+        'downloadlink' => true,
+        'last_updated' => true,
+        'homepage'     => true,
+        'tags'         => true,
+        'num_ratings'  => true,
+        'parent'       => true,
+    ) );
+    $args = wp_parse_args( $request, array(
         'per_page' => 20,
-        'fields'   => array(
-            'description'  => true,
-            'sections'     => false,
-            'tested'       => true,
-            'requires'     => true,
-            'rating'       => true,
-            'downloaded'   => true,
-            'downloadlink' => true,
-            'last_updated' => true,
-            'homepage'     => true,
-            'tags'         => true,
-            'num_ratings'  => true,
-        ),
     ) );
 
@@ -177 +180 @@
 
 function wporg_themes_theme_info() {
-    $args  = wp_unslash( $_REQUEST );
+    $request = wp_unslash( $_REQUEST['request'] );
+    $request['fields'] = wp_parse_args( $request['fields'], array(
+        'description'  => true,
+        'sections'     => false,
+        'tested'       => true,
+        'requires'     => true,
+        'rating'       => true,
+        'ratings'      => true,
+        'downloaded'   => true,
+        'downloadlink' => true,
+        'last_updated' => true,
+        'homepage'     => true,
+        'tags'         => true,
+        'num_ratings'  => true,
+        'parent'       => true,
+    ) );
 
     include_once API_WPORGPATH . 'themes/info/1.0/class-themes-api.php';
-    $api = new Themes_API( 'query_themes', array( 'slug' => $args['slug'] ) );
+    $api = new Themes_API( 'theme_information', $request );
     $api = $api->response;
 
-    if ( is_wp_error( $api ) ) {
+    if ( empty( $api ) ) {
         wp_send_json_error();
     }