Fix issue where profile changes could be interpreted as tag renames, …

…resulting in loss of data.

Bump to version 1.4.21

build.gradle

@@ -27,7 +27,7 @@ dependencies {
 }
 
 group = 'com.banktaglayouts'
-version = '1.4.20'
+version = '1.4.21'
 sourceCompatibility = '1.8'
 
 tasks.withType(JavaCompile) {

src/main/java/com/banktaglayouts/BankTagLayoutsPlugin.java

@@ -366,6 +366,10 @@ void checkVersionUpgrade() {
 	}
 
 	private void handlePotentialTagRename(ConfigChanged event) {
+		// Profile changes can look like tag renames sometimes, but we do not want to modify the config in that case
+		// because it can cause people to lose their data. Real renames come through on the client thread.
+		if (!client.isClientThread()) return;
+
 		String oldValue = event.getOldValue();
 		String newValue = event.getNewValue();
 		Set<String> oldTags = new HashSet<>(Text.fromCSV(oldValue == null ? "" : oldValue));