Make WordPress Core

Opened 18 years ago

Closed 18 years ago

Last modified 2 years 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 Owned by:
Priority: normal Milestone:
Component: XML-RPC Version: 2.5
Severity: normal Keywords: enclosure xmlrpc editPost newPost meta meta_value has-patch has-unit-tests
Cc: Focuses:

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 18 years ago.

Download all attachments as: .zip

Change History (8)

#1 @lloydbudd
18 years ago

  • Milestone 2.52.6

#2 @josephscott
18 years ago

  • Cc josephscott added

#3 @josephscott
18 years ago

  • Milestone 2.6
  • Resolutionduplicate
  • Status newclosed

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.


2 years 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:


2 years 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:


2 years 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:


2 years 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.