Merge pull request #14 from egbuna/fix/not-null-constraint

fix: non null constraint on tv poster path
skydoves · Apr 25, 2020 · a286f35 · a286f35
2 parents 82c1079 + 08dddcd
commit a286f35
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/app/src/main/java/com/skydoves/themovies/binding/ViewBinding.kt b/app/src/main/java/com/skydoves/themovies/binding/ViewBinding.kt
@@ -110,7 +110,7 @@ fun bindBackDrop(view: ImageView, tv: Tv) {
     Glide.with(view.context).load(Api.getBackdropPath(tv.backdrop_path))
       .listener(view.requestGlideListener())
       .into(view)
-  } else {
+  } else if (tv.poster_path != null) {
     Glide.with(view.context).load(Api.getBackdropPath(tv.poster_path))
       .listener(view.requestGlideListener())
       .into(view)

diff --git a/app/src/main/java/com/skydoves/themovies/models/entity/Tv.kt b/app/src/main/java/com/skydoves/themovies/models/entity/Tv.kt
@@ -40,7 +40,7 @@ data class Tv(
   var keywords: List<Keyword>? = ArrayList(),
   var videos: List<Video>? = ArrayList(),
   var reviews: List<Review>? = ArrayList(),
-  val poster_path: String,
+  val poster_path: String?,
   val popularity: Float,
   val backdrop_path: String?,
   val vote_average: Float,

diff --git a/app/src/main/java/com/skydoves/themovies/view/viewholder/TvListViewHolder.kt b/app/src/main/java/com/skydoves/themovies/view/viewholder/TvListViewHolder.kt
@@ -55,7 +55,7 @@ class TvListViewHolder constructor(
       item_poster_title.text = tv.name
       tv.poster_path.let {
         Glide.with(context)
-          .load(Api.getPosterPath(it))
+          .load(Api.getPosterPath(it!!))
           .listener(GlidePalette.with(Api.getPosterPath(it))
             .use(BitmapPalette.Profile.VIBRANT)
             .intoBackground(item_poster_palette)