From 9cc42c187942f6c4f9e1075c605a5dd467dd3045 Mon Sep 17 00:00:00 2001
From: Sebastian Goscik <sebastian.goscik@live.co.uk>
Date: Fri, 12 Aug 2016 13:47:54 +0100
Subject: [PATCH] Moved get_type_name outside of ConfigurationPoint

---
 wlauto/core/configuration/configuration.py | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/wlauto/core/configuration/configuration.py b/wlauto/core/configuration/configuration.py
index 9681feb0..c8e0076a 100644
--- a/wlauto/core/configuration/configuration.py
+++ b/wlauto/core/configuration/configuration.py
@@ -129,6 +129,15 @@ KIND_MAP = {
 }
 
 
+def get_type_name(kind):
+    typename = str(kind)
+    if '\'' in typename:
+        typename = typename.split('\'')[1]
+    elif typename.startswith('<function'):
+        typename = typename.split()[1]
+    return typename
+
+
 class ConfigurationPoint(object):
     """
     This defines a generic configuration point for workload automation. This is
@@ -231,7 +240,7 @@ class ConfigurationPoint(object):
             try:
                 value = self.kind(value)
             except (ValueError, TypeError):
-                typename = self.get_type_name()
+                typename = get_type_name(self.kind)
                 msg = 'Bad value "{}" for {}; must be {} {}'
                 article = get_article(typename)
                 raise ConfigError(msg.format(value, self.name, article, typename))
@@ -241,14 +250,6 @@ class ConfigurationPoint(object):
             value = merge_config_values(getattr(obj, self.name), value)
         setattr(obj, self.name, value)
 
-    def get_type_name(self):
-        typename = str(self.kind)
-        if '\'' in typename:
-            typename = typename.split('\'')[1]
-        elif typename.startswith('<function'):
-            typename = typename.split()[1]
-        return typename
-
     def validate(self, obj):
         value = getattr(obj, self.name, None)
         if value is not None: