Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#25574 closed enhancement (fixed)

Pass export filters to the `export_wp` action

Reported by: nickdaugherty's profile nickdaugherty Owned by: wonderboymusic's profile wonderboymusic
Milestone: 3.8 Priority: normal
Severity: normal Version: 3.8
Component: Export Keywords: has-patch commit
Focuses: Cc:

Description

Currently, the export_wp action in wp-admin/export.php isn't passed the export args, making it difficult to take action based on the user selected options.

Better to pass the $args to the action.

Attachments (2)

export_wp.action.args.diff (497 bytes) - added by nickdaugherty 11 years ago.
Patches wp-admin.export.php to pass the export args to the export_wp action
export_wp.action.args.2.diff (436 bytes) - added by nickdaugherty 11 years ago.
Patch wp-admin/includes/export.php to pass $args to the export_wp action.

Download all attachments as: .zip

Change History (6)

@nickdaugherty
11 years ago

Patches wp-admin.export.php to pass the export args to the export_wp action

#1 @helen
11 years ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 3.8

This seems like a no-brainer to me.

#2 follow-up: @leewillis77
11 years ago

That action call looks like it's in wp-admin/includes/export.php in trunk, not wp-admin/export.php

@nickdaugherty
11 years ago

Patch wp-admin/includes/export.php to pass $args to the export_wp action.

#3 in reply to: ↑ 2 @nickdaugherty
11 years ago

Replying to leewillis77:

That action call looks like it's in wp-admin/includes/export.php in trunk, not wp-admin/export.php

Ah looks like you're right - I've attached an updated patch.

#4 @wonderboymusic
11 years ago

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

In 26011:

Pass $args to the export_wp action in export_wp(), which is currently passed no arguments.

Props nickdaugherty.
Fixes #25574.

Note: See TracTickets for help on using tickets.