summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorColin Marsch <colinmarsch@google.com>2020-08-14 16:37:08 -0500
committerColin Marsch <colinmarsch@google.com>2020-08-14 16:41:49 -0500
commitd85ec99fb5af9beee36d7aec6732d43017502a0d (patch)
treefca4541b448049bb6e47143a5b080b6423a42f04 /src
parentd36fa085d6acb9976a2c3810eceef92c5daba9e1 (diff)
downloadCamera2-d85ec99fb5af9beee36d7aec6732d43017502a0d.tar.gz
AOSP/Camera2 - Fix bug causing edit, share, delete buttons to not appear
The edit, share and delete buttons were not showing up on the images in the filmstrip view. This was caused by the recent storage API changes causing the images to be not found at the same path as before. Test: manual, ensured the Camera2 app UI functioned correctly and the buttons appear as desired in the filmstrip view. BUG: 163082488 Change-Id: I3e99c982c733e48c63457ffe9c21b259ef53bb3f
Diffstat (limited to 'src')
-rw-r--r--src/com/android/camera/data/FilmstripContentQueries.java6
-rw-r--r--src/com/android/camera/data/FilmstripItemBase.java3
2 files changed, 4 insertions, 5 deletions
diff --git a/src/com/android/camera/data/FilmstripContentQueries.java b/src/com/android/camera/data/FilmstripContentQueries.java
index 2aba065f3..4b607d40a 100644
--- a/src/com/android/camera/data/FilmstripContentQueries.java
+++ b/src/com/android/camera/data/FilmstripContentQueries.java
@@ -32,7 +32,6 @@ import java.util.List;
*/
public class FilmstripContentQueries {
private static final Log.Tag TAG = new Log.Tag("LocalDataQuery");
- private static final String SELECT_BY_PATH = MediaStore.MediaColumns.DATA + " LIKE ?";
public interface CursorToFilmstripItemFactory<I extends FilmstripItem> {
@@ -60,9 +59,8 @@ public class FilmstripContentQueries {
public static <I extends FilmstripItem> List<I> forCameraPath(ContentResolver contentResolver,
Uri contentUri, String[] projection, long minimumId, String orderBy,
CursorToFilmstripItemFactory<I> factory) {
- String selection = SELECT_BY_PATH + " AND " + MediaStore.MediaColumns._ID + " > ?";
- String cameraPath = Storage.instance().DIRECTORY + "%";
- String[] selectionArgs = new String[] { cameraPath, Long.toString(minimumId) };
+ String selection = MediaStore.MediaColumns._ID + " > ?";
+ String[] selectionArgs = new String[] { Long.toString(minimumId) };
Cursor cursor = contentResolver.query(contentUri, projection,
selection, selectionArgs, orderBy);
diff --git a/src/com/android/camera/data/FilmstripItemBase.java b/src/com/android/camera/data/FilmstripItemBase.java
index e3c589f99..1cc73c47b 100644
--- a/src/com/android/camera/data/FilmstripItemBase.java
+++ b/src/com/android/camera/data/FilmstripItemBase.java
@@ -141,7 +141,8 @@ public abstract class FilmstripItemBase<T extends FilmstripItemData> implements
private void deleteIfEmptyCameraSubDir(File directory) {
// Make sure 'directory' refers to a valid existing empty directory.
- if (!directory.exists() || !directory.isDirectory() || directory.list().length != 0) {
+ if (!directory.exists() || !directory.isDirectory()
+ || directory.list() == null || directory.list().length != 0) {
return;
}