You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by "Joe Bowser (JIRA)" <ji...@apache.org> on 2013/02/21 01:13:12 UTC

[jira] [Comment Edited] (CB-2501) FileUtils.getMimeType(String file) does not work for content:// URIs

    [ https://issues.apache.org/jira/browse/CB-2501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13582702#comment-13582702 ] 

Joe Bowser edited comment on CB-2501 at 2/21/13 12:11 AM:
----------------------------------------------------------

This is good for now, but normally I'd create a new bug because it's trying to solve a different, but related problem.  We desperately need to fix how we deal with contributions, our current system with git pull requests is terrible.

Also, wouldn't MimeTypeMap.getFileExtensionFromUrl(url).toLowerCase() case a NPE, since the URL would still be in upper case? 
                
      was (Author: bowserj):
    This is good for now, but normally I'd create a new bug because it's trying to solve a different problem.  We desperately need to fix how we deal with contributions, our current system with git pull requests is terrible.
                  
> FileUtils.getMimeType(String file) does not work for content:// URIs
> --------------------------------------------------------------------
>
>                 Key: CB-2501
>                 URL: https://issues.apache.org/jira/browse/CB-2501
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Android
>    Affects Versions: 2.2.0
>            Reporter: Christian Kuetbach
>            Assignee: Max Woghiren
>            Priority: Minor
>             Fix For: 2.4.0
>
>
> There is a bug in Android: http://code.google.com/p/android/issues/detail?id=8806
> Because of this the returned mime type will be wrong, if the file extension is not lowercase.
> myImage.jpg -> image/jpg
> myImage.JPG -> null
> I will provide a patch, which simply converts the filepath to lower case, but think there is a better solution than using the MimeTypeMap. The Class ContentResolver has a method getType() 
> http://developer.android.com/reference/android/content/ContentResolver.html#getType(android.net.Uri)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira