Ticket #5347 (closed enhancement: fixed)

Opened 4 years ago

Last modified 4 years ago

Move export_wp() to an include file

Reported by: tellyworth Owned by: anonymous
Priority: normal Milestone: 2.5
Component: General Version:
Severity: normal Keywords:
Cc:

Description

This is a simple refactoring job that makes it possible to call export_wp() for unit testing. The function now lives in an include file, and it no longer calls die() at the end.

Attachments

export-refactor-r6324.patch Download (17.9 KB) - added by tellyworth 4 years ago.
export-refactor-r6324-a.patch Download (17.9 KB) - added by tellyworth 4 years ago.
pass author as a function parameter

Change History

tellyworth — 4 years ago

comment:1   ryan — 4 years ago

How about passing author_id to export_wp() instead of it relying on _GET?

tellyworth — 4 years ago

pass author as a function parameter

comment:2   ryan — 4 years ago

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

(In [6335]) Move export_wp() to an include. Props tellyworth. fixes #5347

comment:3   darkdragon — 4 years ago

  • Milestone changed from 2.5 to 2.4

Shouldn't this be moved to 2.4, since it is trunk and this was fixed on the trunk?

Note: See TracTickets for help on using tickets.