WordPress.org
Get WordPress

Make WordPress Core

Opened 16 years ago

Closed 16 years ago

#12165 closed enhancement (fixed)

Edit media form has get variables in action

Reported by: reaperhulk's profile reaperhulk Owned by:
Milestone: 3.0 Priority: lowest
Severity: trivial Version: 2.9.1
Component: Media Keywords: has-patch
Focuses: Cc:

Description

The wp-admin/media.php edit media form does not remove the edit and attachment_id query args when inserting the action URI. This does not cause any issues in practice because the wp_reset_vars function preferentially chooses $_POST over $_GET, but it does result in confusion for people trying to follow the program logic.

Attachments (1)

12165.patch​ (691 bytes) - added by reaperhulk 16 years ago.

Download all attachments as: .zip

Change History (7)

@reaperhulk
16 years ago

#1 @reaperhulk
16 years ago

  • Keywords has-patch added

#2 @scribu
16 years ago

  • Milestone changed from Unassigned to 3.0

#3 follow-up: @nacin
16 years ago

We might as well just post to itself (i.e. empty action attribute), no?

#4 in reply to: ↑ 3 @scribu
16 years ago

Replying to nacin:

We might as well just post to itself (i.e. empty action attribute), no?

+1 (less is better :)

#5 @reaperhulk
16 years ago

Agreed, easy fix and no need for the remove_query_arg call then.

#6 @nacin
16 years ago

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

r13521

Note: See TracTickets for help on using tickets.