Merge pull request #125 from josemigallas/AEROGEAR-7834

Prevent selecting same resolver twice
aerogear-attic · Sep 5, 2018 · fc4c010 · fc4c010
2 parents a198f8e + 7108bc8
commit fc4c010
Showing 1 changed file with 10 additions and 4 deletions.
diff --git a/ui/components/resolvers/ResolversContainer.js b/ui/components/resolvers/ResolversContainer.js
@@ -25,13 +25,19 @@ class ResolversContainer extends Component {
         return [];
     }
 
-    onResolverClicked(resolver) {
-        const { isResolverSaved } = this.state;
+    onResolverClicked(clickedResolver) {
+        const { resolver, isResolverSaved } = this.state;
+
+        if (resolver !== null
+            && resolver.type === clickedResolver.type
+            && resolver.field === clickedResolver.field) {
+            return;
+        }
 
         if (isResolverSaved) {
-            this.setState({ resolver });
+            this.setState({ resolver: clickedResolver });
         } else {
-            this.setState({ showChangeConfirmation: true, clickedResolver: resolver });
+            this.setState({ showChangeConfirmation: true, clickedResolver });
         }
     }