Make WordPress Core

Opened 17 years ago

Closed 16 years ago

Last modified 4 months ago

#6392 closed defect (bug) (duplicate)

Add support to XMLRPC to handle enclosure from editPost/newPost - Patch for Wordpress 2.5 included

Reported by: amandato's profile amandato Owned by:
Milestone: Priority: normal
Severity: normal Version: 2.5
Component: XML-RPC Keywords: enclosure xmlrpc editPost newPost meta meta_value has-patch has-unit-tests
Focuses: Cc:

Description

Attached code adds enclosure support to the XMLRPC functions. Enclosures are stored in wordpress in the postmeta table. Enclosure is a special struct within the content struct passed to the editPost/newPost functions. Adding this functionality will allow podcasters to post their media from another blogging application or service.

The code attached uses the styling of the attach_uploads() function and relies on the set_custom_fields() function. Through tests I've performed, the enclosure information is inserted correctly and displayed within the RSS 2.0 feed.

Please refer to Dave Winer's documentation for a reference how the enclosure struct is immpemented within the content struct of the addPost/editPost MetaWeblogAPI functions.
http://www.xmlrpc.com/metaWeblogApi

Attached patch was applied to the development version of Wordpress 2.5, last checked out revision 7524 from the Subversion repository.

Attachments (1)

xmlrpc_enclosure_support.diff (2.5 KB) - added by amandato 17 years ago.

Download all attachments as: .zip

Change History (8)

#1 @lloydbudd
17 years ago

  • Milestone changed from 2.5 to 2.6

#2 @josephscott
17 years ago

  • Cc josephscott added

#3 @josephscott
16 years ago

  • Milestone 2.6 deleted
  • Resolution set to duplicate
  • Status changed from new to closed

Ticket #3405 was recently committed to -trunk [7847] to support enclosures for metaWeblog.newPost/editPost.

This ticket was mentioned in PR #6701 on WordPress/wordpress-develop by @joemcgill.


4 months ago
#4

  • Keywords has-patch has-unit-tests added

This includes a few updates to #6392 that fixes the test failures in that PR. See https://github.com/WordPress/wordpress-develop/pull/6392#discussion_r1622969056.

@joemcgill commented on PR #6701:


4 months ago
#5

Note that commit 76435fc cherry-picks the PR changes from #6696 so this should not be merged until that PR is approved.

@joemcgill commented on PR #6701:


4 months ago
#6

The changes to https://github.com/WordPress/wordpress-develop/pull/6696 are now committed to trunk, so this should be good to merge.

@joemcgill commented on PR #6701:


4 months ago
#7

The changes here have now been applied to the main PR in 2083f0b, so am closing this as no longer necessary.

Note: See TracTickets for help on using tickets.