[TIMOB-1866] Android: camera basic test case - unexpected output (type undefined and bitmap not found)
GitHub Issue | n/a |
---|---|
Type | New Feature |
Priority | Medium |
Status | Closed |
Resolution | Fixed |
Resolution Date | 2011-04-17T01:57:23.000+0000 |
Affected Version/s | n/a |
Fix Version/s | Release 1.6.0 M06 |
Components | Android |
Labels | android, camera, feature, release-1.6.0 |
Reporter | Thomas Huelbert |
Assignee | Don Thorp |
Created | 2011-04-15T03:04:15.000+0000 |
Updated | 2011-04-17T01:57:23.000+0000 |
Description
Seen on g1 running 1.6. TiMobileSDK 1.4.2.dc7b90
1.Phone>camera>basic
2.connect device to machine and adb logcat
3.take a pic and view output
results:
W/TiMedia ( 6756): (main) [12431,2869573] bitmap not found:
content://media/external/images/media/448
D/TiAPI ( 6756): (kroll$36) [35,2869608] Our type was:
undefined
Is the image actually on the SDK card?
iPhone added types to media as constants. Ideally it should have been the mimetype and not a static constant. This API needs to be reviewed.
The
bitmap not found
warning doesn't seem to occur anymore (and the image shows -- so that probably got fixed.)The
Our type was: undefined
has now changed toOur type was: null
.I went ahead and made this conform with what's happening on iOS just to get past this ticket, but it's kinda silly -- it's really apple-specific and based on http://developer.apple.com/library/iOS/#documentation/FileManagement/Conceptual/understanding_utis/understand_utis_conc/understand_utis_conc.html"> Apple UTIs. This notion of "media type" (in the context of UTIs) is not really relevant to Android. Mime type is much more interesting, seems to me.
Hopefully
Titanium.Media
is up for some API-scrubbing prior to Ti2.(from [797de3b284f49d2e2d6a3f98a0c86efaf551411e]) [#1866 state:fixed-in-qa] support the mediaType in showCamera callback args https://github.com/appcelerator/titanium_mobile/commit/797de3b284f49d2e2d6a3f98a0c86efaf551411e"> https://github.com/appcelerator/titanium_mobile/commit/797de3b284f4...
Verified on G1/1.6 and Nexus One/2.2.1 using build #e1cb22a