better method: limit the total of sms+mms to MESSAGES_LIMIT

this causes the sms and mms to load together for a smoother flow

Author: yparitcher

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

@@ -514,7 +514,7 @@ class ThreadActivity : SimpleActivity() {
             return
         }
 
-        val firstItem = messages.first{ !it.isMMS }
+        val firstItem = messages.first()
         val dateOfFirstItem = firstItem.date
         if (oldestMessageDate == dateOfFirstItem) {
             allMessagesFetched = true
@@ -529,7 +529,7 @@ class ThreadActivity : SimpleActivity() {
                 .filter { message -> !messages.contains(message) }
 
             messages.addAll(0, olderMessages)
-            allMessagesFetched = olderMessages.size < MESSAGES_LIMIT || olderMessages.isEmpty()
+            allMessagesFetched = olderMessages.isEmpty()
             threadItems = getThreadItems()
 
             runOnUiThread {

app/src/main/kotlin/com/simplemobiletools/smsmessenger/extensions/Context.kt

@@ -135,6 +135,7 @@ fun Context.getMessages(
         .filter { it.participants.isNotEmpty() }
         .filterNot { it.isScheduled && it.millis() < System.currentTimeMillis() }
         .sortedWith(compareBy<Message> { it.date }.thenBy { it.id })
+        .takeLast(limit)
         .toMutableList() as ArrayList<Message>
 
     return messages