WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 2 years ago

#18214 closed enhancement (wontfix)

make_clickable() not applied to the_content

Reported by: scribu Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.2.1
Component: Formatting Keywords: has-patch needs-testing 2nd-opinion
Focuses: Cc:

Description

Maybe I'm missing something, but if I add a non-embedable link to the post content, like so:

Some paragraph

http://google.com

Another pagragraph

it's not made clickable.

Attachments (1)

18214.diff (531 bytes) - added by scribu 3 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 scribu3 years ago

  • Version set to 3.2.1

comment:2 nacin3 years ago

Correct.

scribu3 years ago

comment:3 scribu3 years ago

  • Keywords has-patch needs-testing added
  • Milestone changed from Awaiting Review to 3.3

18214.diff seems to work fine with embeds. Probably needs a good soak to find out any potential edge cases.

Last edited 3 years ago by scribu (previous) (diff)

comment:4 nacin3 years ago

  • Keywords 2nd-opinion added
  • Milestone changed from 3.3 to Awaiting Review

Why do we need make_clickable in content? That's a heavy regex to apply to a lot of content, and no one's bothered previously. Sounds like a plugin.

comment:5 scribu3 years ago

Just seemed obvious, since we now encourage users to paste plain links to be embedded. That's why I marked this as a defect, instead of an enhancement.

comment:6 nacin3 years ago

  • Type changed from defect (bug) to enhancement

comment:7 scribu2 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Nevermind; it's easy enough to enable.

Note: See TracTickets for help on using tickets.