Make WordPress Core

Opened 17 years ago

Closed 17 years ago

#11746 closed defect (bug) (fixed)

deprecate generate_random_password()

Reported by: Denis-de-Bernardy Owned by:
Priority: normal Milestone: 3.0
Component: Multisite Version: 3.0
Severity: normal Keywords: has-patch needs-testing
Cc: Focuses:

Description

WP has a wp_generate_password() function, which we should use instead.

Attachments (4)

11746.diff​ (2.7 KB ) - added by ryan 17 years ago.
Untested
11746.2.diff​ (1.2 KB ) - added by nacin 17 years ago.
Some inline docs added as well.
11746.3.diff​ (1.2 KB ) - added by nacin 17 years ago.
Little better inline documentation
random_password.diff​ (393 bytes ) - added by uglyrobot 17 years ago.
I know the function is pluggable, but can we keep the random_password filter? We use it.

Download all attachments as: .zip

Change History (13)

#1 @Denis-de-Bernardy
17 years ago

atm, it's used in ms-edit.php and ms-functions,php

@ryan
17 years ago

Untested

#2 @ryan
17 years ago

  • Keywords has-patch needs-testing added

#3 @ryan
17 years ago

  • Resolution → fixed
  • Status new → closed

(In [12624]) Deprecate generate_random_password(). fixes #11746

#4 @nacin
17 years ago

  • Resolution fixed
  • Status closed → reopened

There's a spelling mistake in the third argument of the _deprecated_functon call.

@nacin
17 years ago

Some inline docs added as well.

@nacin
17 years ago

Little better inline documentation

#5 @westi
17 years ago

Docs look good.
I'm wondering why we are preserving the exact functionality of this function can't we just use the function we recommend in this function like we do for other deprecated functions.

#6 @westi
17 years ago

  • Resolution → fixed
  • Status reopened → closed

(In [12635]) phpDoc for ms-deprecated.php props nacin.
Make generate_random_password use wp_generate_password()
Fixes #11746.

@uglyrobot
17 years ago

I know the function is pluggable, but can we keep the random_password filter? We use it.

#7 @uglyrobot
17 years ago

  • Resolution fixed
  • Status closed → reopened

Add filter

#9 @Denis-de-Bernardy
17 years ago

  • Resolution → fixed
  • Status reopened → closed
Note: See TracTickets for help on using tickets.