WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 15 years ago

#18214 closed enhancement (wontfix)

make_clickable() not applied to the_content

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

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 15 years ago.

Download all attachments as: .zip

Change History (8)

#1 @scribu
15 years ago

  • Version → 3.2.1

#2 @nacin
15 years ago

Correct.

@scribu
15 years ago

#3 @scribu
15 years ago

  • Keywords has-patch needs-testing added
  • Milestone Awaiting Review → 3.3

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

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

#4 @nacin
15 years ago

  • Keywords 2nd-opinion added
  • Milestone 3.3 → 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
15 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
15 years ago

  • Type defect (bug) → enhancement

#7 @scribu
15 years ago

  • Milestone Awaiting Review
  • Resolution → wontfix
  • Status new → closed

Nevermind; it's easy enough to enable.

Note: See TracTickets for help on using tickets.