From ddb5654b9fd6b0cb8343d59359e6c59dd093ce82 Mon Sep 17 00:00:00 2001
From: tibbi <tibor@kaputa.sk>
Date: Sat, 8 Dec 2018 23:04:20 +0100
Subject: [PATCH] toggle checklist item checked state on click

---
 .../notes/pro/adapters/ChecklistAdapter.kt    |  2 +-
 .../notes/pro/fragments/ChecklistFragment.kt  | 27 +++++++++++++------
 2 files changed, 20 insertions(+), 9 deletions(-)

diff --git a/app/src/main/kotlin/com/simplemobiletools/notes/pro/adapters/ChecklistAdapter.kt b/app/src/main/kotlin/com/simplemobiletools/notes/pro/adapters/ChecklistAdapter.kt
index cff0de39..5d80ea19 100644
--- a/app/src/main/kotlin/com/simplemobiletools/notes/pro/adapters/ChecklistAdapter.kt
+++ b/app/src/main/kotlin/com/simplemobiletools/notes/pro/adapters/ChecklistAdapter.kt
@@ -52,7 +52,7 @@ class ChecklistAdapter(activity: BaseSimpleActivity, var items: ArrayList<Checkl
 
     override fun getItemCount() = items.size
 
-    fun initDrawables() {
+    private fun initDrawables() {
         val res = activity.resources
         crossDrawable = res.getColoredDrawableWithColor(R.drawable.ic_cross_big, res.getColor(R.color.theme_dark_red_primary_color))
         checkDrawable = res.getColoredDrawableWithColor(R.drawable.ic_check_big, res.getColor(R.color.md_green_700))
diff --git a/app/src/main/kotlin/com/simplemobiletools/notes/pro/fragments/ChecklistFragment.kt b/app/src/main/kotlin/com/simplemobiletools/notes/pro/fragments/ChecklistFragment.kt
index 5e5c8eb3..de82569f 100644
--- a/app/src/main/kotlin/com/simplemobiletools/notes/pro/fragments/ChecklistFragment.kt
+++ b/app/src/main/kotlin/com/simplemobiletools/notes/pro/fragments/ChecklistFragment.kt
@@ -64,13 +64,7 @@ class ChecklistFragment : NoteFragment(), RefreshRecyclerViewListener {
                     val currentMaxId = items.maxBy { it.id }?.id ?: 0
                     val checklistItem = ChecklistItem(currentMaxId + 1, it, false)
                     items.add(checklistItem)
-                    setupAdapter()
-                    Thread {
-                        if (note != null && context != null) {
-                            note!!.value = Gson().toJson(items)
-                            context?.notesDB?.insertOrUpdate(note!!)
-                        }
-                    }.start()
+                    saveNote(-1)
                 }
             }
         }
@@ -79,12 +73,29 @@ class ChecklistFragment : NoteFragment(), RefreshRecyclerViewListener {
 
     private fun setupAdapter() {
         ChecklistAdapter(activity as SimpleActivity, items, this, view.checklist_list) {
-
+            val clickedNote = it as ChecklistItem
+            clickedNote.isDone = !clickedNote.isDone
+            saveNote(items.indexOfFirst { it.id == clickedNote.id })
         }.apply {
             view.checklist_list.adapter = this
         }
     }
 
+    private fun saveNote(refreshIndex: Int) {
+        Thread {
+            if (note != null && context != null) {
+                if (refreshIndex != -1) {
+                    view.checklist_list.post {
+                        view.checklist_list.adapter?.notifyItemChanged(refreshIndex)
+                    }
+                }
+
+                note!!.value = Gson().toJson(items)
+                context?.notesDB?.insertOrUpdate(note!!)
+            }
+        }.start()
+    }
+
     override fun refreshItems() {
     }
 }