diff --git a/src/channelsyncroniser.ts b/src/channelsyncroniser.ts
index 3078a61c8c134d382324fcc18a358e3e703a4678..bb0939a7a766dc02e577b7f250a0835289c2e2e7 100644
--- a/src/channelsyncroniser.ts
+++ b/src/channelsyncroniser.ts
@@ -179,7 +179,8 @@ export class ChannelSyncroniser {
             }
             
             const oldIconUrl = remoteRoom.remote.get("discord_iconurl");
-            if (remoteRoom.remote.get("update_icon") && oldIconUrl !== iconUrl) { // no force on icon update as we don't want to duplicate ALL the icons
+            // no force on icon update as we don't want to duplicate ALL the icons
+            if (remoteRoom.remote.get("update_icon") && oldIconUrl !== iconUrl) {
                 log.verbose(`Channel ${mxid} icon should be updated`);
                 if (iconUrl !== null) {
                     singleChannelState.iconUrl = iconUrl;
diff --git a/tools/chanfix.ts b/tools/chanfix.ts
index 706446f7ebd6f730c954ff9f1c64205ac17f8034..4c57c858480cdc7fc7ffe0e01a5958b44ddd6591 100644
--- a/tools/chanfix.ts
+++ b/tools/chanfix.ts
@@ -114,8 +114,8 @@ bridge.loadDatabases().catch((e) => {
         if (entry.remote.get("plumbed")) {
             return; // skipping plumbed rooms
         }
-        const update_icon = entry.remote.get("update_icon");
-        if (update_icon !== undefined && update_icon !== null) {
+        const updateIcon = entry.remote.get("update_icon");
+        if (updateIcon !== undefined && updateIcon !== null) {
             return; // skipping because something was set manually
         }
         entry.remote.set("update_icon", true);