fix: NPE when image url is not given (#3863)

openfoodfacts · Mar 3, 2021 · 87b4bd7 · 87b4bd7
1 parent 57a6272
commit 87b4bd7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...foodfacts/github/scrachx/openfood/features/product/view/summary/SummaryProductFragment.kt b/...foodfacts/github/scrachx/openfood/features/product/view/summary/SummaryProductFragment.kt
@@ -811,7 +811,7 @@ class SummaryProductFragment : BaseFragment(), ISummaryProductPresenter.View {
                 productBarcode,
                 productName.orEmpty(),
                 productDetails,
-                imageUrl!!
+                imageUrl.orEmpty()
         )
         addToListRecyclerView.layoutManager = LinearLayoutManager(activity)
         addToListRecyclerView.adapter = addToListAdapter

diff --git a/...ain/java/openfoodfacts/github/scrachx/openfood/features/productlist/ProductListAdapter.kt b/...ain/java/openfoodfacts/github/scrachx/openfood/features/productlist/ProductListAdapter.kt
@@ -35,7 +35,7 @@ class ProductListAdapter(
         holder.tvDetails.text = products[position].productDetails
         holder.tvBarcode.text = products[position].barcode
 
-        if (!isLowBatteryMode) {
+        if (!isLowBatteryMode && products[position].imageUrl.isNotEmpty()) {
             Picasso.get()
                     .load(products[position].imageUrl)
                     .placeholder(R.drawable.placeholder_thumb)