WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#18214 closed enhancement (wontfix)

make_clickable() not applied to the_content

Reported by: scribu's profile 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 13 years ago.

Download all attachments as: .zip

Change History (8)

#1 @scribu
13 years ago

  • Version set to 3.2.1

#2 @nacin
13 years ago

Correct.

@scribu
13 years ago

#3 @scribu
13 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 13 years ago by scribu (previous) (diff)

#4 @nacin
13 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.

#5 @scribu
13 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.

#6 @nacin
13 years ago

  • Type changed from defect (bug) to enhancement

#7 @scribu
13 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.