Opened 4 years ago

Last modified 11 days ago

#10762 assigned defect (bug)

Bulk editing creates invalid URIs

Reported by: hakre Owned by:
Priority: normal Milestone: Future Release
Component: Administration Version: 2.8.4
Severity: normal Keywords: needs-patch
Cc: WordPress@…

Description

They simply get too long for example if you move 999 posts to trash. It might be helpfull to switch to the post form method.

Change History (16)

Never did understand why a GET request is used in this case. Should use POST.

Bulk Edit submits the main form on the page which uses GET. It could be refactored to use POST but that's not a trivial task.

comment:3 follow-up: ↓ 6   hakre4 years ago

For me it seems that replacing $_GET with $_REQUEST is pretty trivial. It might not be trivial to test if that's working then.

Related: #11114

Related: #10360

comment:6 in reply to: ↑ 3   nacin3 years ago

  • Milestone changed from 2.9 to 3.0

Replying to hakre:

For me it seems that replacing $_GET with $_REQUEST is pretty trivial. It might not be trivial to test if that's working then.

If you bulk-delete posts, the IDs are passed back to the form, that way the undo button can be generated. So hidden fields would also need to be created to ensure that the URI can't get too long.

It's a complicated process to go through however many forms use bulk actions and redo all of the add_query_arg code and such. Definitely quite the task.

comment:7   dd323 years ago

  • Milestone changed from 3.0 to Future Release

As this isnt a Regression from a previous version, and there is no patch, I'm moving to Future Release.

  • Keywords gsoc added
  • Owner set to scribu
  • Status changed from new to assigned

See #14579.

  • Cc WordPress@… added
  • Milestone changed from Future Release to 3.1
  • Resolution set to fixed
  • Status changed from assigned to closed
  • Component changed from Accessibility to Administration
  • Keywords gsoc removed
  • Milestone changed from 3.1 to Future Release
  • Resolution fixed deleted
  • Status changed from closed to reopened

#16335. We'll need to fix this in conjunction with ajax redux.

  • Owner scribu deleted
  • Status changed from reopened to assigned

#22062 was marked as a duplicate.

#22062 was marked as a duplicate.

#24323 was marked as a duplicate.

Note: See TracTickets for help on using tickets.