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 Owned by:
Priority: lowest Milestone: 3.0
Component: Media Version: 2.9.1
Severity: trivial Keywords: has-patch
Cc: Focuses:

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 Unassigned → 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 → fixed
  • Status new → closed

r13521

Note: See TracTickets for help on using tickets.