Fix Chat Features causing issues on journals

Fix #415

Author: mclemente

src/module/hooks.js

@@ -26,13 +26,13 @@ export default class PolyglotHooks {
 	static updateActor(actor, data, options, userId) {
 		if (actor.hasPlayerOwner && actor.testUserPermission(game.user, "OWNER")) {
 			game.polyglot.updateUserLanguages();
-			game.polyglot.updateChatMessages();
+			if (game.polyglot._enableChatFeatures) game.polyglot.updateChatMessages();
 		}
 	}
 
 	static controlToken() {
 		game.polyglot.updateUserLanguages();
-		game.polyglot.updateChatMessages();
+		if (game.polyglot._enableChatFeatures) game.polyglot.updateChatMessages();
 	}
 
 	/**
@@ -209,31 +209,20 @@ export default class PolyglotHooks {
 		const isOwnerOrGM = sheet.document?.isOwner || game.user.isGM;
 		const isEditable = data.editable;
 		const isTextSheet = sheet instanceof JournalTextPageSheet;
+		const isJQuery = html instanceof jQuery;
 
 		if (isTextSheet && !(sheet.object.parent.isOwner || isOwnerOrGM || isEditable)) {
 			if (sheet.document.isOwner) game.polyglot.insertHeaderButton(sheet.object.parent.sheet, html);
 			else game.polyglot.scrambleSpans(sheet, html);
-		} else if (html.find(".polyglot-journal").length) {
+		} else if (isJQuery && html.find(".polyglot-journal").length) {
 			if (isOwnerOrGM && html.find('[data-engine="prosemirror"]').length) game.polyglot.insertHeaderButton(sheet, html);
 			else if (!(isOwnerOrGM || isEditable)) game.polyglot.scrambleSpans(sheet, html);
+		} else if (!isJQuery && html.querySelectorAll(".polyglot-journal").length) {
+			if (isOwnerOrGM && html.querySelectorAll('[data-engine="prosemirror"]').length) game.polyglot.insertHeaderButton(sheet, html);
+			else if (!(isOwnerOrGM || isEditable)) game.polyglot.scrambleSpansV2(sheet, html);
 		}
 	}
 
-	/** @see renderDocumentSheet */
-	static renderActorSheet(sheet, html, data) {
-		PolyglotHooks.renderDocumentSheet(sheet, html, data);
-	}
-
-	/** @see renderDocumentSheet */
-	static renderItemSheet(sheet, html, data) {
-		PolyglotHooks.renderDocumentSheet(sheet, html, data);
-	}
-
-	/** @see renderDocumentSheet */
-	static renderJournalTextPageSheet(journalTextPageSheet, html, data) {
-		PolyglotHooks.renderDocumentSheet(journalTextPageSheet, html, data);
-	}
-
 	/**
 	 * Renders a journal entry, adding the scrambling button to its header in case user is the document's owner or a GM.
 	 *
@@ -247,11 +236,6 @@ export default class PolyglotHooks {
 		}
 	}
 
-	/** @see renderJournalSheet */
-	static renderStorySheet(sheet, html) {
-		PolyglotHooks.renderJournalSheet(sheet, html);
-	}
-
 	static getProseMirrorMenuDropDowns(menu, items) {
 		if ("format" in items) {
 			if ("format" in items) {

src/module/logic.js

@@ -14,24 +14,27 @@ export class Polyglot {
 	}
 
 	init() {
-		if (game.settings.get("polyglot", "enableChatFeatures")) {
+		this._enableChatFeatures = game.settings.get("polyglot", "enableChatFeatures");
+		if (this._enableChatFeatures) {
 			Hooks.on("renderChatLog", PolyglotHooks.renderChatLog);
-			Hooks.on("updateActor", PolyglotHooks.updateActor);
-			Hooks.on("controlToken", PolyglotHooks.controlToken);
-			Hooks.on("updateUser", PolyglotHooks.updateUser);
-			Hooks.on("updateActiveEffect", PolyglotHooks.updateActiveEffect);
 			Hooks.on("preCreateChatMessage", PolyglotHooks.preCreateChatMessage);
 			Hooks.on("renderChatMessageHTML", PolyglotHooks.renderChatMessageHTML);
 			Hooks.on("createChatMessage", PolyglotHooks.createChatMessage);
 			Hooks.on("renderActorDirectoryPF2e", PolyglotHooks.renderActorDirectoryPF2e);
 			Hooks.on("vinoPrepareChatDisplayData", PolyglotHooks.vinoPrepareChatDisplayData);
 		}
+		Hooks.on("updateActor", PolyglotHooks.updateActor);
+		Hooks.on("controlToken", PolyglotHooks.controlToken);
+		Hooks.on("updateUser", PolyglotHooks.updateUser);
+		Hooks.on("updateActiveEffect", PolyglotHooks.updateActiveEffect);
+		Hooks.on("renderDocumentSheetV2", PolyglotHooks.renderDocumentSheet);
 		Hooks.on("renderDocumentSheet", PolyglotHooks.renderDocumentSheet);
-		Hooks.on("renderActorSheet", PolyglotHooks.renderActorSheet);
-		Hooks.on("renderItemSheet", PolyglotHooks.renderItemSheet);
-		Hooks.on("renderJournalTextPageSheet", PolyglotHooks.renderJournalTextPageSheet);
+		Hooks.on("renderActorSheet", PolyglotHooks.renderDocumentSheet);
+		Hooks.on("renderItemSheet", PolyglotHooks.renderDocumentSheet);
+		Hooks.on("renderJournalTextPageSheet", PolyglotHooks.renderDocumentSheet);
 		Hooks.on("renderJournalSheet", PolyglotHooks.renderJournalSheet);
-		Hooks.on("renderStorySheet", PolyglotHooks.renderStorySheet);
+		Hooks.on("renderStorySheet", PolyglotHooks.renderJournalSheet);
+		Hooks.on("renderJournalEntryPageSheet", PolyglotHooks.renderDocumentSheet);
 		Hooks.on("getProseMirrorMenuDropDowns", PolyglotHooks.getProseMirrorMenuDropDowns);
 		Polyglot.handleTinyMCE();
 
@@ -506,6 +509,21 @@ export class Polyglot {
 		});
 	}
 
+	scrambleSpansV2(document, html) {
+		html.querySelectorAll("span.polyglot-journal").forEach((e) => {
+			const lang = e.dataset.language;
+			if (!lang) return;
+			const conditions = !game.polyglot._isTruespeech(lang)
+				&& !game.polyglot.isLanguageKnown(game.polyglot.comprehendLanguages)
+				&& !game.polyglot.languageProvider.conditions(lang);
+			if (conditions) {
+				e.dataset.tooltip = "????";
+				e.textContent = game.polyglot.scrambleString(e.textContent, document.id, lang);
+				e.style.font = game.polyglot._getFontStyle(lang);
+			}
+		});
+	}
+
 	knows(lang) {
 		return this.knownLanguages.has(lang);
 	}