WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 3 years ago

Last modified 16 months ago

#15261 closed enhancement (fixed)

Add do_action to form tag in edit-tag-form.php

Reported by: linguasite Owned by: nacin
Milestone: 3.7 Priority: normal
Severity: normal Version: 3.1
Component: Administration Keywords: has-patch
Focuses: Cc:

Description

How about adding additional actions to the form tags in admin taxonomy pages like edit-tag-form.php?

An example can be found in user-edit.php line 183:

<form id="your-profile" ... method="post"<?php do_action('user_edit_form_tag'); ?>>

Attachments (2)

tag_edit_form_tag_action.patch (905 bytes) - added by JustinSainton 6 years ago.
Patch for adding do_action.
15261.1.diff (1.5 KB) - added by JustinSainton 3 years ago.

Download all attachments as: .zip

Change History (15)

#1 @scribu
6 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed
  • Type changed from feature request to enhancement

There already are plenty of hooks in wp-admin/edit-tag-form.php

#2 @scribu
6 years ago

  • Milestone set to Awaiting Review
  • Resolution invalid deleted
  • Status changed from closed to reopened

Sorry, misread.

What is the use case for this?

#3 @linguasite
6 years ago

With an action like this it would be possible to set the form enctype to multipart, for example. Necessary for file upload fields in the form. The only way to change the form encoding at the moment is doing it with JS.
Furthermore it could be useful to add events or a target to the form tag itself.

#4 @nacin
6 years ago

  • Keywords needs-patch added; action edit tag removed
  • Milestone changed from Awaiting Review to Future Release

@JustinSainton
6 years ago

Patch for adding do_action.

#5 @JustinSainton
6 years ago

  • Cc JustinSainton added
  • Keywords has-patch needs-testing added; needs-patch removed

#6 @JustinSainton
3 years ago

Heh, refreshing the first patch I ever posted to Trac.

#7 @JustinSainton
3 years ago

  • Keywords needs-testing removed

#8 @nacin
3 years ago

  • Milestone changed from Future Release to 3.7

I really hate the existing actions but it's the pattern we have, so let's go with it.

#10 @mordauk
3 years ago

  • Cc pippin@… added

#11 @desrosj
3 years ago

Patch looks good. Works fine for me. Tested on categories, and a custom taxonomy.

#12 @nacin
3 years ago

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

In 25034:

Add actions for term editing/creation form tags, for adding enctype attributes. Matches the pattern we've used elsewhere. props JustinSainton, fixes #15261.

#13 @johnbillion
16 months ago

#20735 was marked as a duplicate.

Note: See TracTickets for help on using tickets.