diff options
| author | yongga <yongga@codeaurora.org> | 2016-01-13 10:12:19 +0800 |
|---|---|---|
| committer | yongga <yongga@codeaurora.org> | 2016-01-13 10:12:19 +0800 |
| commit | 7d47cf1e979b611683dc90c144bed4188dcc1fe4 (patch) | |
| tree | 1baac2316472da313bfed05a93d371a7d445a69a /src/com/android | |
| parent | 24af2c506c4b7ad200669cc951c43f664228eac9 (diff) | |
| download | android_packages_apps_Gallery2-7d47cf1e979b611683dc90c144bed4188dcc1fe4.tar.gz android_packages_apps_Gallery2-7d47cf1e979b611683dc90c144bed4188dcc1fe4.tar.bz2 android_packages_apps_Gallery2-7d47cf1e979b611683dc90c144bed4188dcc1fe4.zip | |
Gallery2: fix the bug that only delete parts of photos when select all.
DataManager.getMediaObject() may return null value, so DataManager.delete()
and DataManager.getSupportedOperations() will be interrupted
and only parts of photos were deleted.
Add null value check, ensure the DataManager.delete() and
DataManager.getSupportedOperations() can execute normally.
Change-Id: If5a4f1118d9a03d9d70c2e237347d20b3142b031
CRs-Fixed: 954040
Diffstat (limited to 'src/com/android')
| -rw-r--r-- | src/com/android/gallery3d/data/DataManager.java | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/src/com/android/gallery3d/data/DataManager.java b/src/com/android/gallery3d/data/DataManager.java index 38865e9f1..5c7664cb9 100644 --- a/src/com/android/gallery3d/data/DataManager.java +++ b/src/com/android/gallery3d/data/DataManager.java @@ -242,7 +242,12 @@ public class DataManager implements StitchingChangeListener { // The following methods forward the request to the proper object. public int getSupportedOperations(Path path) { - return getMediaObject(path).getSupportedOperations(); + int support = 0; + MediaObject mediaObject = getMediaObject(path); + if (mediaObject != null) { + support = mediaObject.getSupportedOperations(); + } + return support; } public void getPanoramaSupport(Path path, PanoramaSupportCallback callback) { @@ -250,7 +255,10 @@ public class DataManager implements StitchingChangeListener { } public void delete(Path path) { - getMediaObject(path).delete(); + MediaObject mediaObject = getMediaObject(path); + if (mediaObject != null) { + mediaObject.delete(); + } } public void rotate(Path path, int degrees) { |
