From 17d18daaf591b4ab27e0a7df0bb1c3126e6488be Mon Sep 17 00:00:00 2001 From: Allan Wang Date: Sat, 19 Aug 2017 00:50:34 -0700 Subject: Remove excess null check (#41) --- .../kotlin/ca/allanwang/kau/mediapicker/MediaPickerCore.kt | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) (limited to 'mediapicker/src') diff --git a/mediapicker/src/main/kotlin/ca/allanwang/kau/mediapicker/MediaPickerCore.kt b/mediapicker/src/main/kotlin/ca/allanwang/kau/mediapicker/MediaPickerCore.kt index 6f0241c..eada173 100644 --- a/mediapicker/src/main/kotlin/ca/allanwang/kau/mediapicker/MediaPickerCore.kt +++ b/mediapicker/src/main/kotlin/ca/allanwang/kau/mediapicker/MediaPickerCore.kt @@ -251,10 +251,6 @@ abstract class MediaPickerCore>( return super.onActivityResult(requestCode, resultCode, data) } KL.d("Media result received") - if (data == null) { - KL.d("Media null intent") - return super.onActivityResult(requestCode, resultCode, data) - } when (requestCode) { MEDIA_ACTION_REQUEST_CAMERA -> onCameraResult(data) MEDIA_ACTION_REQUEST_PICKER -> onPickerResult(data) @@ -262,12 +258,12 @@ abstract class MediaPickerCore>( } } - private fun onCameraResult(data: Intent) { + private fun onCameraResult(data: Intent?) { val f: File if (tempPath != null) { f = File(tempPath) tempPath = null - } else if (data.data != null) { + } else if (data?.data != null) { f = File(data.data.path) } else { KL.d("Media camera no file found") @@ -282,12 +278,12 @@ abstract class MediaPickerCore>( } } - private fun onPickerResult(data: Intent) { + private fun onPickerResult(data: Intent?) { val items = mutableListOf() - if (data.data != null) { + if (data?.data != null) { KL.v("Media picker data uri", data.data.path) items.add(data.data) - } else { + } else if (data != null) { val clip = data.clipData if (clip != null) { items.addAll((0 until clip.itemCount).map { -- cgit v1.2.3