Make WordPress Core

Opened 11 years ago

Closed 10 years ago

#25034 closed defect (bug) (fixed)

Searches reverts to previous search criteria

Reported by: webtechglobal's profile WebTechGlobal Owned by: nacin's profile nacin
Milestone: 3.7 Priority: normal
Severity: normal Version: 3.1
Component: Administration Keywords: has-patch needs-testing
Focuses: Cc:

Description

I submitted a comments search. Then made a selection in the Bulk Actions menu but then changed my mind on that. I attempted to perform another search however the criteria for the new search reverted to the previous search in both URL and the text field itself.

I can confirm that it is the changing of the Bulk Actions menu causing this. Although minor, anyone who does happen to make a selection in the menu then attempt a search. May not notice and think their search results are valid.

Filter menu does not have the same effect.

Originally I thought the worst case scenario is a bad comments search and a rare set of motions to make that happen. Until I decided to try post search and the same thing happens there.

Attachments (7)

25034.patch (406 bytes) - added by c3mdigital 11 years ago.
25034.3.patch (1.6 KB) - added by c3mdigital 11 years ago.
25034.4.patch (944 bytes) - added by c3mdigital 11 years ago.
consolidate fix into common.js so same code is not repeated in inline-edit.js
25034.5.patch (974 bytes) - added by vinod dalvi 11 years ago.
fixes the search form bug in posts if we search by pressing Enter button in search input field.
25034.6.patch (491 bytes) - added by c3mdigital 11 years ago.
use input [type="search"] for selector
25034.7.patch (901 bytes) - added by c3mdigital 11 years ago.
Remove dead code from inline-edit-post.js
25034.8.patch (945 bytes) - added by vinod dalvi 11 years ago.
Handles search form submission on clicking submit button.

Download all attachments as: .zip

Change History (21)

#1 @SergeyBiryukov
11 years ago

  • Component changed from General to Comments
  • Keywords reporter-feedback added

This is about the Comments screen in the admin, right?

Could not reproduce on a clean install using your description. Could you please add more exact steps to reproduce?

#2 @WebTechGlobal
11 years ago

Sure...

  1. Enter search keyword/phrase in either post or comment search
  2. Submit the search
  3. Proceed to make a selection in Bulk Actions menu
  4. Now lets say we change our mind on doing that
  5. Attempt to execute a different search

I discovered this when I decided I needed to change my search just a little. So I changed one character, submitted and noticed the search value showing in address bar was the first search.

My guess here is on interacting with Bulk Actions menu the current URL becomes the form action.

#3 @c3mdigital
11 years ago

I can confirm this bug on trunk.

Steps to reproduce

  1. go to edit.php
  2. Entered he in the search form and pressed return
  3. Selected "Edit" in bulk actions
  4. Checked a few boxes on posts to edit
  5. Clicked Apply
  6. Clicked Cancel on inline bulk edit
  7. Changed search term to hello pressed return

Result: Search reverted back to he. I tried mulitple times with different search terms and same result.

Looks like a 302 redirect:

Request URL:http://local-server.dev/wp-admin/edit.php?s=post+world&post_status=all&post_type=post&_wpnonce=527aed7f16&_wp_http_referer=%2Fwp-admin%2Fedit.php%3Fs%3Dpos%26post_status%3Dall%26post_type%3Dpost%26action%3D-1%26m%3D0%26cat%3D0%26paged%3D1%26mode%3Dlist%26action2%3D-1&action=edit&m=0&cat=0&paged=1&mode=list&action2=-1
Request Method:GET
Status Code:302 Found
Request Headersview parsed
GET /wp-admin/edit.php?s=post+world&post_status=all&post_type=post&_wpnonce=527aed7f16&_wp_http_referer=%2Fwp-admin%2Fedit.php%3Fs%3Dpos%26post_status%3Dall%26post_type%3Dpost%26action%3D-1%26m%3D0%26cat%3D0%26paged%3D1%26mode%3Dlist%26action2%3D-1&action=edit&m=0&cat=0&paged=1&mode=list&action2=-1 HTTP/1.1
Host: local-server.dev
Connection: keep-alive
Cache-Control: no-cache
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
Pragma: no-cache
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_8_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/28.0.1500.95 Safari/537.36
DNT: 1
Referer: http://local-server.dev/wp-admin/edit.php?s=pos&post_status=all&post_type=post&action=-1&m=0&cat=0&paged=1&mode=list&action2=-1
Accept-Encoding: gzip,deflate,sdch
Accept-Language: en-US,en;q=0.8

Gets Redirected to previous search:

Request URL:http://local-server.dev/wp-admin/edit.php?s=pos&post_status=all&post_type=post&action=-1&m=0&cat=0&paged=1&mode=list&action2=-1
Request Method:GET
Status Code:200 OK
Request Headersview parsed
GET /wp-admin/edit.php?s=pos&post_status=all&post_type=post&action=-1&m=0&cat=0&paged=1&mode=list&action2=-1 HTTP/1.1
Host: local-server.dev
Connection: keep-alive
Cache-Control: no-cache
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
Pragma: no-cache
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_8_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/28.0.1500.95 Safari/537.36
DNT: 1
Referer: http://local-server.dev/wp-admin/edit.php?s=pos&post_status=all&post_type=post&action=-1&m=0&cat=0&paged=1&mode=list&action2=-1
Accept-Encoding: gzip,deflate,sdch
Accept-Language: en-US,en;q=0.8

#4 @c3mdigital
11 years ago

  • Component changed from Comments to Administration
  • Keywords needs-patch added; 2nd-opinion reporter-feedback removed
  • Severity changed from trivial to normal
  • Type changed from enhancement to defect (bug)

#5 @ocean90
11 years ago

  • Milestone changed from Awaiting Review to 3.7

My steps to reproduce

  • Comments screen
  • Search for "foo"
  • Change bulk action to something else then "Bulk Actions" (Don't click Apply)
  • Search for "bar"
  • Result: The old search value "foo" and bulk action was executed

#6 @vinod dalvi
11 years ago

  • Cc mozillavvd@… added

#7 @helen
11 years ago

#25069 was marked as a duplicate.

#8 @c3mdigital
11 years ago

  • Keywords has-patch added; needs-patch removed
  • Version changed from 3.6 to 3.1

We broke this in [17006] inline-edit-post.js wasn't changed to reflect the new id attribute on the submit button.

@c3mdigital
11 years ago

#9 @c3mdigital
11 years ago

Same issue on comments list table and media list table. 25034.3 includes fixes for all 3 list table searches.

@c3mdigital
11 years ago

@c3mdigital
11 years ago

consolidate fix into common.js so same code is not repeated in inline-edit.js

#10 @c3mdigital
11 years ago

25034.4.patch fixes the search form bug in posts, comments, users, plugins, and media list table search forms.

@vinod dalvi
11 years ago

fixes the search form bug in posts if we search by pressing Enter button in search input field.

@c3mdigital
11 years ago

use input [type="search"] for selector

#11 follow-up: @nacin
11 years ago

"#post-query-submit" is for the "Filter" box. It seems like inline-edit-post.js handles this currently (as seen in 25034.4.patch) but is it needed? Original report says "Filter menu does not have the same effect."

#12 in reply to: ↑ 11 @c3mdigital
11 years ago

Replying to nacin:

"#post-query-submit" is for the "Filter" box. It seems like inline-edit-post.js handles this currently (as seen in 25034.4.patch) but is it needed? Original report says "Filter menu does not have the same effect."

It's not needed anymore since it's handled in common.js now. revert() was never needed since we are refreshing the page 25034.7.patch removes it.

@c3mdigital
11 years ago

Remove dead code from inline-edit-post.js

@vinod dalvi
11 years ago

Handles search form submission on clicking submit button.

#13 @vinod dalvi
11 years ago

  • Keywords needs-testing added

#14 @nacin
10 years ago

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

In 25677:

List tables: When executing a search, reset the Bulk Actions dropdown to avoid the search from failing.

props c3mdigital, vinod-dalvi.
fixes #25034.

Note: See TracTickets for help on using tickets.