Skip to content

Commit

Permalink
Merge pull request #10554 from woocommerce/issue/10264-media-url-null
Browse files Browse the repository at this point in the history
Crash: Media URL null returned by the API
  • Loading branch information
0nko authored Jan 16, 2024
2 parents 7b663de + 272ad60 commit 2c20f8a
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ package com.woocommerce.android.media
import android.content.Context
import android.net.Uri
import com.woocommerce.android.R
import com.woocommerce.android.extensions.isNotNullOrEmpty
import com.woocommerce.android.media.MediaFilesRepository.UploadResult.UploadFailure
import com.woocommerce.android.media.MediaFilesRepository.UploadResult.UploadSuccess
import com.woocommerce.android.tools.SelectedSite
Expand Down Expand Up @@ -145,7 +146,7 @@ class MediaFilesRepository @Inject constructor(

event.completed -> {
val media = event.media
val channelResult = if (media != null && media.url.isNotBlank()) {
val channelResult = if (media != null && media.url.isNotNullOrEmpty()) {
WooLog.i(T.MEDIA, "MediaFilesRepository > uploaded media ${media.id}")
producerScope.trySendBlocking(
UploadSuccess(media)
Expand Down

0 comments on commit 2c20f8a

Please sign in to comment.