Changeset 12632 for sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/class-themes-api.php

Timestamp:

06/08/2023 02:34:42 AM (3 years ago)

Author:

dd32

Message:

Theme Directory: Refactor the API calls to properly query the user with favorites.

This reverts [12630] which was a quick-fix for the issue, but it was actually caused by [12600].

This refactors the API query code to avoid using jsonp requests for ease of debugging, and slims down the request to what it actually needs to query for.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/class-themes-api.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/class-themes-api.php

@@ -123 +123 @@
         // Favorites requests require a user to fetch favorites for.
         if ( isset( $this->request->browse ) && 'favorites' === $this->request->browse && ! isset( $this->request->user ) ) {
-            if ( is_user_logged_in() ) {
-                $this->request->user = wp_get_current_user()->user_nicename;
-            } else {
-                $this->request->user = '';
-                $this->bad_input = true;
-            }
+            $this->request->user = '';
+            $this->bad_input = true;
         }