Arsharth/hydrogen-web
1
0
Fork 0
forked from mystiq/hydrogen-web
Code Issues Pull requests Projects Releases Packages Wiki Activity

add logging for room list sorting

Browse source
This commit is contained in:
Bruno Windels 2020-08-27 10:40:49 +02:00
parent 3e8e1bab67
commit 41a7448c74
1 changed files with 33 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

41
src/domain/session/roomlist/RoomTileViewModel.js
View file

@ -49,34 +49,59 @@ export class RoomTileViewModel extends ViewModel {
}
compare(other) {
/*
put unread rooms first
then put rooms with a timestamp first, and sort by name
then sort by name for rooms without a timestamp
*/
const myRoom = this._room;
const theirRoom = other._room;
let buf = "";
function log(...args) {
buf = buf + args.join(" ") + "\n";
}
function logResult(result) {
if (result === 0) {
log("rooms are equal (should not happen)", result);
} else if (result > 0) {
log(`${theirRoom.name || theirRoom.id} comes first`, result);
} else {
log(`${myRoom.name || myRoom.id} comes first`, result);
}
console.log(buf);
return result;
}
log(`comparing ${myRoom.name || theirRoom.id} and ${theirRoom.name || theirRoom.id} ...`);
log("comparing isUnread...");
if (isSortedAsUnread(this) !== isSortedAsUnread(other)) {
if (isSortedAsUnread(this)) {
return -1;
return logResult(-1);
}
return 1;
return logResult(1);
}
const myTimestamp = myRoom.lastMessageTimestamp;
const theirTimestamp = theirRoom.lastMessageTimestamp;
// rooms with a timestamp come before rooms without one
if ((myTimestamp === null) !== (theirTimestamp === null)) {
log("checking if either does not have lastMessageTimestamp ...");
if (theirTimestamp === null) {
return -1;
return logResult(-1);
}
return 1;
return logResult(1);
}
const timeDiff = theirTimestamp - myTimestamp;
if (timeDiff === 0) {
if (timeDiff === 0 || !Number.isSafeInteger(theirTimestamp) || !Number.isSafeInteger(myTimestamp)) {
log("checking name ...");
// sort alphabetically
const nameCmp = this.name.localeCompare(other.name);
if (nameCmp === 0) {
return this._room.id.localeCompare(other._room.id);
return logResult(this._room.id.localeCompare(other._room.id));
}
return nameCmp;
return logResult(nameCmp);
}
return timeDiff;
log("checking timestamp ...");
return logResult(timeDiff);
}
get isOpen() {