From c50875e42ef5af5013433d32e6a2017db1234733 Mon Sep 17 00:00:00 2001 From: Jeff Sharkey Date: Fri, 15 Jul 2016 15:34:41 -0600 Subject: [PATCH] Build both filename views before deleting. Otherwise we'll throw when trying to create the second (visible) filename because we just deleted it. Bug: 30087518 Change-Id: I940a8f81c264fc17d477fadf46fd48eae29f2f6f --- .../com/android/externalstorage/ExternalStorageProvider.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/packages/ExternalStorageProvider/src/com/android/externalstorage/ExternalStorageProvider.java b/packages/ExternalStorageProvider/src/com/android/externalstorage/ExternalStorageProvider.java index a3070bddd0f81..78b99274e444b 100644 --- a/packages/ExternalStorageProvider/src/com/android/externalstorage/ExternalStorageProvider.java +++ b/packages/ExternalStorageProvider/src/com/android/externalstorage/ExternalStorageProvider.java @@ -484,6 +484,8 @@ public class ExternalStorageProvider extends DocumentsProvider { @Override public void deleteDocument(String docId) throws FileNotFoundException { final File file = getFileForDocId(docId); + final File visibleFile = getFileForDocId(docId, true); + final boolean isDirectory = file.isDirectory(); if (isDirectory) { FileUtils.deleteContents(file); @@ -492,7 +494,6 @@ public class ExternalStorageProvider extends DocumentsProvider { throw new IllegalStateException("Failed to delete " + file); } - final File visibleFile = getFileForDocId(docId, true); if (visibleFile != null) { final ContentResolver resolver = getContext().getContentResolver(); final Uri externalUri = MediaStore.Files.getContentUri("external");