Make WordPress Core

Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#8627 closed defect (bug) (fixed)

Emoticons count as links in comments

Reported by: mtdewvirus's profile mtdewvirus Owned by: tott's profile tott
Milestone: 2.7.1 Priority: normal
Severity: normal Version: 2.7
Component: Comments Keywords: comment moderation link emoticon, has-patch, needs-testing
Focuses: Cc:

Description

If "Convert emoticons like :-) and :-P to graphics on display" is enabled on the blog r9782 doesn't work as intended.

Each emoticon gets counted as a link in the comment (because the img has been inserted) and many comments are held in moderation that shouldn't be.

Change History (8)

#1 @mtdewvirus
16 years ago

  • Owner changed from anonymous to tott

#2 @ninjaWR
16 years ago

  • Component changed from General to Comments
  • Keywords moderation added

#3 @dome90
16 years ago

Two workarounds for right now:
1-to increase the number of links a comment can have (wp-admin/options-discussion.php)

2-to deactivte the converting of emoticons like :-) and :-P to graphics(/wp-admin/options-writing.php)

#4 @Otto42
16 years ago

Fix: Change the regular expression to this:

|(href\t*?=\t*?['\"]?)(https?:)?//|i

Eliminate that center question mark. If we're applying the comment_text filter first, then any unlinked http uris will have been autolinked already, meaning that the href= will exist around them. Making it not necessary for it to be optional.

This will also eliminate the false positive on an img src="http:whatever"

#5 @mrmist
16 years ago

See also #7268

#6 @tott
16 years ago

  • Keywords has-patch needs-testing added

added a revised version of the original patch to #7821 and tested against the current trunk. none of the issues appears.

#7 @ryan
16 years ago

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

(In [10453]) Don't count emoticons toward link total. Props tott. fixes #8627 #7821 for trunk

#8 @ryan
16 years ago

(In [10454]) Don't count emoticons toward link total. Props tott. fixes #8627 #7821 for 2.7

Note: See TracTickets for help on using tickets.