Changeset 11588 for sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/class-wporg-themes-upload.php

Timestamp:

02/21/2022 08:18:07 AM (3 years ago)

Author:

dd32

Message:

Theme Directory: Add an email for failed theme imports from SVN.

Theme Check still runs on theme updates via SVN, and will reject themes that do not need requirements.

See #5899.

File:

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

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

@@ -53 +53 @@
      * @var string
      */
-    protected $tmp_dir = '';
+    public $tmp_dir = '';
 
     /**
@@ -60 +60 @@
      * @var string
      */
-    protected $tmp_svn_dir = '';
+    public $tmp_svn_dir = '';
 
     /**
@@ -67 +67 @@
      * @var string
      */
-    protected $theme_dir = '';
+    public $theme_dir = '';
 
     /**
@@ -74 +74 @@
      * @var WP_Theme
      */
-    protected $theme;
+    public $theme;
 
     /**
@@ -81 +81 @@
      * @var string
      */
-    protected $theme_slug = '';
+    public $theme_slug = '';
 
     /**
@@ -88 +88 @@
      * @var WP_Post
      */
-    protected $theme_post;
+    public $theme_post;
 
     /**
@@ -95 +95 @@
      * @var WP_User
      */
-    protected $author;
+    public $author;
 
     /**
@@ -102 +102 @@
      * @var array
      */
-    protected $readme = array();
+    public $readme = array();
 
     /**
@@ -109 +109 @@
      * @var object
      */
-    protected $trac_ticket;
+    public $trac_ticket;
 
     /**
@@ -116 +116 @@
      * @var int
      */
-    protected $trac_changeset = 0;
+    public $trac_changeset = 0;
 
     /**
@@ -123 +123 @@
      * @var Trac
      */
-    protected $trac;
+    public $trac;
 
     /**
@@ -144 +144 @@
      * @var string
      */
-    protected $commit_msg = '';
+    public $commit_msg = '';
 
     /**
@@ -1761 +1761 @@
                             // When importing from SVN, include a 'Compare' link as the Changeset likely won't show a Diff unless the author did a `svn cp`.
                             'svn' === $this->importing_from && ! empty( $this->theme_post->max_version ) ?
-                            "<https://themes.trac.wordpress.org/changeset?old_path={$this->theme_slug}/{$this->theme_post->max_version}&new_path={$this->theme_slug}/{$this->theme->display( 'Version' )}|Compare>" :
+                            "<https://themes.trac.wordpress.org/changeset?old_path={$this->theme_slug}/{$this->theme_post->_last_live_version}&new_path={$this->theme_slug}/{$this->theme->display( 'Version' )}|Compare>" :
                             ''
                         ),