From 447995977b0adb7e88cd88d7861f824a4796fa4c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Manuel=20V=C3=B6gele?= <develop@manuel-voegele.de>
Date: Thu, 15 Apr 2021 17:17:38 +0200
Subject: [PATCH] Don't show the difficult terrain measurement hint when no
 terrain module is enabled

---
 CHANGELOG.md         |  5 +++++
 src/compatibility.js | 34 ++++++++++++++++++----------------
 2 files changed, 23 insertions(+), 16 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 9cca697..86d171f 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,8 @@
+## In development
+### Bugfixes
+- The hint that tells users how to enable difficult terrain measurement in Drag Ruler is no longer shown if no terrain layer module is installed.
+
+
 ## 1.5.0
 ### New features
 - In combat Drag Ruler will now remember the path that was taken by a token during the turn. Picking the token up during the same turn will continue the previous measurement, taking steps that are already taken into account.
diff --git a/src/compatibility.js b/src/compatibility.js
index 7fa7a36..0ac073d 100644
--- a/src/compatibility.js
+++ b/src/compatibility.js
@@ -61,23 +61,25 @@ export function checkDependencies() {
 			else if (game.modules.get("TerrainLayer")?.active) {
 				enabledTerrainModule = game.modules.get("TerrainLayer").data.title;
 			}
-			new Dialog({
-				title: game.i18n.localize("drag-ruler.dependencies.terrain-ruler.title"),
-				content: `<h2>${game.i18n.localize("drag-ruler.dependencies.terrain-ruler.title")}</h2><p>${game.i18n.format("drag-ruler.dependencies.terrain-ruler.text", {moduleName: enabledTerrainModule})}</p>`,
-				buttons: {
-					ok: {
-						icon: '<i class="fas fa-check"></i>',
-						label: game.i18n.localize("drag-ruler.dependencies.ok"),
-						callback: () => game.settings.set(settingsKey, "lastTerrainRulerHintTime", Date.now()),
+			if (enabledTerrainModule) {
+				new Dialog({
+					title: game.i18n.localize("drag-ruler.dependencies.terrain-ruler.title"),
+					content: `<h2>${game.i18n.localize("drag-ruler.dependencies.terrain-ruler.title")}</h2><p>${game.i18n.format("drag-ruler.dependencies.terrain-ruler.text", {moduleName: enabledTerrainModule})}</p>`,
+					buttons: {
+						ok: {
+							icon: '<i class="fas fa-check"></i>',
+							label: game.i18n.localize("drag-ruler.dependencies.ok"),
+							callback: () => game.settings.set(settingsKey, "lastTerrainRulerHintTime", Date.now()),
+						},
+						neverShowAgain: {
+							icon: '<i class="fas fa-times"></i>',
+							label: game.i18n.localize("drag-ruler.dependencies.terrain-ruler.neverShowAgain"),
+							callback: () => game.settings.set(settingsKey, "neverShowTerrainRulerHint", true),
+						}
 					},
-					neverShowAgain: {
-						icon: '<i class="fas fa-times"></i>',
-						label: game.i18n.localize("drag-ruler.dependencies.terrain-ruler.neverShowAgain"),
-						callback: () => game.settings.set(settingsKey, "neverShowTerrainRulerHint", true),
-					}
-				},
-				close: () => game.settings.set(settingsKey, "lastTerrainRulerHintTime", Date.now())
-			}).render(true);
+					close: () => game.settings.set(settingsKey, "lastTerrainRulerHintTime", Date.now())
+				}).render(true);
+			}
 		}
 	}
 }