feat: created getters in message model

replaced all expirationType calls with getExpirationType
pull/2971/head
William Grant 2 years ago
parent 1861131ce2
commit ebaedb2e4c

@ -1809,7 +1809,7 @@ export class ConversationModel extends Backbone.Model<ConversationAttributes> {
identifier: id, identifier: id,
timestamp: sentAt, timestamp: sentAt,
attachments, attachments,
expirationType: message.get('expirationType'), expirationType: message.getExpirationType(),
expireTimer: message.get('expireTimer'), expireTimer: message.get('expireTimer'),
preview: preview ? [preview] : [], preview: preview ? [preview] : [],
quote, quote,

@ -294,7 +294,7 @@ export class MessageModel extends Backbone.Model<MessageAttributes> {
} }
public getPropsForExpiringMessage(): PropsForExpiringMessage { public getPropsForExpiringMessage(): PropsForExpiringMessage {
const expirationType = this.get('expirationType'); const expirationType = this.getExpirationType();
const expirationLength = this.get('expireTimer') const expirationLength = this.get('expireTimer')
? this.get('expireTimer') * DURATION.SECONDS ? this.get('expireTimer') * DURATION.SECONDS
: null; : null;
@ -517,7 +517,7 @@ export class MessageModel extends Backbone.Model<MessageAttributes> {
public getPropsForMessage(): PropsForMessageWithoutConvoProps { public getPropsForMessage(): PropsForMessageWithoutConvoProps {
const sender = this.getSource(); const sender = this.getSource();
const expirationType = this.get('expirationType'); const expirationType = this.getExpirationType();
const expirationLength = this.get('expireTimer') * DURATION.SECONDS; const expirationLength = this.get('expireTimer') * DURATION.SECONDS;
const expireTimerStart = this.get('expirationStartTimestamp'); const expireTimerStart = this.get('expirationStartTimestamp');
const expirationTimestamp = const expirationTimestamp =
@ -1107,7 +1107,7 @@ export class MessageModel extends Backbone.Model<MessageAttributes> {
const convo = this.getConversation(); const convo = this.getConversation();
const canBeDeleteAfterRead = convo && !convo.isMe() && convo.isPrivate(); const canBeDeleteAfterRead = convo && !convo.isMe() && convo.isPrivate();
const expirationType = this.get('expirationType'); const expirationType = this.getExpirationType();
const expireTimer = this.get('expireTimer'); const expireTimer = this.get('expireTimer');
if (canBeDeleteAfterRead && expirationType && expireTimer > 0) { if (canBeDeleteAfterRead && expirationType && expireTimer > 0) {
@ -1349,6 +1349,30 @@ export class MessageModel extends Backbone.Model<MessageAttributes> {
} }
return this.get('body'); return this.get('body');
} }
// NOTE We want to replace Backbone .get() calls with these getters as we migrate to Redux completely eventually
// region Start of getters
public getExpirationType() {
return this.get('expirationType');
}
public getExpireTimer() {
return this.get('expireTimer');
}
public getExpirationStartTimestamp() {
return this.get('expirationStartTimestamp');
}
public getExpiresAt() {
return this.get('expires_at');
}
public getExpirationTimerUpdate() {
return this.get('expirationTimerUpdate');
}
// endregon
} }
const throttledAllMessagesDispatch = debounce( const throttledAllMessagesDispatch = debounce(

@ -708,8 +708,8 @@ async function applyConvoVolatileUpdateFromWrapper(
messages2Expire messages2Expire
.filter(m => .filter(m =>
Boolean( Boolean(
m.get('expirationType') && m.getExpirationType() &&
m.get('expirationType') !== 'deleteAfterSend' && m.getExpirationType() !== 'deleteAfterSend' &&
m.get('expireTimer') > 0 m.get('expireTimer') > 0
) )
) )

@ -339,7 +339,7 @@ async function markConvoAsReadIfOutgoingMessage(
if (isOutgoingMessage) { if (isOutgoingMessage) {
const sentAt = message.get('sent_at') || message.get('serverTimestamp'); const sentAt = message.get('sent_at') || message.get('serverTimestamp');
if (sentAt) { if (sentAt) {
const expirationType = message.get('expirationType'); const expirationType = message.getExpirationType();
const expireTimer = message.get('expireTimer'); const expireTimer = message.get('expireTimer');
// NOTE starting disappearing messages timer for all outbound messages // NOTE starting disappearing messages timer for all outbound messages
if ( if (
@ -399,7 +399,7 @@ export async function handleMessageJob(
) { ) {
const expirationMode = changeToDisappearingConversationMode( const expirationMode = changeToDisappearingConversationMode(
conversation, conversation,
messageModel.get('expirationType'), messageModel.getExpirationType(),
messageModel.get('expireTimer') messageModel.get('expireTimer')
); );

@ -163,7 +163,7 @@ async function send(
await foundMessage.updateMessageHash(messageSendHash); await foundMessage.updateMessageHash(messageSendHash);
const convo = foundMessage.getConversation(); const convo = foundMessage.getConversation();
const expireTimer = foundMessage.get('expireTimer'); const expireTimer = foundMessage.get('expireTimer');
const expirationType = foundMessage.get('expirationType'); const expirationType = foundMessage.getExpirationType();
if ( if (
convo && convo &&

@ -132,7 +132,7 @@ async function handleMessageSentSuccess(
const convo = fetchedMessage.getConversation(); const convo = fetchedMessage.getConversation();
const expireTimer = fetchedMessage.get('expireTimer'); const expireTimer = fetchedMessage.get('expireTimer');
const expirationType = fetchedMessage.get('expirationType'); const expirationType = fetchedMessage.getExpirationType();
if ( if (
convo && convo &&
@ -188,7 +188,7 @@ async function handleMessageSentFailure(
}); });
// Disappeared messages that fail to send should not disappear // Disappeared messages that fail to send should not disappear
if (fetchedMessage.get('expirationType') && fetchedMessage.get('expireTimer') > 0) { if (fetchedMessage.getExpirationType() && fetchedMessage.get('expireTimer') > 0) {
fetchedMessage.set({ fetchedMessage.set({
expirationStartTimestamp: undefined, expirationStartTimestamp: undefined,
}); });

@ -646,7 +646,7 @@ export async function updateMessageExpiryOnSwarm(
window.log.debug(`WIP: [updateMessageExpiryOnSwarm] called from: ${callLocation} `); window.log.debug(`WIP: [updateMessageExpiryOnSwarm] called from: ${callLocation} `);
} }
if (!message.get('expirationType') || !message.get('expireTimer')) { if (!message.getExpirationType() || !message.get('expireTimer')) {
window.log.debug( window.log.debug(
`WIP: [updateMessageExpiryOnSwarm] Message ${message.get( `WIP: [updateMessageExpiryOnSwarm] Message ${message.get(
'messageHash' 'messageHash'

Loading…
Cancel
Save