[wp-trac] [WordPress Trac] #26644: Theme screenshots in admin relative to Site Address (URL)

[WordPress Trac]

#26644: Theme screenshots in admin relative to Site Address (URL)
-------------------------------+--------------------
 Reporter:  diegogb            |       Owner:
     Type:  defect (bug)       |      Status:  new
 Priority:  normal             |   Milestone:  3.8.1
Component:  Themes             |     Version:  3.8
 Severity:  normal             |  Resolution:
 Keywords:  reporter-feedback  |
-------------------------------+--------------------
Changes (by morganestes):

 * keywords:   => reporter-feedback


Comment:

 @diegogb The default usage of `get_screenshot()` is to use the absolute
 URI of the `screenshot.ext` file that is returned by
 `get_stylesheet_directory_uri()`. It's called with the `'relative'`
 parameter in two places in `class-wp-theme.php`: [source:trunk/src/wp-
 includes/class-wp-theme.php#L343 Line 343] and [source:trunk/src/wp-
 includes/class-wp-theme.php#L425 Line 425], but other than that it should
 be pulling from whatever is eventually returned by `get_theme_root_uri()`.
 Even that is filtered to `siteurl`, though, which eventually boils down to
 the "WordPress Address (URL)" setting in admin.

 Since core only calls for the relative path in those two places, I'm going
 to venture a guess that your plugin is filtering out that URL somewhere. I
 suggest checking on a clean install of the plugin (use
 `define('WPHTTPS_RESET', true);` in the plugin itself) and see how that
 works for you.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/26644#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress blogging software