From c527a543a3a9bc79ca6bb77b0c24ecce9bf6f225 Mon Sep 17 00:00:00 2001
From: Konstantin Ulitin <konstantin.ulitin@jetbrains.com>
Date: Thu, 26 Oct 2023 16:26:37 +0200
Subject: [PATCH] [inlay hints] fix settings saving (WEB-60172)

GitOrigin-RevId: 1816515041e4d6506b5aa12b4c7b3f2ce6912272
---
 .../hints/declarative/DeclarativeHintsSettings.kt   | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/platform/lang-api/src/com/intellij/codeInsight/hints/declarative/DeclarativeHintsSettings.kt b/platform/lang-api/src/com/intellij/codeInsight/hints/declarative/DeclarativeHintsSettings.kt
index 0685ef38b60f..7d4a284c4c15 100644
--- a/platform/lang-api/src/com/intellij/codeInsight/hints/declarative/DeclarativeHintsSettings.kt
+++ b/platform/lang-api/src/com/intellij/codeInsight/hints/declarative/DeclarativeHintsSettings.kt
@@ -35,11 +35,16 @@ class DeclarativeInlayHintsSettings : SimplePersistentStateComponent<Declarative
   private fun getSerializedId(providerId: String, optionId: String) = "$providerId#$optionId"
 
   @RequiresWriteLock
-  fun setOptionEnabled(optionId: String, providerId: String, value: Boolean) {
-    if (!value) {
-      state.disabledOptions.add(getSerializedId(providerId, optionId))
+  fun setOptionEnabled(optionId: String, providerId: String, isEnabled: Boolean) {
+    val serializedId = getSerializedId(providerId, optionId)
+    val areSame = state.disabledOptions.contains(serializedId) != isEnabled
+    if (!isEnabled) {
+      state.disabledOptions.add(serializedId)
     } else {
-      state.disabledOptions.remove(getSerializedId(providerId, optionId))
+      state.disabledOptions.remove(serializedId)
+    }
+    if (!areSame) {
+      state.intIncrementModificationCount()
     }
   }