Make WordPress Core

Opened 6 years ago

Last modified 6 years ago

#45052 reviewing defect (bug)

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

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

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 6 years ago.
45052.1.diff (1.8 KB) - added by killua99 6 years 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
6 years ago

#1 @SergeyBiryukov
6 years ago

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

#2 @pento
6 years ago

  • Milestone changed from 5.0.1 to 5.0.2

#3 @pento
6 years ago

  • Milestone changed from 5.0.2 to 5.0.3

#4 @audrasjb
6 years 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 6 years ago by audrasjb (previous) (diff)

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


6 years ago

#6 @desrosj
6 years 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
6 years 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.


6 years ago

#9 @killua99
6 years 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 years ago

#11 @JeffPaul
6 years 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 years 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.