Make WordPress Core

Opened 19 years ago

Closed 12 years ago

#5310 closed enhancement (fixed)

XMLRPC interface should expose mechanism for listing and deleting media resources

Reported by: redsweater Owned by: SergeyBiryukov
Priority: normal Milestone: 4.0
Component: XML-RPC Version: 2.3.1
Severity: normal Keywords: good-first-bug has-patch commit
Cc: Focuses:

Description

Right now the XMLRPC interface supports the "newMediaObject" mechanism that is part of the MetaWeblog API, but lacks any method for inspecting or otherwise manipulating media objects.

I think to support a richer remote client experience, the XLMRPC interface should support at least a method for listing the media objects that have been previously uploaded (similar to what appears in the "Browse" tab of the wp-admin post editor), and for deleting a media asset that is present.

Attachments (2)

5310_patch.diff (652 bytes ) - added by fahmiadib 12 years ago.
5310.diff (1.0 KB ) - added by SergeyBiryukov 12 years ago.

Download all attachments as: .zip

Change History (20)

#1 @josephscott
19 years ago

  • Cc josephscott added

#2 @Denis-de-Bernardy
17 years ago

itching to move this in Media

#3 @westi
17 years ago

  • Keywords needs-patch added
  • Milestone 2.9Future Release

Needs a patch.

Moving to Future Release for now.

#4 follow-up: @johnbillion
13 years ago

  • Keywords good-first-bug added

We now have a method for listing items in the media library (wp.getMediaLibrary) but still none for deleting media items.

#5 in reply to: ↑ 4 @maxcutler
13 years ago

Replying to johnbillion:

... but still none for deleting media items.

Actually, since media items are really just a post type, you can use wp.deletePost method to delete them. Likewise, you can use wp.editPost to modify most of the media item fields (except image and alt-text, the latter of which is a protected meta).

It might be nice to have a simple wrapper to make this more obvious to new users, or else add some documentation to the Codex.

#6 @markoheijnen
13 years ago

That was exactly the reason why it wasn't added in 3.4 or 3.5

#7 @nacin
13 years ago

  • Owner josephscott removed
  • Status newassigned

#8 @nacin
12 years ago

We could just add an alias in the methods list.

#9 @fahmiadib
12 years ago

Add wp.deleteMediaItem as an alias to wp_deletePost

#10 @SergeyBiryukov
12 years ago

  • Keywords has-patch added; needs-patch removed
  • Milestone Future Release4.0

#11 @samuelsidler
12 years ago

  • Keywords commit added

This ticket was mentioned in IRC in #wordpress-dev by helen. View the logs.


12 years ago

#13 @SergeyBiryukov
12 years ago

  • Owner set to SergeyBiryukov
  • Resolutionfixed
  • Status assignedclosed

In 28849:

XML-RPC: Add wp.deleteMediaItem as an alias to wp_deletePost.

props fahmiadib.
fixes #5310.

@SergeyBiryukov
12 years ago

#14 @SergeyBiryukov
12 years ago

  • Resolution fixed
  • Status closedreopened

I wonder if the alias should be called wp.deleteFile, for consistency with wp.uploadFile. See 5310.diff.

Version 0, edited 12 years ago by SergeyBiryukov (next)

This ticket was mentioned in IRC in #wordpress-dev by SergeyBiryukov. View the logs.


12 years ago

#16 follow-up: @DrewAPicture
12 years ago

+1 for swapping the aliases.

#17 in reply to: ↑ 16 @westi
12 years ago

Replying to DrewAPicture:

+1 for swapping the aliases.

Agreed.

I think wp.deleteFile is a better name and more consistent with the others.

#18 @westi
12 years ago

  • Resolutionfixed
  • Status reopenedclosed

In 29255:

XMLRPC: Switch the file deletion alias to be deleteFile to more closely make uploadFile.

Fixes #5310 props SergeyBiryukov.

Note: See TracTickets for help on using tickets.