Make WordPress Core

Opened 9 years ago

Closed 9 years ago

#34709 closed defect (bug) (fixed)

oEmbed REST namespace has trailing slash

Reported by: rmccue's profile rmccue Owned by: rmccue's profile rmccue
Milestone: 4.4 Priority: normal
Severity: normal Version: 4.4
Component: REST API Keywords:
Focuses: Cc:

Description

The oEmbed namespace used when registering the routes has a trailing slash, which is redundant (and not recommended). This means that the namespace in the index doesn't match the expected format.

Attachments (1)

34709.diff (573 bytes) - added by swissspidy 9 years ago.

Download all attachments as: .zip

Change History (4)

@swissspidy
9 years ago

#1 follow-up: @swissspidy
9 years ago

  • Owner set to rmccue
  • Status changed from new to reviewing

I assume changing it in the oEmbed controller is easier than in register_rest_route().

#2 in reply to: ↑ 1 @rmccue
9 years ago

Replying to swissspidy:

I assume changing it in the oEmbed controller is easier than in register_rest_route().

Indeed :) The route should continue to have the leading / though.

#3 @rmccue
9 years ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 35656:

oEmbed: Drop the trailing slash from the namespace.

Props swissspidy.
Fixes #34709.

Note: See TracTickets for help on using tickets.