Styjun

I have an AllMoviesActivity and Two Fragments. First One is MovieFragment and another one is UpdateMovieFragment.

I am showing all the movies in a RecyclerView in MovieFragment. The structure of Movie model class is:

class Movie() : Parcelable 
 var movieID: String? = ""
 var movieName: String? = ""
 var movieImage: Uri? = null
 ...

Whenever I click on the ImageView of the Movie Item in RecyclerView, It goes to UpdateMovieFragment where I select the image for the movie and enter the name of the movie. I am doing the transition from MovieFragment to UpdateMovieFragment from the MovieAdapter class by setting the Movie Item as Fragment's Argument.

val fragment = UpdateMovieFragment()
 val bundle = Bundle()
 bundle.putParcelable("movie", allMovies[p1])
 fragment.arguments = bundle
 val transaction = (activity as FragmentActivity).supportFragmentManager.beginTransaction()
 transaction.addToBackStack(null)
 transaction.replace(R.id.frame_layout, fragment).commit()

Now, I am setting the current time as MovieID to each movie to keep it unique so that I can update that specific Movie Item at RecyclerView Later.

Now, I am using a menu button in the activity for the submit operation of both fragments. Whenever the submit button is clicked, I am checking which fragment is on display. If it is UpdateMovieFragment, then my goal is to return the updated movie to the MovieFragment.

override fun onOptionsItemSelected(item: MenuItem): Boolean 
 // Handle action bar item clicks here. The action bar will
 // automatically handle clicks on the Home/Up button, so long
 // as you specify a parent activity in AndroidManifest.xml.
 return when (item.itemId) 
 R.id.action_save -> 
 val fragment = supportFragmentManager.findFragmentById(R.id.list_fragment_container)
 when (fragment) 
 is UpdateMovieFragment -> 
 supportFragmentManager.popBackStack()

 val fragmentNew = supportFragmentManager.findFragmentByTag("LIST")
 if (fragmentNew is MovieFragment) 
 fragmentNew.refreshMovies(fragment.movie)
 else 
 CommonMethods().showToast(applicationContext, "UpdateMovieFragment")
 
 

 is MovieFragment -> 
 CommonMethods().showToast(applicationContext, "Do nothing for now")
 
 
 true
 
 else -> super.onOptionsItemSelected(item)
 
 

Here, I am not receiving the updated movie in MovieFragment. For the refreshMovies method, I am doing this:

fun refreshMovies(movie: Movie) 
 for (i in 0 until movieItems.size) 
 val secondItem = movieItems[i]

 if (movie.movieID == secondItem.movieID) 
 rankItems[i] = rankItem
 break
 
 

 itemAdapter.notifyDataSetChanged()
 

What is the best way to solve this?