[snappymail] Rework FolderCollectionModel .js a bit

Following the previous bugfix, rework the .js code a bit to reduce the
amount of nested ifdeffery.
This commit is contained in:
David Härdeman 2021-08-25 00:19:39 +02:00
parent 0190170edd
commit 17f107057a

View file

@ -72,9 +72,11 @@ export class FolderCollectionModel extends AbstractCollectionModel
if (oCacheFolder) { if (oCacheFolder) {
oFolder.SubFolders = FolderCollectionModel.reviveFromJson(oFolder.SubFolders); oFolder.SubFolders = FolderCollectionModel.reviveFromJson(oFolder.SubFolders);
oFolder.SubFolders && oCacheFolder.subFolders(oFolder.SubFolders); oFolder.SubFolders && oCacheFolder.subFolders(oFolder.SubFolders);
} } else {
oCacheFolder = FolderModel.reviveFromJson(oFolder);
if (!oCacheFolder)
return null;
if (!oCacheFolder && (oCacheFolder = FolderModel.reviveFromJson(oFolder))) {
if (1 == SystemFolders.indexOf(oFolder.FullNameRaw)) { if (1 == SystemFolders.indexOf(oFolder.FullNameRaw)) {
oCacheFolder.type(FolderType.Inbox); oCacheFolder.type(FolderType.Inbox);
Cache.setFolderInboxName(oFolder.FullNameRaw); Cache.setFolderInboxName(oFolder.FullNameRaw);
@ -82,7 +84,6 @@ export class FolderCollectionModel extends AbstractCollectionModel
Cache.setFolder(oCacheFolder.fullNameHash, oFolder.FullNameRaw, oCacheFolder); Cache.setFolder(oCacheFolder.fullNameHash, oFolder.FullNameRaw, oCacheFolder);
} }
if (oCacheFolder) {
let type = SystemFolders.indexOf(oFolder.FullNameRaw); let type = SystemFolders.indexOf(oFolder.FullNameRaw);
if (1 < type) { if (1 < type) {
oCacheFolder.type(type); oCacheFolder.type(type);
@ -105,7 +106,6 @@ export class FolderCollectionModel extends AbstractCollectionModel
oCacheFolder.messageCountUnread(oFolder.Extended.MessageUnseenCount); oCacheFolder.messageCountUnread(oFolder.Extended.MessageUnseenCount);
} }
} }
}
return oCacheFolder; return oCacheFolder;
}); });
} }