Changeset 14523 for sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/class-wporg-themes-repo-package.php

Timestamp:

09/04/2025 04:46:46 AM (3 months ago)

Author:

dd32

Message:

Theme Directory: Upgrade the blueprint used for Theme Review live previews.

This removes the blueprint from being static in the trac ticket, allowing for future maintenance not to break older links.

Fixes #8077.
Merges ​https://github.com/WordPress/wordpress.org/pull/506.
See ​https://github.com/WordPress/wordpress-playground/issues/2548.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/class-wporg-themes-repo-package.php (modified) (5 diffs)

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

@@ -17 +17 @@
 
     /**
+     * Holds the Version string for this instance of the class.
+     *
+     * NOTE: This may not be the latest version of the theme.
+     *
+     * @var string
+     */
+    public $version = '';
+
+    /**
      * Construct a new Package for the given post ID or object.
      *
      * @param WP_Post|int|slug $wp_post The Post object, Post ID, or theme slug of the package.
+     * @param string|bool $version Optional. The version string, or false for latest version.
      */
-    public function __construct( $wp_post = 0 ) {
+    public function __construct( $wp_post = 0, $version = false ) {
         global $post;
         if ( ! $wp_post ) {
@@ -53 +63 @@
             }
         }
+
+        $this->version = $version;
+        if ( $this->wp_post ) {
+            if ( ! $version || 'latest' === $version || 'latest-stable' === $version ) {
+                $this->version = $this->latest_version();
+            }
+        }
     }
 
@@ -61 +78 @@
      */
     public function screenshot_url() {
-        $screen  = 'screenshot.png';
-        $version = $this->latest_version();
+        $screen = 'screenshot.png';
 
-        if ( ! empty( $this->wp_post->_screenshot[ $version ] ) ) {
-            $screen = $this->wp_post->_screenshot[ $version ];
+        if ( ! empty( $this->wp_post->_screenshot[ $this->version ] ) ) {
+            $screen = $this->wp_post->_screenshot[ $this->version ];
         }
 
         return sprintf( 'https://i0.wp.com/themes.svn.wordpress.org/%1$s/%2$s/%3$s',
             $this->wp_post->post_name,
-            $version,
+            $this->version,
             $screen
         );
@@ -109 +125 @@
      * @return string
      */
-    public function download_url( $version = 'latest-stable' ) {
+    public function download_url( $version = false ) {
+        $version = $version ?: $this->version;
         if ( 'latest-stable' === $version ) {
             $version = $this->latest_version();
@@ -137 +154 @@
      */
     public function __get( $name ) {
-        $version = $this->latest_version();
         switch ( $name ) {
             case 'version' :
                 return $version;
             case 'theme-url' :
-                return $this->wp_post->_theme_url[ $version ] ?? '';
+                return $this->wp_post->_theme_url[ $this->version ] ?? '';
             case 'author-url' :
-                return $this->wp_post->_author_url[ $version ] ?? '';
+                return $this->wp_post->_author_url[ $this->version ] ?? '';
             case 'ticket' :
-                return $this->wp_post->_ticket_id[ $version ] ?? '';
+                return $this->wp_post->_ticket_id[ $this->version ] ?? '';
             case 'requires':
-                return $this->wp_post->_requires[ $version ] ?? '';
+                return $this->wp_post->_requires[ $this->version ] ?? '';
             case 'requires-php':
-                return $this->wp_post->_requires_php[ $version ] ?? '';
+                return $this->wp_post->_requires_php[ $this->version ] ?? '';
             default:
                 return $this->wp_post->$name;