Fix: After updating a blog post, ViewBlogFragment and BlogFragment must reflect the changes.

Author: mitchtabian

app/src/main/java/com/codingwithmitch/openapi/repository/main/BlogRepositoryImpl.kt

@@ -242,6 +242,11 @@ constructor(
                         data =  BlogViewState(
                             viewBlogFields = ViewBlogFields(
                                 blogPost = updatedBlogPost
+                            ),
+                            updatedBlogFields = UpdatedBlogFields(
+                                updatedBlogTitle = updatedBlogPost.title,
+                                updatedBlogBody = updatedBlogPost.body,
+                                updatedImageUri = null
                             )
                         ),
                         stateEvent = stateEvent

app/src/main/java/com/codingwithmitch/openapi/ui/main/account/ChangePasswordFragment.kt

@@ -64,6 +64,12 @@ constructor(
         viewModel.stateMessage.observe(viewLifecycleOwner, Observer { stateMessage ->
 
             stateMessage?.let {
+
+                if(stateMessage.response.message.equals(RESPONSE_PASSWORD_UPDATE_SUCCESS)){
+                    uiCommunicationListener.hideSoftKeyboard()
+                    findNavController().popBackStack()
+                }
+
                 uiCommunicationListener.onResponseReceived(
                     response = it.response,
                     stateMessageCallback = object: StateMessageCallback {

app/src/main/java/com/codingwithmitch/openapi/ui/main/blog/UpdateBlogFragment.kt

@@ -13,13 +13,15 @@ import androidx.lifecycle.lifecycleScope
 import androidx.navigation.fragment.findNavController
 import com.bumptech.glide.RequestManager
 import com.codingwithmitch.openapi.R
+import com.codingwithmitch.openapi.models.BlogPost
 import com.codingwithmitch.openapi.ui.main.blog.state.BLOG_VIEW_STATE_BUNDLE_KEY
 import com.codingwithmitch.openapi.ui.main.blog.state.BlogStateEvent
 import com.codingwithmitch.openapi.ui.main.blog.state.BlogViewState
 import com.codingwithmitch.openapi.ui.main.blog.viewmodel.*
 import com.codingwithmitch.openapi.util.*
 import com.codingwithmitch.openapi.util.Constants.Companion.GALLERY_REQUEST_CODE
 import com.codingwithmitch.openapi.util.ErrorHandling.Companion.SOMETHING_WRONG_WITH_IMAGE
+import com.codingwithmitch.openapi.util.SuccessHandling.Companion.SUCCESS_BLOG_UPDATED
 import com.theartofdev.edmodo.cropper.CropImage
 import com.theartofdev.edmodo.cropper.CropImageView
 import kotlinx.android.synthetic.main.fragment_update_blog.*
@@ -164,6 +166,11 @@ constructor(
         viewModel.stateMessage.observe(viewLifecycleOwner, Observer { stateMessage ->
 
             stateMessage?.let {
+
+                if(stateMessage.response.message.equals(SUCCESS_BLOG_UPDATED)){
+                   viewModel.updateListItem()
+                }
+
                 uiCommunicationListener.onResponseReceived(
                     response = it.response,
                     stateMessageCallback = object: StateMessageCallback {
@@ -177,9 +184,11 @@ constructor(
     }
 
     fun setBlogProperties(title: String?, body: String?, image: Uri?){
-        requestManager
-            .load(image)
-            .into(blog_image)
+        image?.let {
+            requestManager
+                .load(it)
+                .into(blog_image)
+        }
         blog_title.setText(title)
         blog_body.setText(body)
     }

app/src/main/java/com/codingwithmitch/openapi/ui/main/blog/viewmodel/Setters.kt

@@ -103,10 +103,11 @@ fun BlogViewModel.removeDeletedBlogPost(){
 
 @FlowPreview
 @UseExperimental(ExperimentalCoroutinesApi::class)
-fun BlogViewModel.updateListItem(newBlogPost: BlogPost){
+fun BlogViewModel.updateListItem(){
     val update = getCurrentViewStateOrNew()
     val list = update.blogFields.blogList?.toMutableList()
     if(list != null){
+        val newBlogPost = getBlogPost()
         for(i in 0..(list.size - 1)){
             if(list[i].pk == newBlogPost.pk){
                 list[i] = newBlogPost
@@ -118,12 +119,20 @@ fun BlogViewModel.updateListItem(newBlogPost: BlogPost){
     }
 }
 
-@FlowPreview
-@UseExperimental(ExperimentalCoroutinesApi::class)
-fun BlogViewModel.onBlogPostUpdateSuccess(blogPost: BlogPost){
-    setBlogPost(blogPost) // update ViewBlogFragment
-    updateListItem(blogPost) // update BlogFragment
-}
+//@FlowPreview
+//@UseExperimental(ExperimentalCoroutinesApi::class)
+//fun BlogViewModel.onBlogPostUpdateSuccess(
+//    title: String,
+//    body: String,
+//    image: String
+//){
+//    val currentBlogPost = getBlogPost()
+//    currentBlogPost.title = title
+//    currentBlogPost.body = body
+//    currentBlogPost.image = image
+//    setBlogPost(currentBlogPost) // update ViewBlogFragment
+//    updateListItem(currentBlogPost) // update BlogFragment
+//}
 
 
 @FlowPreview

app/src/main/java/com/codingwithmitch/openapi/util/SuccessHandling.kt

@@ -11,6 +11,7 @@ class SuccessHandling {
         const val RESPONSE_HAS_PERMISSION_TO_EDIT = "You have permission to edit that."
         const val SUCCESS_BLOG_CREATED = "created"
         const val SUCCESS_BLOG_DELETED = "deleted"
+        const val SUCCESS_BLOG_UPDATED = "updated"
 
     }
 }