Added pinning conversations (#197)

app/src/main/kotlin/com/simplemobiletools/smsmessenger/activities/MainActivity.kt

@@ -28,6 +28,7 @@ import org.greenrobot.eventbus.EventBus
 import org.greenrobot.eventbus.Subscribe
 import org.greenrobot.eventbus.ThreadMode
 import java.util.*
+import kotlin.collections.ArrayList
 
 class MainActivity : SimpleActivity() {
     private val MAKE_DEFAULT_APP_REQUEST = 1
@@ -170,7 +171,7 @@ class MainActivity : SimpleActivity() {
     private fun getCachedConversations() {
         ensureBackgroundThread {
             val conversations = try {
-                conversationsDB.getAll().sortedByDescending { it.date }.toMutableList() as ArrayList<Conversation>
+                conversationsDB.getAll().toMutableList() as ArrayList<Conversation>
             } catch (e: Exception) {
                 ArrayList()
             }
@@ -226,6 +227,10 @@ class MainActivity : SimpleActivity() {
 
     private fun setupConversations(conversations: ArrayList<Conversation>) {
         val hasConversations = conversations.isNotEmpty()
+        val sortedConversations = conversations.sortedWith(
+            compareByDescending<Conversation>{ config.pinnedConversations.contains(it.threadId.toString()) }
+                .thenByDescending{ it.date }
+        ).toMutableList() as ArrayList<Conversation>
         conversations_list.beVisibleIf(hasConversations)
         no_conversations_placeholder.beVisibleIf(!hasConversations)
         no_conversations_placeholder_2.beVisibleIf(!hasConversations)
@@ -237,7 +242,7 @@ class MainActivity : SimpleActivity() {
 
         val currAdapter = conversations_list.adapter
         if (currAdapter == null) {
-            ConversationsAdapter(this, conversations, conversations_list, conversations_fastscroller) {
+            ConversationsAdapter(this, sortedConversations, conversations_list, conversations_fastscroller) {
                 Intent(this, ThreadActivity::class.java).apply {
                     putExtra(THREAD_ID, (it as Conversation).threadId)
                     putExtra(THREAD_TITLE, it.title)
@@ -254,7 +259,7 @@ class MainActivity : SimpleActivity() {
             }
         } else {
             try {
-                (currAdapter as ConversationsAdapter).updateConversations(conversations)
+                (currAdapter as ConversationsAdapter).updateConversations(sortedConversations)
             } catch (ignored: Exception) {
             }
         }

app/src/main/kotlin/com/simplemobiletools/smsmessenger/activities/ThreadActivity.kt

@@ -146,6 +146,7 @@ class ThreadActivity : SimpleActivity() {
         }
 
         updateMenuItemColors(menu)
+        checkPinBtnVisibility(menu)
         return true
     }
 
@@ -159,6 +160,8 @@ class ThreadActivity : SimpleActivity() {
             R.id.delete -> askConfirmDelete()
             R.id.manage_people -> managePeople()
             R.id.mark_as_unread -> markAsUnread()
+            R.id.pin_conversation -> pinConversation(true)
+            R.id.unpin_conversation -> pinConversation(false)
             else -> return super.onOptionsItemSelected(item)
         }
         return true
@@ -852,6 +855,25 @@ class ThreadActivity : SimpleActivity() {
         return participants
     }
 
+    private fun pinConversation(pin: Boolean) {
+        if (pin) {
+            config.addPinnedConversationByThreadId(threadId)
+        }
+        else {
+            config.removePinnedConversationByThreadId(threadId)
+        }
+
+        runOnUiThread {
+            refreshMessages()
+        }
+    }
+
+    private fun checkPinBtnVisibility(menu: Menu) {
+        val pinnedConversations = config.pinnedConversations
+        menu.findItem(R.id.pin_conversation).isVisible = !pinnedConversations.contains(threadId.toString())
+        menu.findItem(R.id.unpin_conversation).isVisible = pinnedConversations.contains(threadId.toString())
+    }
+
     @SuppressLint("MissingPermission")
     @Subscribe(threadMode = ThreadMode.ASYNC)
     fun refreshMessages(event: Events.RefreshMessages) {

app/src/main/kotlin/com/simplemobiletools/smsmessenger/adapters/ConversationsAdapter.kt

@@ -22,10 +22,7 @@ import com.simplemobiletools.commons.views.FastScroller
 import com.simplemobiletools.commons.views.MyRecyclerView
 import com.simplemobiletools.smsmessenger.R
 import com.simplemobiletools.smsmessenger.activities.SimpleActivity
-import com.simplemobiletools.smsmessenger.extensions.deleteConversation
+import com.simplemobiletools.smsmessenger.extensions.*
-import com.simplemobiletools.smsmessenger.extensions.getSmsDraft
-import com.simplemobiletools.smsmessenger.extensions.markThreadMessagesRead
-import com.simplemobiletools.smsmessenger.extensions.markThreadMessagesUnread
 import com.simplemobiletools.smsmessenger.helpers.refreshMessages
 import com.simplemobiletools.smsmessenger.models.Conversation
 import kotlinx.android.synthetic.main.item_conversation.view.*
@@ -48,6 +45,8 @@ class ConversationsAdapter(
             findItem(R.id.cab_add_number_to_contact).isVisible = isOneItemSelected() && getSelectedItems().firstOrNull()?.isGroupConversation == false
             findItem(R.id.cab_dial_number).isVisible = isOneItemSelected() && getSelectedItems().firstOrNull()?.isGroupConversation == false
             findItem(R.id.cab_copy_number).isVisible = isOneItemSelected() && getSelectedItems().firstOrNull()?.isGroupConversation == false
+
+            checkPinBtnVisibility(this)
         }
     }
 
@@ -64,6 +63,8 @@ class ConversationsAdapter(
             R.id.cab_delete -> askConfirmDelete()
             R.id.cab_mark_as_read -> markAsRead()
             R.id.cab_mark_as_unread -> markAsUnread()
+            R.id.cab_pin_conversation -> pinConversation(true)
+            R.id.cab_unpin_conversation -> pinConversation(false)
             R.id.cab_select_all -> selectAll()
         }
     }
@@ -237,6 +238,33 @@ class ConversationsAdapter(
 
     private fun getSelectedItems() = conversations.filter { selectedKeys.contains(it.hashCode()) } as ArrayList<Conversation>
 
+    private fun pinConversation(pin: Boolean) {
+        val conversations = getSelectedItems()
+
+        if (conversations.size == 0) {
+            return
+        }
+
+        if (pin) {
+            activity.config.addPinnedConversations(conversations)
+        }
+        else {
+            activity.config.removePinnedConversations(conversations)
+        }
+
+        activity.runOnUiThread {
+            refreshMessages()
+            finishActMode()
+        }
+    }
+
+    private fun checkPinBtnVisibility(menu: Menu) {
+        val pinnedConversations = activity.config.pinnedConversations
+        val selectedConversations = getSelectedItems()
+        menu.findItem(R.id.cab_pin_conversation).isVisible = selectedConversations.any { !pinnedConversations.contains(it.threadId.toString()) }
+        menu.findItem(R.id.cab_unpin_conversation).isVisible = selectedConversations.any { pinnedConversations.contains(it.threadId.toString()) }
+    }
+
     override fun onViewRecycled(holder: ViewHolder) {
         super.onViewRecycled(holder)
         if (!activity.isDestroyed && !activity.isFinishing) {
@@ -265,6 +293,8 @@ class ConversationsAdapter(
             draft_indicator.beVisibleIf(smsDraft != null)
             draft_indicator.setTextColor(adjustedPrimaryColor)
 
+            pin_indicator.beVisibleIf(activity.config.pinnedConversations.contains(conversation.threadId.toString()))
+
             conversation_frame.isSelected = selectedKeys.contains(conversation.hashCode())
 
             conversation_address.apply {

app/src/main/kotlin/com/simplemobiletools/smsmessenger/helpers/Config.kt

@@ -2,6 +2,8 @@ package com.simplemobiletools.smsmessenger.helpers
 
 import android.content.Context
 import com.simplemobiletools.commons.helpers.BaseConfig
+import com.simplemobiletools.smsmessenger.models.Conversation
+import java.util.HashSet
 
 class Config(context: Context) : BaseConfig(context) {
     companion object {
@@ -33,4 +35,24 @@ class Config(context: Context) : BaseConfig(context) {
     var mmsFileSizeLimit: Long
         get() = prefs.getLong(MMS_FILE_SIZE_LIMIT, FILE_SIZE_1_MB)
         set(mmsFileSizeLimit) = prefs.edit().putLong(MMS_FILE_SIZE_LIMIT, mmsFileSizeLimit).apply()
+
+    var pinnedConversations: Set<String>
+        get() = prefs.getStringSet(PINNED_CONVERSATIONS, HashSet<String>())!!
+        set(pinnedConversations) = prefs.edit().putStringSet(PINNED_CONVERSATIONS, pinnedConversations).apply()
+
+    fun addPinnedConversationByThreadId(threadId: Long) {
+        pinnedConversations = pinnedConversations.plus(threadId.toString())
+    }
+
+    fun addPinnedConversations(conversations: List<Conversation>) {
+        pinnedConversations = pinnedConversations.plus(conversations.map { it.threadId.toString() })
+    }
+
+    fun removePinnedConversationByThreadId(threadId: Long) {
+        pinnedConversations = pinnedConversations.minus(threadId.toString())
+    }
+
+    fun removePinnedConversations(conversations: List<Conversation>) {
+        pinnedConversations = pinnedConversations.minus(conversations.map { it.threadId.toString() })
+    }
 }

app/src/main/kotlin/com/simplemobiletools/smsmessenger/helpers/Constants.kt

@@ -17,6 +17,7 @@ const val USE_SIMPLE_CHARACTERS = "use_simple_characters"
 const val LOCK_SCREEN_VISIBILITY = "lock_screen_visibility"
 const val ENABLE_DELIVERY_REPORTS = "enable_delivery_reports"
 const val MMS_FILE_SIZE_LIMIT = "mms_file_size_limit"
+const val PINNED_CONVERSATIONS = "pinned_conversations"
 
 private const val PATH = "com.simplemobiletools.smsmessenger.action."
 const val MARK_AS_READ = PATH + "mark_as_read"

app/src/main/res/layout/item_conversation.xml

@@ -26,6 +26,24 @@
             android:layout_marginStart="@dimen/normal_margin"
             android:layout_marginEnd="@dimen/normal_margin" />
 
+        <RelativeLayout
+            android:id="@+id/pin_indicator_l"
+            android:layout_width="@dimen/normal_icon_size"
+            android:layout_height="@dimen/normal_icon_size"
+            android:layout_alignTop="@+id/conversation_address"
+            android:layout_alignBottom="@+id/conversation_date"
+            android:layout_centerVertical="true"
+            android:layout_marginStart="@dimen/normal_margin"
+            android:layout_marginEnd="@dimen/normal_margin">
+
+            <ImageView
+                android:id="@+id/pin_indicator"
+                android:layout_width="@dimen/pin_icon_size"
+                android:layout_height="@dimen/pin_icon_size"
+                android:layout_alignParentRight="true"
+                android:src="@drawable/ic_pin" />
+        </RelativeLayout>
+
         <TextView
             android:id="@+id/conversation_address"
             android:layout_width="match_parent"

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

@@ -33,6 +33,14 @@
         android:id="@+id/cab_mark_as_unread"
         android:title="@string/mark_as_unread"
         app:showAsAction="never" />
+    <item
+        android:id="@+id/cab_pin_conversation"
+        android:title="@string/pin_conversation"
+        app:showAsAction="never" />
+    <item
+        android:id="@+id/cab_unpin_conversation"
+        android:title="@string/unpin_conversation"
+        app:showAsAction="never" />
     <item
         android:id="@+id/cab_select_all"
         android:title="@string/select_all"

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

@@ -19,4 +19,12 @@
         android:id="@+id/mark_as_unread"
         android:title="@string/mark_as_unread"
         app:showAsAction="never" />
+    <item
+        android:id="@+id/pin_conversation"
+        android:title="@string/pin_conversation"
+        app:showAsAction="never" />
+    <item
+        android:id="@+id/unpin_conversation"
+        android:title="@string/unpin_conversation"
+        app:showAsAction="never" />
 </menu>

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

@@ -17,6 +17,8 @@
     <string name="sending">Odesílá se…</string>
     <string name="export_messages">Export zpráv</string>
     <string name="import_messages">Import zpráv</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Nová konverzace</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Sender…</string>
     <string name="export_messages">Eksporter beskeder</string>
     <string name="import_messages">Importer beskeder</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Ny Samtale</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Sende…</string>
     <string name="export_messages">Export messages</string>
     <string name="import_messages">Import messages</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Neuer Chat</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Γίνεται αποστολή…</string>
     <string name="export_messages">Εξαγωγή μηνυμάτων</string>
     <string name="import_messages">Εισαγωγή μηνυμάτων</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Νέα συνομιλία</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Sending…</string>
     <string name="export_messages">Export messages</string>
     <string name="import_messages">Import messages</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Nueva conversación</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Lähetetään…</string>
     <string name="export_messages">Export messages</string>
     <string name="import_messages">Import messages</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Uusi keskustelu</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Envoi en cours…</string>
     <string name="export_messages">Export de messages</string>
     <string name="import_messages">Import de messages</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Nouvelle conversation</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Sending…</string>
     <string name="export_messages">Export messages</string>
     <string name="import_messages">Import messages</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Nova conversa</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Sending…</string>
     <string name="export_messages">Export messages</string>
     <string name="import_messages">Import messages</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Percakapan baru</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Invio…</string>
     <string name="export_messages">Export messages</string>
     <string name="import_messages">Import messages</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Nuova conversazione</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Sending…</string>
     <string name="export_messages">Export messages</string>
     <string name="import_messages">Import messages</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">新しい会話</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Sending…</string>
     <string name="export_messages">Export messages</string>
     <string name="import_messages">Import messages</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Naujas pokalbis</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Sending…</string>
     <string name="export_messages">Export messages</string>
     <string name="import_messages">Import messages</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">പുതിയ സംഭാഷണം</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Versturen…</string>
     <string name="export_messages">Berichten exporteren</string>
     <string name="import_messages">Berichten importeren</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Nieuw gesprek</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Wysyłanie…</string>
     <string name="export_messages">Eksportuj wiadomości</string>
     <string name="import_messages">Importuj wiadomości</string>
+    <string name="pin_conversation">Przypnij na górze</string>
+    <string name="unpin_conversation">Odepnij</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Nowa rozmowa</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">A enviar…</string>
     <string name="export_messages">Exportar mensagens</string>
     <string name="import_messages">Importar mensagens</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Nova conversa</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Отправка…</string>
     <string name="export_messages">Экспорт сообщений</string>
     <string name="import_messages">Импорт сообщений</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Новая переписка</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Odosiela sa…</string>
     <string name="export_messages">Exportovať správy</string>
     <string name="import_messages">Importovať správy</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Nová konverzácia</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Sending…</string>
     <string name="export_messages">Export messages</string>
     <string name="import_messages">Import messages</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Yeni görüşme</string>

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

@@ -17,6 +17,8 @@
     <string name="sending">Sending…</string>
     <string name="export_messages">Export messages</string>
     <string name="import_messages">Import messages</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">Нове листування</string>

app/src/main/res/values-zh-rCN/strings.xml

@@ -17,6 +17,8 @@
     <string name="sending">Sending…</string>
     <string name="export_messages">Export messages</string>
     <string name="import_messages">Import messages</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">新的对话</string>

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

@@ -5,4 +5,5 @@
     <dimen name="play_outline_size">36dp</dimen>
     <dimen name="attachment_preview_size">60dp</dimen>
     <dimen name="remove_attachment_size">24dp</dimen>
+    <dimen name="pin_icon_size">15dp</dimen>
 </resources>

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

@@ -17,6 +17,8 @@
     <string name="sending">Sending…</string>
     <string name="export_messages">Export messages</string>
     <string name="import_messages">Import messages</string>
+    <string name="pin_conversation">Pin to the top</string>
+    <string name="unpin_conversation">Unpin</string>
 
     <!-- New conversation -->
     <string name="new_conversation">New conversation</string>