added comment on why reset state only if res is modified (redhat-deve…

…loper#634) Signed-off-by: Andre Dietisheim <[email protected]>
adietish · Jul 20, 2023 · 97924f2 · 97924f2
1 parent 405ace7
commit 97924f2
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/main/kotlin/com/redhat/devtools/intellij/kubernetes/editor/EditorResource.kt b/src/main/kotlin/com/redhat/devtools/intellij/kubernetes/editor/EditorResource.kt
@@ -57,14 +57,19 @@ open class EditorResource(
      * @see [areEqual]
      * @see [isSameResource]
      * @see [setState]
+     * @see [getState]
      */
     fun setResource(new: HasMetadata) {
         resourceChangeMutex.withLock {
             val existing = this.resource
             if (new.isSameResource(existing)
                 && !areEqual(new, existing)) {
                 this.resource = new
-                setState(null) // reset state
+                /**
+                 * only reset state if resource is modified
+                 * to preserve existing error, pushed/pulled state
+                 */
+                setState(null)
             }
         }
     }