Skip to content

Commit 2c00ac4

Browse files
authored
Always echo policy list changes (#401)
closes #392.
1 parent a970733 commit 2c00ac4

File tree

1 file changed

+3
-8
lines changed

1 file changed

+3
-8
lines changed

src/ProtectedRoomsSet.ts

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,7 @@ export class ProtectedRoomsSet {
206206
public async syncLists(verbose = true) {
207207
for (const list of this.policyLists) {
208208
const changes = await list.updateList();
209-
await this.printBanlistChanges(changes, list, true);
209+
await this.printBanlistChanges(changes, list);
210210
}
211211

212212
let hadErrors = false;
@@ -277,7 +277,7 @@ export class ProtectedRoomsSet {
277277
});
278278
}
279279
// This can fail if the change is very large and it is much less important than applying bans, so do it last.
280-
await this.printBanlistChanges(changes, policyList, true);
280+
await this.printBanlistChanges(changes, policyList);
281281
}
282282

283283
/**
@@ -408,14 +408,9 @@ export class ProtectedRoomsSet {
408408
/**
409409
* Print the changes to a banlist to the management room.
410410
* @param changes A list of changes that have been made to a particular ban list.
411-
* @param ignoreSelf Whether to exclude changes that have been made by Mjolnir.
412411
* @returns true if the message was sent, false if it wasn't (because there there were no changes to report).
413412
*/
414-
private async printBanlistChanges(changes: ListRuleChange[], list: PolicyList, ignoreSelf = false): Promise<boolean> {
415-
if (ignoreSelf) {
416-
const sender = await this.client.getUserId();
417-
changes = changes.filter(change => change.sender !== sender);
418-
}
413+
private async printBanlistChanges(changes: ListRuleChange[], list: PolicyList): Promise<boolean> {
419414
if (changes.length <= 0) return false;
420415

421416
let html = "";

0 commit comments

Comments
 (0)