Skip to:
Content

bbPress.org

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#2110 closed defect (bug) (fixed)

Use urlencode instead of esc_attr for search terms

Reported by:jmdodd's profile jmdoddOwned by:
Milestone:2.3Priority:normal
Severity:normalVersion:trunk
Component:Component - SearchKeywords:has-
Cc:

Description

Spaces between terms are getting lost with current esc_attr.

Attachments (1)

2110.1.diff (850 bytes) - added by jmdodd 12 years ago.

Download all attachments as: .zip

Change History (4)

@jmdodd
12 years ago

#1 @johnjamesjacoby
12 years ago

Still worth using esc_attr() in the input box (I think) but the URL should definitely get the urlencode() treatment.

#2 @jmdodd
12 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [4581]) Replace esc_attr() with urlencode() when adding search terms query args.

#3 @jmdodd
12 years ago

form-search.php uses esc_attr(); this was specifically for dealing with the add_args situations like pagination.

Note: See TracTickets for help on using tickets.