#12518 closed defect (bug) (invalid)
XML-RPC newMediaObject should try to work out mime-type if not supplied
| Reported by: | jonquark | Owned by: | josephscott |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | XML-RPC | Version: | |
| Severity: | normal | Keywords: | |
| Cc: | Focuses: |
Description
At least one XML-RPC client (MaStory on Maemo) does not set the type parameter on uploaded objects. This means we don't create appropriate thumbnails correctly.
Given some clients are on fairly low-power, simple devices, it's probably worth Wordpress attempting to determine the mime-type, especially as functions calls we /already/ make on attachments will give us that information already for the common-case of image uploads.
Attachments (2)
Change History (8)
#2
@
17 years ago
- Cc added
I'd suggest running this through get_allowed_mime_types() before assuming that the upload is an image.
#3
@
17 years ago
Hi Joseph,
When I came to address your comment, I realised that there is a wp_check_filetype() that works with a broader range of mime types.
I've attached a version of the patch that uses it (and also stops passing type to wp_upload_bits as that is deprecated in favour of a call to wp_check_filetype()).
Does that seem reasonable?
#5
@
12 years ago
- Keywords close needs-refresh added; has-patch removed
- Resolution → invalid
- Status new → closed
The XMLRPC uploader now calls mw_newMediaObject in 4.0 which has the changed type param in the wp_upload_bits call to NULL. wp_upload_bits also now checks the type of file using wp_check_filetype() so we're good to go on that front to. This patch (which would need to be refreshed anyways) is no longer needed.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
If not supplied, determine mime-type for uploaded images