phlux/mattermostest
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

[MM-46405] Drop support for asterisk-based unreads (#2239)

Browse Source
This commit is contained in:
Devin Binnie
2022-08-25 14:23:29 -04:00
committed by GitHub
parent 78d9a41a7b
commit 840c640fdf
2 changed files with 7 additions and 32 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/main/views/MattermostView.test.js
View File

@@ -342,17 +342,12 @@ describe('main/views/MattermostView', () => {
it('should parse mentions from title', () => {
mattermostView.updateMentionsFromTitle('(7) Mattermost');
expect(appState.updateMentions).toHaveBeenCalledWith(mattermostView.tab.name, 7, undefined);
expect(appState.updateMentions).toHaveBeenCalledWith(mattermostView.tab.name, 7);
});
it('should parse unreads from title', () => {
mattermostView.updateMentionsFromTitle('* Mattermost');
expect(appState.updateMentions).toHaveBeenCalledWith(mattermostView.tab.name, 0, true);
});
it('should not parse unreads when title is on a channel with an asterisk before it', () => {
mattermostView.updateMentionsFromTitle('*testChannel - Mattermost');
expect(appState.updateMentions).toHaveBeenCalledWith(mattermostView.tab.name, 0, false);
expect(appState.updateMentions).toHaveBeenCalledWith(mattermostView.tab.name, 0);
});
});
});

30
src/main/views/MattermostView.ts
View File

@@ -39,7 +39,6 @@ export enum Status {
ERROR = -1,
}
const ASTERISK_GROUP = 3;
const MENTIONS_GROUP = 2;
export class MattermostView extends EventEmitter {
@@ -54,12 +53,6 @@ export class MattermostView extends EventEmitter {
removeLoading?: number;
/**
* for backward compatibility when reading the title.
* null means we have yet to figure out if it uses it or not but we consider it false until proven wrong
*/
usesAsteriskForUnreads?: boolean;
currentFavicon?: string;
hasBeenShown: boolean;
contextMenu: ContextMenu;
@@ -391,33 +384,20 @@ export class MattermostView extends EventEmitter {
}
updateMentionsFromTitle = (title: string) => {
//const title = this.view.webContents.getTitle();
const resultsIterator = title.matchAll(this.titleParser);
const results = resultsIterator.next(); // we are only interested in the first set
// if not using asterisk (version > v5.28), it'll be marked as undefined and wont be used to check if there are unread channels
const hasAsterisk = results && results.value && results.value[ASTERISK_GROUP];
if (typeof hasAsterisk !== 'undefined') {
this.usesAsteriskForUnreads = true;
}
let unreads;
if (this.usesAsteriskForUnreads) {
unreads = Boolean(hasAsterisk);
}
const mentions = (results && results.value && parseInt(results.value[MENTIONS_GROUP], 10)) || 0;
appState.updateMentions(this.tab.name, mentions, unreads);
appState.updateMentions(this.tab.name, mentions);
}
handleFaviconUpdate = (e: Event, favicons: string[]) => {
log.silly('MattermostView.handleFaviconUpdate', {tabName: this.tab.name, favicons});
if (!this.usesAsteriskForUnreads) {
// if unread state is stored for that favicon, retrieve value.
// if not, get related info from preload and store it for future changes
this.currentFavicon = favicons[0];
this.findUnreadState(favicons[0]);
}
// if unread state is stored for that favicon, retrieve value.
// if not, get related info from preload and store it for future changes
this.currentFavicon = favicons[0];
this.findUnreadState(favicons[0]);
}
// if favicon is null, it will affect appState, but won't be memoized