Refactor usage of type() (ansible#3412)

Addresses ansible#18440
vperron · Dec 8, 2016 · 4617458 · 4617458
1 parent e44362b
commit 4617458
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/lib/ansible/modules/extras/system/osx_defaults.py b/lib/ansible/modules/extras/system/osx_defaults.py
@@ -285,7 +285,8 @@ def run(self):
             return True
 
         # There is a type mismatch! Given type does not match the type in defaults
-        if self.current_value is not None and not isinstance(self.current_value, type(self.value)):
+        value_type = type(self.value)
+        if self.current_value is not None and not isinstance(self.current_value, value_type):
             raise OSXDefaultsException("Type mismatch. Type in defaults: " + type(self.current_value).__name__)
 
         # Current value matches the given value. Nothing need to be done. Arrays need extra care