WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 11 years ago

#26199 closed enhancement (fixed)

Pass original value to sanitize_option_* filter

Reported by: simonwheatley Owned by: chriscct7
Priority: normal Milestone: 4.3
Component: Options, Meta APIs Version: 3.2
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

Currently the sanitize_option_{$option_name} filter differs from other esc and sanitization filters in that it does not pass the original value. It should.

Attachments (4)

26199.diff​ (615 bytes ) - added by simonwheatley 13 years ago.
Adds the original value to the sanitize_option_* filter (now without my dirty test which breaks TwentyThirteen)
26199.2.diff​ (772 bytes ) - added by simonwheatley 12 years ago.
Refreshed for 4.0a trunk
26199.3.diff​ (772 bytes ) - added by MikeHansenMe 12 years ago.
refreshed for 4.1
26199.4.diff​ (878 bytes ) - added by MikeHansenMe 11 years ago.
Refreshed for 4.3

Download all attachments as: .zip

Change History (10)

@simonwheatley
13 years ago

Adds the original value to the sanitize_option_* filter (now without my dirty test which breaks TwentyThirteen)

#1 @SergeyBiryukov
13 years ago

  • Keywords 3.9-early added
  • Milestone Awaiting Review → Future Release

@simonwheatley
12 years ago

Refreshed for 4.0a trunk

#2 @simonwheatley
12 years ago

I've refreshed the patch:

  • The intended effect still works (original value passed as a third parameter)
  • Four unit tests fail, see below, but these appear unrelated
Version 0, edited 12 years ago by simonwheatley (next)

@MikeHansenMe
12 years ago

refreshed for 4.1

#3 @chriscct7
11 years ago

  • Keywords 3.9-early removed
  • Owner set to chriscct7
  • Severity minor → normal
  • Status new → reviewing
  • Version 3.2.1 → 3.2

@MikeHansenMe
11 years ago

Refreshed for 4.3

#4 @MikeHansenMe
11 years ago

Refreshed the patch for 4.3.

#5 @boonebgorges
11 years ago

  • Component Security → Options, Meta APIs
  • Milestone Future Release → 4.3

#6 @boonebgorges
11 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 32520:

In sanitize_option(), pass the unsanitized $value to the filter.

Props simonwheatley, MikeHansenMe.
Fixes #26199.

Note: See TracTickets for help on using tickets.