From 7b84ec77fdcd3b286bfe707f19e35d4ccee24faf Mon Sep 17 00:00:00 2001 From: Agnieszka C <85929121+Aga-C@users.noreply.github.com> Date: Fri, 3 Sep 2021 19:00:23 +0200 Subject: [PATCH 1/3] Added SMS drafts support (#83) --- .../smsmessenger/activities/ThreadActivity.kt | 14 +++++ .../adapters/ConversationsAdapter.kt | 9 +++- .../smsmessenger/extensions/Context.kt | 52 +++++++++++++++++++ app/src/main/res/layout/item_conversation.xml | 16 +++++- 4 files changed, 89 insertions(+), 2 deletions(-) diff --git a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/activities/ThreadActivity.kt b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/activities/ThreadActivity.kt index f235f327..5b7af23a 100644 --- a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/activities/ThreadActivity.kt +++ b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/activities/ThreadActivity.kt @@ -108,11 +108,25 @@ class ThreadActivity : SimpleActivity() { override fun onResume() { super.onResume() + + val smsDraft = getSmsDraft(threadId) + if (smsDraft != null) { + thread_type_message.setText(smsDraft) + } isActivityVisible = true } override fun onPause() { super.onPause() + + if (thread_type_message.value != "" && attachmentUris.isEmpty()) { + saveSmsDraft(thread_type_message.value, threadId) + } else { + deleteSmsDraft(threadId) + } + + bus?.post(Events.RefreshMessages()) + isActivityVisible = false } diff --git a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/adapters/ConversationsAdapter.kt b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/adapters/ConversationsAdapter.kt index c895ea85..40ad72d8 100644 --- a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/adapters/ConversationsAdapter.kt +++ b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/adapters/ConversationsAdapter.kt @@ -23,6 +23,7 @@ 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.getSmsDraft import com.simplemobiletools.smsmessenger.helpers.refreshMessages import com.simplemobiletools.smsmessenger.models.Conversation import kotlinx.android.synthetic.main.item_conversation.view.* @@ -220,6 +221,8 @@ class ConversationsAdapter( private fun setupView(view: View, conversation: Conversation) { view.apply { + val smsDraft = context.getSmsDraft(conversation.threadId) + conversation_frame.isSelected = selectedKeys.contains(conversation.hashCode()) conversation_address.apply { @@ -228,10 +231,14 @@ class ConversationsAdapter( } conversation_body_short.apply { - text = conversation.snippet + text = smsDraft ?: conversation.snippet setTextSize(TypedValue.COMPLEX_UNIT_PX, fontSize * 0.9f) } + draft_indicator.apply { + visibility = if (smsDraft != null) View.VISIBLE else View.GONE + } + conversation_date.apply { text = conversation.date.formatDateOrTime(context, true, false) setTextSize(TypedValue.COMPLEX_UNIT_PX, fontSize * 0.8f) diff --git a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/extensions/Context.kt b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/extensions/Context.kt index cb55e7c8..9d00d0cd 100644 --- a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/extensions/Context.kt +++ b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/extensions/Context.kt @@ -764,3 +764,55 @@ fun Context.getLockScreenVisibilityText(type: Int) = getString( else -> R.string.nothing } ) + +fun Context.getSmsDraft(threadId: Long): String? { + val uri = Sms.Draft.CONTENT_URI + val projection = arrayOf(Sms.BODY) + val selection = "${Sms.THREAD_ID} = ?" + val selectionArgs = arrayOf(threadId.toString()) + + try { + val cursor = contentResolver.query(uri, projection, selection, selectionArgs, null) + cursor.use { + if (cursor?.moveToFirst() == true) { + return cursor.getString(0) + } + } + } catch (e: Exception) { + } + + return null +} + +fun Context.saveSmsDraft(body: String, threadId: Long) { + val uri = Sms.Draft.CONTENT_URI + val contentValues = ContentValues().apply { + put(Sms.BODY, body) + put(Sms.DATE, System.currentTimeMillis().toString()) + put(Sms.TYPE, Sms.MESSAGE_TYPE_DRAFT) + put(Sms.THREAD_ID, threadId) + } + + try { + contentResolver.insert(uri, contentValues) + } catch (e: Exception) { + } +} + +fun Context.deleteSmsDraft(threadId: Long) { + val lookupUri = Sms.Draft.CONTENT_URI + val lookupProjection = arrayOf(Sms._ID) + val lookupSelection = "${Sms.THREAD_ID} = ?" + val lookupSelectionArgs = arrayOf(threadId.toString()) + try { + val cursor = contentResolver.query(lookupUri, lookupProjection, lookupSelection, lookupSelectionArgs, null) + cursor.use { + if (cursor?.moveToFirst() == true) { + val draftId = cursor.getLong(0) + val uri = Uri.withAppendedPath(Sms.CONTENT_URI, "/${draftId}") + contentResolver.delete(uri, null, null) + } + } + } catch (e: Exception) { + } +} diff --git a/app/src/main/res/layout/item_conversation.xml b/app/src/main/res/layout/item_conversation.xml index 5fce6297..6cde1caa 100644 --- a/app/src/main/res/layout/item_conversation.xml +++ b/app/src/main/res/layout/item_conversation.xml @@ -37,12 +37,26 @@ android:textSize="@dimen/big_text_size" tools:text="John" /> + + Date: Wed, 8 Sep 2021 20:00:28 +0200 Subject: [PATCH 2/3] fixing a compile error + some renaming --- .../smsmessenger/extensions/Context.kt | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/extensions/Context.kt b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/extensions/Context.kt index 6415e79d..fbe15066 100644 --- a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/extensions/Context.kt +++ b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/extensions/Context.kt @@ -35,9 +35,9 @@ import com.simplemobiletools.smsmessenger.interfaces.MessagesDao import com.simplemobiletools.smsmessenger.models.* import com.simplemobiletools.smsmessenger.receivers.DirectReplyReceiver import com.simplemobiletools.smsmessenger.receivers.MarkAsReadReceiver +import me.leolin.shortcutbadger.ShortcutBadger import java.util.* import kotlin.collections.ArrayList -import me.leolin.shortcutbadger.ShortcutBadger val Context.config: Config get() = Config.newInstance(applicationContext) @@ -800,19 +800,20 @@ fun Context.saveSmsDraft(body: String, threadId: Long) { } fun Context.deleteSmsDraft(threadId: Long) { - val lookupUri = Sms.Draft.CONTENT_URI - val lookupProjection = arrayOf(Sms._ID) - val lookupSelection = "${Sms.THREAD_ID} = ?" - val lookupSelectionArgs = arrayOf(threadId.toString()) + val uri = Sms.Draft.CONTENT_URI + val projection = arrayOf(Sms._ID) + val selection = "${Sms.THREAD_ID} = ?" + val selectionArgs = arrayOf(threadId.toString()) try { - val cursor = contentResolver.query(lookupUri, lookupProjection, lookupSelection, lookupSelectionArgs, null) + val cursor = contentResolver.query(uri, projection, selection, selectionArgs, null) cursor.use { if (cursor?.moveToFirst() == true) { val draftId = cursor.getLong(0) - val uri = Uri.withAppendedPath(Sms.CONTENT_URI, "/${draftId}") - contentResolver.delete(uri, null, null) + val draftUri = Uri.withAppendedPath(Sms.CONTENT_URI, "/${draftId}") + contentResolver.delete(draftUri, null, null) } } + } catch (e: Exception) { } } From aa71a39f1bf928eb002e890a8e45d8f6eee33bec Mon Sep 17 00:00:00 2001 From: Tibor Kaputa Date: Wed, 8 Sep 2021 20:03:35 +0200 Subject: [PATCH 3/3] minor code cleanup + color customization fix --- .../smsmessenger/adapters/ConversationsAdapter.kt | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/adapters/ConversationsAdapter.kt b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/adapters/ConversationsAdapter.kt index 13994142..2990622a 100644 --- a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/adapters/ConversationsAdapter.kt +++ b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/adapters/ConversationsAdapter.kt @@ -257,6 +257,8 @@ class ConversationsAdapter( private fun setupView(view: View, conversation: Conversation) { view.apply { val smsDraft = context.getSmsDraft(conversation.threadId) + draft_indicator.beVisibleIf(smsDraft != null) + draft_indicator.setTextColor(adjustedPrimaryColor) conversation_frame.isSelected = selectedKeys.contains(conversation.hashCode()) @@ -270,10 +272,6 @@ class ConversationsAdapter( setTextSize(TypedValue.COMPLEX_UNIT_PX, fontSize * 0.9f) } - draft_indicator.apply { - visibility = if (smsDraft != null) View.VISIBLE else View.GONE - } - conversation_date.apply { text = conversation.date.formatDateOrTime(context, true, false) setTextSize(TypedValue.COMPLEX_UNIT_PX, fontSize * 0.8f)