Make WordPress Core

Opened 18 years ago

Closed 18 years ago

Last modified 7 years ago

#2995 closed defect (bug) (fixed)

rel="nofollow" duplicated when comments are edited

Reported by: macmanx's profile macmanx Owned by: filosofo's profile filosofo
Milestone: Priority: normal
Severity: normal Version: 2.1
Component: General Keywords: comments editing nofollow bg|has-patch
Focuses: Cc:

Description

Under WordPress v2.0.4, the rel="nofollow" attribute added to links inside the comments is duplicated whenever a comment is edited.

For example, a link inside a comment displays the following structure are the first posting:

<a href="url" rel="nofollow">title</a>

After the first edit, the structure becomes:

<a href="url" rel="nofollow" rel="nofollow">title</a>

After the second edit, the structure becomes:

<a href="url" rel="nofollow" rel="nofollow" rel="nofollow">title</a>

And so forth.

Fortunately, the rel="nofollow" attributes are visible in the comment editing interface and can be removed to return the links to only one rel="nofollow" attribute.

I have tested this with both my current configuration and under the Default theme with no active plugins.

Attachments (1)

formatting.php.diff (582 bytes) - added by filosofo 18 years ago.

Download all attachments as: .zip

Change History (7)

#1 @filosofo
18 years ago

  • Keywords bg|has-patch added
  • Owner changed from anonymous to filosofo
  • Version changed from 2.0.4 to 2.1

My patch fixes this problem.

#2 @ryan
18 years ago

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

(In [4135]) Don't duplicate nofollow on edit. Props filosofo. fixes #2995

#3 @ryan
18 years ago

(In [4233]) Don't duplicate nofollow on edit. Props filosofo. fixes #2995

#4 @foolswisdom
18 years ago

  • Milestone changed from 2.1 to 2.0.5

#5 @(none)
18 years ago

  • Milestone 2.0.5 deleted

Milestone 2.0.5 deleted

This ticket was mentioned in Slack in #community-events by colmtroy. View the logs.


7 years ago

Note: See TracTickets for help on using tickets.