Ticket #4237 (closed task: fixed)

Opened 2 years ago

Last modified 1 year ago

Convert to using wp_parse_args()

Reported by: rob1n Assigned to: rob1n
Priority: normal Milestone: 2.3
Component: General Version:
Severity: normal Keywords:
Cc:

Attachments

4237-wpincludes.diff (16.8 kB) - added by rob1n on 05/11/07 02:55:04.

Change History

05/09/07 04:51:23 changed by rob1n

  • owner changed from anonymous to rob1n.
  • status changed from new to assigned.

05/09/07 21:15:56 changed by ryan

While we're at it, change wp_parse_args() so that it doesn't eliminate args that aren't in defaults. Plugins need to pass things along.

05/10/07 01:06:14 changed by rob1n

Yep, makes sense.

05/10/07 02:33:03 changed by rob1n

(In [5428]) Don't discard non-default args in wp_parse_args(). see #4237

05/11/07 02:55:04 changed by rob1n

  • attachment 4237-wpincludes.diff added.

05/11/07 03:10:05 changed by rob1n

  • status changed from assigned to closed.
  • resolution set to fixed.

(In [5444]) Use wp_parse_args(). fixes #4237

06/14/07 17:52:25 changed by mdawaffe

FYI, [5428] implies that wp_parse_args() followed by an extract() cannot be trusted if given user imput.

06/14/07 18:07:16 changed by ryan

We should probably be using the EXTR_SKIP argument to extract everywhere.