diff --git a/app/src/main/java/org/thoughtcrime/securesms/database/MessageDatabase.java b/app/src/main/java/org/thoughtcrime/securesms/database/MessageDatabase.java index fb33b27fe..e34c99a7c 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/database/MessageDatabase.java +++ b/app/src/main/java/org/thoughtcrime/securesms/database/MessageDatabase.java @@ -206,7 +206,7 @@ public abstract class MessageDatabase extends Database implements MmsSmsColumns public abstract @NonNull List getUnreadStoryThreadRecipientIds(); public abstract boolean containsStories(long threadId); public abstract boolean hasSelfReplyInStory(long parentStoryId); - public abstract boolean hasSelfReplyInGroupStory(long parentStoryId); + public abstract boolean hasGroupReplyOrReactionInStory(long parentStoryId); public abstract @NonNull Cursor getStoryReplies(long parentStoryId); public abstract @Nullable Long getOldestStorySendTimestamp(boolean hasSeenReleaseChannelStories); public abstract int deleteStoriesOlderThan(long timestamp, boolean hasSeenReleaseChannelStories); diff --git a/app/src/main/java/org/thoughtcrime/securesms/database/MmsDatabase.java b/app/src/main/java/org/thoughtcrime/securesms/database/MmsDatabase.java index ae07e5d18..66af01c91 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/database/MmsDatabase.java +++ b/app/src/main/java/org/thoughtcrime/securesms/database/MmsDatabase.java @@ -896,17 +896,8 @@ public class MmsDatabase extends MessageDatabase { } @Override - public boolean hasSelfReplyInGroupStory(long parentStoryId) { - SQLiteDatabase db = databaseHelper.getSignalReadableDatabase(); - String[] columns = new String[]{"COUNT(*)"}; - String where = PARENT_STORY_ID + " = ? AND (" + - getOutgoingTypeClause() + ") AND (" - + MESSAGE_BOX + " & " + Types.SPECIAL_TYPES_MASK + " != " + Types.SPECIAL_TYPE_STORY_REACTION + ")"; - String[] whereArgs = SqlUtil.buildArgs(parentStoryId); - - try (Cursor cursor = db.query(TABLE_NAME, columns, where, whereArgs, null, null, null, null)) { - return cursor != null && cursor.moveToNext() && cursor.getInt(0) > 0; - } + public boolean hasGroupReplyOrReactionInStory(long parentStoryId) { + return hasSelfReplyInStory(-parentStoryId); } @Override diff --git a/app/src/main/java/org/thoughtcrime/securesms/database/SmsDatabase.java b/app/src/main/java/org/thoughtcrime/securesms/database/SmsDatabase.java index babe25c46..17c199446 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/database/SmsDatabase.java +++ b/app/src/main/java/org/thoughtcrime/securesms/database/SmsDatabase.java @@ -1492,7 +1492,7 @@ public class SmsDatabase extends MessageDatabase { } @Override - public boolean hasSelfReplyInGroupStory(long parentStoryId) { + public boolean hasGroupReplyOrReactionInStory(long parentStoryId) { throw new UnsupportedOperationException(); } diff --git a/app/src/main/java/org/thoughtcrime/securesms/notifications/v2/NotificationStateProvider.kt b/app/src/main/java/org/thoughtcrime/securesms/notifications/v2/NotificationStateProvider.kt index eb5520379..82b0bfbf6 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/notifications/v2/NotificationStateProvider.kt +++ b/app/src/main/java/org/thoughtcrime/securesms/notifications/v2/NotificationStateProvider.kt @@ -43,7 +43,7 @@ object NotificationStateProvider { } val hasSelfRepliedToGroupStory = conversationId.groupStoryId?.let { - SignalDatabase.mms.hasSelfReplyInGroupStory(it) + SignalDatabase.mms.hasGroupReplyOrReactionInStory(it) } messages += NotificationMessage(