make part and mms insertion a single transaction

fixes #3051
closes #3059
// FREEBIE
This commit is contained in:
Jake McGinty 2015-04-28 00:16:37 -05:00
parent d0008b57b5
commit 26c4d746d2

View File

@ -749,15 +749,21 @@ public class MmsDatabase extends MessagingDatabase {
contentValues.put(PART_COUNT, PartParser.getSupportedMediaPartCount(body)); contentValues.put(PART_COUNT, PartParser.getSupportedMediaPartCount(body));
long messageId = db.insert(TABLE_NAME, null, contentValues); db.beginTransaction();
try {
long messageId = db.insert(TABLE_NAME, null, contentValues);
addressDatabase.insertAddressesForId(messageId, headers); addressDatabase.insertAddressesForId(messageId, headers);
partsDatabase.insertParts(masterSecret, messageId, body); partsDatabase.insertParts(masterSecret, messageId, body);
notifyConversationListeners(contentValues.getAsLong(THREAD_ID)); notifyConversationListeners(contentValues.getAsLong(THREAD_ID));
DatabaseFactory.getThreadDatabase(context).update(contentValues.getAsLong(THREAD_ID)); DatabaseFactory.getThreadDatabase(context).update(contentValues.getAsLong(THREAD_ID));
db.setTransactionSuccessful();
return messageId;
} finally {
db.endTransaction();
}
return messageId;
} }
public boolean delete(long messageId) { public boolean delete(long messageId) {