mirror/opencamera
0
0
Fork 0
mirror of https://git.code.sf.net/p/opencamera/code.git synced 2024-09-19 19:42:29 +02:00
Code

Prefer using MediaStore.getMediaUri() to get the mediastore URI from a SAF URI.

Browse Source
This commit is contained in:
Mark Harman 2024-02-28 21:47:53 +00:00
parent a041de3ef1
commit b2926637e4
2 changed files with 25 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
_docs/history.html
View File

@ -58,6 +58,8 @@ FIXED If force destroyed when in settings, the camera would be incorrectly ope
was recreated (camera should only be reopened when leaving settings).
FIXED Long pressing on the shutter button in video mode meant nothing happened when releasing
touch.
FIXED Clicking on gallery icon when using Storage Access Framework would open contacts on some
devices.
ADDED Camera vendor extensions show percentage progress on supported Android 14 devices.
ADDED Long press on switch camera icons now bring up a menu to jump to any camera (for devices
that expose multiple cameras).

24
app/src/main/java/net/sourceforge/opencamera/StorageUtils.java
View File

@ -260,8 +260,10 @@ public class StorageUtils {
* call this function for DNGs, so that they show up on MTP.
*/
public void broadcastFile(final File file, final boolean is_new_picture, final boolean is_new_video, final boolean set_last_scanned, final boolean hasnoexifdatetime, final Uri saf_uri) {
if( MyDebug.LOG )
if( MyDebug.LOG ) {
Log.d(TAG, "broadcastFile: " + file.getAbsolutePath());
Log.d(TAG, "saf_uri: " + saf_uri);
}
// note that the new method means that the new folder shows up as a file when connected to a PC via MTP (at least tested on Windows 8)
if( file.isDirectory() ) {
//this.sendBroadcast(new Intent(Intent.ACTION_MEDIA_MOUNTED, Uri.fromFile(file)));
@ -283,6 +285,26 @@ public class StorageUtils {
Log.d(TAG, "Scanned " + path + ":");
Log.d(TAG, "-> uri=" + uri);
}
if( saf_uri != null && Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q ) {
// Prefer using MediaStore.getMediaUri() to get the mediastore URI from a SAF URI.
// Fixes bug on Pixel 6 Pro with SAF where the URI recieved by onScanCompleted() is
// of the form =content://media/external_primary/images/media/123456, when this is not
// recognised by gallery apps (causes strange bug where clicking on gallery icon opens
// contacts!) The correct URI is returned by MediaStore.getMediaUri(), and (for
// Pixel 6 Pro at least) is of the form content://media/external_primary/file/123456.
try {
Uri media_uri_from_saf_uri = MediaStore.getMediaUri(context, saf_uri);
if( media_uri_from_saf_uri != null ) {
uri = media_uri_from_saf_uri;
if( MyDebug.LOG ) {
Log.d(TAG, "prefer getMediaUri from SAF: " + uri);
}
}
}
catch(Exception e) {
e.printStackTrace();
}
}
if( set_last_scanned ) {
boolean is_raw = filenameIsRaw(file.getName());
setLastMediaScanned(uri, is_raw, hasnoexifdatetime, saf_uri != null ? saf_uri : uri);