WP Oembed in multisite fail when the permalink structure is not default

[imath]

This is a follow up of #40673

In case blogs are using pretty urls (which should be the most common case), when you try to embed a post from the main site into a sub site, the embedded URL is not interpreted and we keep that simple URL.

Thats because the main site is using a reserved blog prefix /blog we need to remove before using get_sites(). I've added a unit test to illustrate the bug.

[audrasjb]

Hi and thanks for the patch,

5.0.3 is going to be released soon. We are currently sorting the remaining tickets in the milestone.

The component maintainers for Embeds are @wonderboymusic @swissspidy and @azaozz

Do you think this ticket can be handled in the next couple of weeks? If not, let's address this in 5.1 which is coming soon.

[desrosj]

Going to punt this to 5.1. It is outside of the scope for 5.0.3 (block editor bugs, regressions, and major bugs). If it ready and committed to trunk before 5.0.3 beta, we can consider backporting it.

[pento]

Looking at the related condition in options-permalink.php​, it seems like there should be some extra checks before removing this?

[killua99]

Patch doesn't apply, require an update.

[JeffPaul]

Per today's bugscrub, we're punting this as not enough time remains to get this into 5.2. Given the minimal movement during the 5.2 release cycle, I'm punting this to Future Release so that we consciously milestone this for a numbered release when we're ready to take it on.

[killua99]

Refresh. Also to mention that the original patch was applied directly on the function wp_filter_pre_oembed_resultthis has change from the current core state to get_oembed_response_data_for_url

Trac ticket: #45052