WordPress.org

Make WordPress Core

Opened 13 months ago

Last modified 6 months ago

#45052 reviewing defect (bug)

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

Reported by: imath Owned by: SergeyBiryukov
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Embeds Keywords: has-patch dev-feedback needs-refresh
Focuses: multisite Cc:
PR Number:

Description

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.

Attachments (2)

45052.patch (1.8 KB) - added by imath 13 months ago.
45052.1.diff (1.8 KB) - added by killua99 6 months ago.
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

Download all attachments as: .zip

Change History (13)

@imath
13 months ago

#1 @SergeyBiryukov
13 months ago

  • Milestone changed from Awaiting Review to 5.0.1
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

#2 @pento
10 months ago

  • Milestone changed from 5.0.1 to 5.0.2

#3 @pento
10 months ago

  • Milestone changed from 5.0.2 to 5.0.3

#4 @audrasjb
10 months ago

  • Keywords dev-feedback added

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.

Last edited 10 months ago by audrasjb (previous) (diff)

This ticket was mentioned in Slack in #core by desrosj. View the logs.


10 months ago

#6 @desrosj
10 months ago

  • Milestone changed from 5.0.3 to 5.1

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.

#7 @pento
9 months ago

  • Milestone changed from 5.1 to 5.2

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

This ticket was mentioned in Slack in #core-media by joemcgill. View the logs.


7 months ago

#9 @killua99
7 months ago

  • Keywords needs-refresh added

Patch doesn't apply, require an update.

This ticket was mentioned in Slack in #core by jeffpaul. View the logs.


6 months ago

#11 @JeffPaul
6 months ago

  • Milestone changed from 5.2 to Future Release

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
6 months ago

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

Note: See TracTickets for help on using tickets.