From e1f1583b17baa223cf160233f2ec283bee541b8d Mon Sep 17 00:00:00 2001
From: tibbi <tibor@kaputa.sk>
Date: Sun, 5 Apr 2020 19:12:02 +0200
Subject: [PATCH] add handling for storing sent messages

---
 app/src/main/AndroidManifest.xml              |  2 ++
 .../smsmessenger/activities/ThreadActivity.kt | 17 +++++++---
 .../smsmessenger/extensions/Context.kt        |  6 ++--
 .../smsmessenger/helpers/Constants.kt         |  4 +++
 .../smsmessenger/receivers/SmsReceiver.kt     |  4 ++-
 .../smsmessenger/receivers/SmsSentReceiver.kt | 34 +++++++++++++++++++
 6 files changed, 59 insertions(+), 8 deletions(-)
 create mode 100644 app/src/main/kotlin/com/simplemobiletools/smsmessenger/receivers/SmsSentReceiver.kt

diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index ba90d2f9..c1e6cfa8 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -111,6 +111,8 @@
             </intent-filter>
         </receiver>
 
+        <receiver android:name=".receivers.SmsSentReceiver"/>
+
         <activity-alias
             android:name=".activities.SplashActivity.Red"
             android:enabled="false"
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 48f8d925..06e5d301 100644
--- a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/activities/ThreadActivity.kt
+++ b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/activities/ThreadActivity.kt
@@ -1,5 +1,7 @@
 package com.simplemobiletools.smsmessenger.activities
 
+import android.app.PendingIntent
+import android.content.Intent
 import android.os.Bundle
 import android.provider.Telephony
 import android.telephony.SmsManager
@@ -12,13 +14,12 @@ import com.simplemobiletools.smsmessenger.adapters.ThreadAdapter
 import com.simplemobiletools.smsmessenger.extensions.config
 import com.simplemobiletools.smsmessenger.extensions.getMessages
 import com.simplemobiletools.smsmessenger.extensions.markSMSRead
-import com.simplemobiletools.smsmessenger.helpers.THREAD_ID
-import com.simplemobiletools.smsmessenger.helpers.THREAD_NAME
-import com.simplemobiletools.smsmessenger.helpers.THREAD_NUMBER
+import com.simplemobiletools.smsmessenger.helpers.*
 import com.simplemobiletools.smsmessenger.models.Events
 import com.simplemobiletools.smsmessenger.models.ThreadDateTime
 import com.simplemobiletools.smsmessenger.models.ThreadError
 import com.simplemobiletools.smsmessenger.models.ThreadItem
+import com.simplemobiletools.smsmessenger.receivers.SmsSentReceiver
 import kotlinx.android.synthetic.main.activity_thread.*
 import org.greenrobot.eventbus.EventBus
 import org.greenrobot.eventbus.Subscribe
@@ -60,6 +61,7 @@ class ThreadActivity : SimpleActivity() {
     }
 
     private fun setupButtons() {
+        val threadId = intent.getIntExtra(THREAD_ID, 0)
         thread_type_message.setColors(config.textColor, config.primaryColor, config.backgroundColor)
         thread_send_message.applyColorFilter(config.textColor)
 
@@ -69,8 +71,15 @@ class ThreadActivity : SimpleActivity() {
                 return@setOnClickListener
             }
 
+            val intent = Intent(this, SmsSentReceiver::class.java).apply {
+                putExtra(MESSAGE_BODY, msg)
+                putExtra(MESSAGE_ADDRESS, targetNumber)
+            }
+
+            val pendingIntent = PendingIntent.getBroadcast(this, threadId, intent, PendingIntent.FLAG_UPDATE_CURRENT)
             val smsManager = SmsManager.getDefault()
-            smsManager.sendTextMessage(targetNumber, null, msg, null, null)
+            smsManager.sendTextMessage(targetNumber, null, msg, pendingIntent, null)
+            thread_type_message.setText("")
         }
 
         thread_send_message.isClickable = false
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 4df480e1..fee19de3 100644
--- a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/extensions/Context.kt
+++ b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/extensions/Context.kt
@@ -162,16 +162,16 @@ fun Context.getNameFromPhoneNumber(number: String): Int? {
     return null
 }
 
-fun Context.insertNewSMS(address: String, subject: String, body: String, date: Long, threadId: Long) {
+fun Context.insertNewSMS(address: String, subject: String, body: String, date: Long, read: Int, threadId: Long, type: Int) {
     val uri = Telephony.Sms.CONTENT_URI
     val contentValues = ContentValues().apply {
         put(Telephony.Sms.ADDRESS, address)
         put(Telephony.Sms.SUBJECT, subject)
         put(Telephony.Sms.BODY, body)
         put(Telephony.Sms.DATE, date)
-        put(Telephony.Sms.READ, 0)
+        put(Telephony.Sms.READ, read)
         put(Telephony.Sms.THREAD_ID, threadId)
-        put(Telephony.Sms.TYPE, Telephony.Sms.MESSAGE_TYPE_INBOX)
+        put(Telephony.Sms.TYPE, type)
     }
 
     contentResolver.insert(uri, contentValues)
diff --git a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/helpers/Constants.kt b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/helpers/Constants.kt
index 9160b077..e8f6b4d0 100644
--- a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/helpers/Constants.kt
+++ b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/helpers/Constants.kt
@@ -13,6 +13,10 @@ const val THREAD_RECEIVED_MESSAGE = 2
 const val THREAD_SENT_MESSAGE = 3
 const val THREAD_SENT_MESSAGE_ERROR = 4
 
+// constants used at passing info to SmsSentReceiver
+const val MESSAGE_BODY = "message_body"
+const val MESSAGE_ADDRESS = "message_address"
+
 fun refreshMessages() {
     EventBus.getDefault().post(Events.RefreshMessages())
 }
diff --git a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/receivers/SmsReceiver.kt b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/receivers/SmsReceiver.kt
index 1c5d26a8..283791f7 100644
--- a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/receivers/SmsReceiver.kt
+++ b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/receivers/SmsReceiver.kt
@@ -38,7 +38,9 @@ class SmsReceiver : BroadcastReceiver() {
             val body = it.messageBody
             val date = it.timestampMillis
             val threadId = context.getThreadId(address)
-            context.insertNewSMS(address, subject, body, date, threadId)
+            val type = Telephony.Sms.MESSAGE_TYPE_INBOX
+            val read = 0
+            context.insertNewSMS(address, subject, body, date, read, threadId, type)
             showNotification(context, address, body, threadId.toInt())
         }
 
diff --git a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/receivers/SmsSentReceiver.kt b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/receivers/SmsSentReceiver.kt
new file mode 100644
index 00000000..945f5ce6
--- /dev/null
+++ b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/receivers/SmsSentReceiver.kt
@@ -0,0 +1,34 @@
+package com.simplemobiletools.smsmessenger.receivers
+
+import android.app.Activity
+import android.content.BroadcastReceiver
+import android.content.Context
+import android.content.Intent
+import android.provider.Telephony
+import com.simplemobiletools.smsmessenger.extensions.getThreadId
+import com.simplemobiletools.smsmessenger.extensions.insertNewSMS
+import com.simplemobiletools.smsmessenger.helpers.MESSAGE_ADDRESS
+import com.simplemobiletools.smsmessenger.helpers.MESSAGE_BODY
+import com.simplemobiletools.smsmessenger.helpers.refreshMessages
+
+class SmsSentReceiver : BroadcastReceiver() {
+    override fun onReceive(context: Context, intent: Intent) {
+        val extras = intent.extras
+        if (extras != null) {
+            val address = extras.getString(MESSAGE_ADDRESS) ?: ""
+            val subject = ""
+            val body = extras.getString(MESSAGE_BODY) ?: ""
+            val date = System.currentTimeMillis()
+            val threadId = context.getThreadId(address)
+            val type = if (resultCode == Activity.RESULT_OK) {
+                Telephony.Sms.MESSAGE_TYPE_SENT
+            } else {
+                Telephony.Sms.MESSAGE_TYPE_FAILED
+            }
+            val read = 1
+            context.insertNewSMS(address, subject, body, date, read, threadId, type)
+        }
+
+        refreshMessages()
+    }
+}