Make WordPress Core

Opened 17 years ago

Closed 17 years ago

#11471 closed defect (bug) (fixed)

After bulk comment actions, spaces in a search are removed

Reported by: markjaquith Owned by: westi
Priority: normal Milestone: 3.0
Component: Administration Version: 2.8.4
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

search for "foo bar" or some other two word term in your comments.

Perform a bulk spam.

On redirect, the space between "foo" and "bar" will be missing.

Attachments (2)

11471.diff (1.9 KB ) - added by nacin 17 years ago.
11471-simple.diff (763 bytes ) - added by nacin 17 years ago.
Alternative: Just urlencode() it and move along.

Download all attachments as: .zip

Change History (6)

#1 @scribu
17 years ago

Confirmed. Really strange.

@nacin
17 years ago

#2 @nacin
17 years ago

  • Keywords has-patch added

edit-comments.php handles redirection after a bulk action unlike the other pages. Instead of using wp_get_referer(), remove the query args it needs to, and adds the query args it needs to, it uses a base of "edit-comments.php" then adds on everything, including existing contextual query args such as the search query.

Patch follows the lead set by other pages. Alternatively, we can simply urlencode() $_REQUESTs on what is currently line 106.

@nacin
17 years ago

Alternative: Just urlencode() it and move along.

#3 @westi
17 years ago

  • Milestone 2.93.0
  • Owner set to westi
  • Status newaccepted
  • Version 2.92.8.4

This also exists in 2.8.

I would prefer to fix it the full way but this will have to wait for 3.0 as it is not a regression.

#4 @westi
17 years ago

  • Resolutionfixed
  • Status acceptedclosed

(In [12461]) Ensure that bulk comment actions do not remove spaces from search terms.
Also refactors comment bulk action redirect url building to be consistent with other operations.
Fixes #11471 props nacin.

Note: See TracTickets for help on using tickets.