fix: replaced getMessageBySenderAndSentAt with getMessageBySenderAndTimestamp

which does the same thing but is more verbose
pull/2757/head
William Grant 2 years ago
parent ad38b15809
commit bd8ce826a8

@ -142,7 +142,6 @@ export const Data = {
removeMessagesByIds, removeMessagesByIds,
getMessageIdsFromServerIds, getMessageIdsFromServerIds,
getMessageById, getMessageById,
getMessageBySenderAndSentAt,
getMessageByServerId, getMessageByServerId,
filterAlreadyFetchedOpengroupMessage, filterAlreadyFetchedOpengroupMessage,
getMessageBySenderAndTimestamp, getMessageBySenderAndTimestamp,
@ -427,24 +426,6 @@ async function getMessageById(
return new MessageModel(message); return new MessageModel(message);
} }
async function getMessageBySenderAndSentAt({
source,
sentAt,
}: {
source: string;
sentAt: number;
}): Promise<MessageModel | null> {
const messages = await channels.getMessageBySenderAndSentAt({
source,
sentAt,
});
if (!messages || !messages.length) {
return null;
}
return new MessageModel(messages[0]);
}
async function getMessageByServerId( async function getMessageByServerId(
conversationId: string, conversationId: string,
serverId: number, serverId: number,

@ -45,7 +45,6 @@ const channelsToMake = new Set([
'getMessageCountByType', 'getMessageCountByType',
'removeAllMessagesInConversation', 'removeAllMessagesInConversation',
'getMessageCount', 'getMessageCount',
'getMessageBySenderAndSentAt',
'filterAlreadyFetchedOpengroupMessage', 'filterAlreadyFetchedOpengroupMessage',
'getMessageBySenderAndTimestamp', 'getMessageBySenderAndTimestamp',
'getMessageIdsFromServerIds', 'getMessageIdsFromServerIds',

@ -1025,21 +1025,6 @@ function getMessageById(id: string) {
return jsonToObject(row.json); return jsonToObject(row.json);
} }
function getMessageBySenderAndSentAt({ source, sentAt }: { source: string; sentAt: number }) {
const rows = assertGlobalInstance()
.prepare(
`SELECT json FROM ${MESSAGES_TABLE} WHERE
source = $source AND
sent_at = $sent_at;`
)
.all({
source,
sent_at: sentAt,
});
return map(rows, row => jsonToObject(row.json));
}
// serverIds are not unique so we need the conversationId // serverIds are not unique so we need the conversationId
function getMessageByServerId(conversationId: string, serverId: number) { function getMessageByServerId(conversationId: string, serverId: number) {
const row = assertGlobalInstance() const row = assertGlobalInstance()
@ -2474,7 +2459,6 @@ export const sqlNode = {
getUnreadCountByConversation, getUnreadCountByConversation,
getMessageCountByType, getMessageCountByType,
getMessageBySenderAndSentAt,
filterAlreadyFetchedOpengroupMessage, filterAlreadyFetchedOpengroupMessage,
getMessageBySenderAndTimestamp, getMessageBySenderAndTimestamp,
getMessageIdsFromServerIds, getMessageIdsFromServerIds,

@ -258,9 +258,9 @@ export async function isSwarmMessageDuplicate({
sentAt: number; sentAt: number;
}) { }) {
try { try {
const result = await Data.getMessageBySenderAndSentAt({ const result = await Data.getMessageBySenderAndTimestamp({
source, source,
sentAt, timestamp: sentAt,
}); });
return Boolean(result); return Boolean(result);

Loading…
Cancel
Save