Make WordPress Core

Opened 20 years ago

Closed 20 years ago

#786 closed defect (bug) (fixed)

Wrong use of current userinfo when handling comments

Reported by: anonymousbugger's profile anonymousbugger Owned by: matt's profile matt
Milestone: Priority: normal
Severity: minor Version: 1.5
Component: General Keywords:
Focuses: Cc:

Description

In file: http://cvs.sourceforge.net/viewcvs.py/cafelog/wordpress/wp-comments-post.php?rev=1.27&view=markup

The code:
<pre> If the user is logged in
get_currentuserinfo();
if ( $user_ID ) :

$comment_author = $user_login;</pre>

Should be replaced with:
<pre> If the user is logged in
get_currentuserinfo();
if ( $user_ID ) :

$comment_author = $user_identity;</pre>

Change History (5)

#1 @anonymousbugger
20 years ago

  • Patch set to No

#2 @sethkinast
20 years ago

Confirmed. While we're on the subject, two lines below that line, "http://" should NOT be removed as this breaks the links on commentors' names.

#3 @dmclark
20 years ago

Agree.

#4 @CarLBanks
20 years ago

I also get this issue so I manually changed it.

#5 @matt
20 years ago

  • fixed_in_version set to 1.5
  • Owner changed from anonymous to matt
  • Resolution changed from 10 to 20
  • Status changed from new to closed

Thanks!

Note: See TracTickets for help on using tickets.