refactor: Use "uncheck all items" uniformly

Author: TopFox

app/src/main/kotlin/org/fossify/notes/activities/MainActivity.kt

@@ -269,7 +269,7 @@ class MainActivity : SimpleActivity() {
             findItem(R.id.delete_note).isVisible = multipleNotesExist
             findItem(R.id.open_search).isVisible = !isCurrentItemChecklist
             findItem(R.id.remove_done_items).isVisible = isCurrentItemChecklist
-            findItem(R.id.uncheck_done_items).isVisible = isCurrentItemChecklist
+            findItem(R.id.uncheck_all_items).isVisible = isCurrentItemChecklist
             findItem(R.id.sort_checklist).isVisible = isCurrentItemChecklist
             findItem(R.id.import_folder).isVisible = !isQPlus()
             findItem(R.id.lock_note).isVisible =
@@ -317,7 +317,7 @@ class MainActivity : SimpleActivity() {
                 R.id.settings -> launchSettings()
                 R.id.about -> launchAbout()
                 R.id.remove_done_items -> fragment?.handleUnlocking { removeDoneItems() }
-                R.id.uncheck_done_items -> fragment?.handleUnlocking { uncheckDoneItems() }
+                R.id.uncheck_all_items -> fragment?.handleUnlocking { uncheckAllItems() }
                 R.id.sort_checklist -> fragment?.handleUnlocking { displaySortChecklistDialog() }
                 else -> return@setOnMenuItemClickListener false
             }
@@ -1557,8 +1557,8 @@ class MainActivity : SimpleActivity() {
         getPagerAdapter().removeDoneCheckListItems(binding.viewPager.currentItem)
     }
 
-    private fun uncheckDoneItems() {
-        getPagerAdapter().uncheckCheckedItems(binding.viewPager.currentItem)
+    private fun uncheckAllItems() {
+        getPagerAdapter().uncheckAllItems(binding.viewPager.currentItem)
     }
 
     private fun displaySortChecklistDialog() {

app/src/main/kotlin/org/fossify/notes/adapters/NotesPagerAdapter.kt

@@ -96,8 +96,8 @@ class NotesPagerAdapter(fm: FragmentManager, val notes: List<Note>, val activity
         (fragments[position] as? TasksFragment)?.removeCheckedItems()
     }
 
-    fun uncheckCheckedItems(position: Int) {
-        (fragments[position] as? TasksFragment)?.uncheckCheckedItems()
+    fun uncheckAllItems(position: Int) {
+        (fragments[position] as? TasksFragment)?.uncheckAllItems()
     }
 
     fun refreshChecklist(position: Int) {

app/src/main/kotlin/org/fossify/notes/fragments/TasksFragment.kt

@@ -256,7 +256,7 @@ class TasksFragment : NoteFragment(), TasksActionListener {
         setupAdapter()
     }
 
-    fun uncheckCheckedItems() {
+    fun uncheckAllItems() {
         tasks = tasks.map { it.copy(isDone = false) }.toMutableList()
         saveAndReload()
     }

app/src/main/res/menu/menu.xml

@@ -41,8 +41,8 @@
         android:title="@string/delete_checked_items"
         app:showAsAction="never" />
     <item
-        android:id="@+id/uncheck_done_items"
-        android:title="@string/uncheck_checked_items"
+        android:id="@+id/uncheck_all_items"
+        android:title="@string/uncheck_all_items"
         app:showAsAction="never" />
     <item
         android:id="@+id/sort_checklist"

app/src/main/res/values/strings.xml

@@ -71,7 +71,7 @@
     <string name="add_new_checklist_items">Add new checklist items</string>
     <string name="checklist_is_empty">The checklist is empty</string>
     <string name="delete_checked_items">Delete checked items</string>
-    <string name="uncheck_checked_items">Uncheck checked items</string>
+    <string name="uncheck_all_items">Uncheck all items</string>
     <string name="add_to_the_top">Add to the top</string>
     <string name="use_for_this_checklist">Use for this checklist only</string>
     <plurals name="num_checked_items">