Merge pull request #265 from tom93/pr/import-fix-timestamps
Fix conversation dates on import
This commit is contained in:
commit
32b20698ff
3 changed files with 26 additions and 3 deletions
|
|
@ -1117,7 +1117,11 @@ fun Context.deleteSmsDraft(threadId: Long) {
|
|||
}
|
||||
|
||||
fun Context.updateLastConversationMessage(threadId: Long) {
|
||||
// update the date and the snippet of the thread, by triggering the
|
||||
updateLastConversationMessage(setOf(threadId))
|
||||
}
|
||||
|
||||
fun Context.updateLastConversationMessage(threadIds: Iterable<Long>) {
|
||||
// update the date and the snippet of the threads, by triggering the
|
||||
// following Android code (which runs even if no messages are deleted):
|
||||
// https://android.googlesource.com/platform/packages/providers/TelephonyProvider/+/android14-release/src/com/android/providers/telephony/MmsSmsProvider.java#1409
|
||||
val uri = Threads.CONTENT_URI
|
||||
|
|
@ -1125,8 +1129,10 @@ fun Context.updateLastConversationMessage(threadId: Long) {
|
|||
"1 = 0" // always-false condition, because we don't actually want to delete any messages
|
||||
try {
|
||||
contentResolver.delete(uri, selection, null)
|
||||
for (threadId in threadIds) {
|
||||
val newConversation = getConversations(threadId)[0]
|
||||
insertOrUpdateConversation(newConversation)
|
||||
}
|
||||
} catch (e: Exception) {
|
||||
}
|
||||
}
|
||||
|
|
|
|||
|
|
@ -96,6 +96,7 @@ class MessagesImporter(private val activity: SimpleActivity) {
|
|||
messagesFailed++
|
||||
}
|
||||
}
|
||||
messageWriter.fixCoversationDates()
|
||||
refreshMessages()
|
||||
} catch (e: Exception) {
|
||||
activity.showErrorToast(e)
|
||||
|
|
|
|||
|
|
@ -11,6 +11,7 @@ import com.klinker.android.send_message.Utils
|
|||
import org.fossify.commons.extensions.getLongValue
|
||||
import org.fossify.commons.extensions.queryCursor
|
||||
import org.fossify.commons.helpers.isRPlus
|
||||
import org.fossify.messages.extensions.updateLastConversationMessage
|
||||
import org.fossify.messages.models.MmsAddress
|
||||
import org.fossify.messages.models.MmsBackup
|
||||
import org.fossify.messages.models.MmsPart
|
||||
|
|
@ -19,12 +20,14 @@ import org.fossify.messages.models.SmsBackup
|
|||
class MessagesWriter(private val context: Context) {
|
||||
private val INVALID_ID = -1L
|
||||
private val contentResolver = context.contentResolver
|
||||
private val modifiedThreadIds = mutableSetOf<Long>()
|
||||
|
||||
fun writeSmsMessage(smsBackup: SmsBackup) {
|
||||
val contentValues = smsBackup.toContentValues()
|
||||
val threadId = Utils.getOrCreateThreadId(context, smsBackup.address)
|
||||
contentValues.put(Sms.THREAD_ID, threadId)
|
||||
if (!smsExist(smsBackup)) {
|
||||
modifiedThreadIds.add(threadId)
|
||||
contentResolver.insert(Sms.CONTENT_URI, contentValues)
|
||||
}
|
||||
}
|
||||
|
|
@ -50,6 +53,7 @@ class MessagesWriter(private val context: Context) {
|
|||
if (threadId != INVALID_ID) {
|
||||
contentValues.put(Mms.THREAD_ID, threadId)
|
||||
if (!mmsExist(mmsBackup)) {
|
||||
modifiedThreadIds.add(threadId)
|
||||
contentResolver.insert(Mms.CONTENT_URI, contentValues)
|
||||
}
|
||||
val messageId = getMmsId(mmsBackup)
|
||||
|
|
@ -152,4 +156,16 @@ class MessagesWriter(private val context: Context) {
|
|||
}
|
||||
return exists
|
||||
}
|
||||
|
||||
/** Fixes the timestamps of all conversations modified by previous writes. */
|
||||
fun fixCoversationDates() {
|
||||
// This method should be called after messages are written, to set the correct conversation
|
||||
// timestamps.
|
||||
//
|
||||
// It is necessary because when we insert a message, Android's Telephony provider sets the
|
||||
// conversation date to the current time rather than the time of the message that was
|
||||
// inserted. Source code reference:
|
||||
// https://android.googlesource.com/platform/packages/providers/TelephonyProvider/+/android14-release/src/com/android/providers/telephony/MmsSmsDatabaseHelper.java#134
|
||||
context.updateLastConversationMessage(modifiedThreadIds)
|
||||
}
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue