Make WordPress Core

Opened 20 years ago

Closed 20 years ago

Last modified 20 years ago

#883 closed defect (bug) (fixed)

wp-comments-post strips http:// from author url for logged in users

Reported by: gbhugo's profile gbhugo Owned by: ryan's profile ryan
Milestone: Priority: normal
Severity: minor Version:
Component: Administration Keywords:
Focuses: Cc:

Description

When a logged in user comments, the wp-comments-post code stripps the http:// from her URI before writing the cookie. Next time around the commenter produces a broken URI.

Change History (9)

#1 @gbhugo
20 years ago

  • Patch set to No

#2 @gbhugo
20 years ago

And of course it's not adminstration but general and it is for 1.5 :-/

Too bad one can't edit the bugreport itself (at least I didn't find an obvious way to do so).

#3 @sethkinast
20 years ago

This was fixed a couple weeks ago in CVS.

#4 @gbhugo
20 years ago

I doubt that it was fixed weeks ago - because it is in the just released tar.gz ...

#5 @ryan
20 years ago

The clean_url() filter should add the http:// back. Stripping it in the first place is indeed needless, but clean_url() should correct it.

edited on: 02-16-05 21:31

#6 @gbhugo
20 years ago

When the stripping was in, I had the following effect: when going to the comment form, the normal URL was shown. After posting the comment, the URL was changed - and in the comment form it was presented without leading http:// - so at least this will irritate users, even if the http:// is added on the next comment. I think the URL should be left untouched in the process so that the cookie contains the full URL.

#7 @ryan
20 years ago

Okay. Fixed.

#8 @ryan
20 years ago

  • Status changed from new to closed

#9 @ryan
20 years ago

  • fixed_in_version set to 1.5
  • Owner changed from anonymous to rboren
  • Resolution changed from 10 to 20
Note: See TracTickets for help on using tickets.