Make WordPress Core

Opened 16 years ago

Closed 13 years ago

#12657 closed defect (bug) (invalid)

wp_signon() adds one filter per call

Reported by: hakre Owned by:
Priority: normal Milestone:
Component: General Version: 3.0
Severity: normal Keywords:
Cc: Focuses:

Description

If wp_signon() is called multiple times, the filter will be added multiple times. Next to this a lot of the function seems to be just typed in in the wish that it does work instead of doing things properly. See quote: "ugly hack to pass this to wp_authenticate_cookie" or leftover TODO markings and the like.

Should be put in order prior to next release.

Attachments (1)

clipboard.patch (876 bytes ) - added by hakre 16 years ago.
Only use filter once

Download all attachments as: .zip

Change History (7)

#1 @hakre
16 years ago

Maybe some code example from that function (too much paranthesis and instructions):

( 0 !== strpos($redirect_to, 'https') ) && ( 0 === strpos($redirect_to, 'http') )

(taken from an if clause therein)

#2 @hakre
16 years ago

please delete my last comment, I wish I could do that.

#3 @hakre
16 years ago

Code in function wp_authenticate() might be related.

#4 @nacin
16 years ago

  • Milestone 3.0Future Release

@hakre
16 years ago

Only use filter once

#5 @hakre
16 years ago

Technically this should not be necessary as the new callback would overwrite the previous one because the generate the same hook identification and therefore replace each other.

#6 @nacin
13 years ago

  • Milestone Future Release
  • Resolutioninvalid
  • Status newclosed

Adding a filter a second time does nothing, a simple overwrite occurs.

Note: See TracTickets for help on using tickets.