Touch up IPAddressType; make work in python 2.

setup.py

@@ -42,6 +42,7 @@ extras_require = {
     ],
     'password': ['passlib >= 1.6, < 2.0'],
     'color': ['colour>=0.0.4'],
+    'ipaddress': ['ipaddr'] if not PY3 else []
 }
 
 

sqlalchemy_utils/types/ip_address.py

@@ -1,45 +1,43 @@
 import six
 
-ipaddress = None
 try:
-    import ipaddress
+    from ipaddress import ip_address
-except:
+
-    pass
+except ImportError:
+    try:
+        from ipaddr import IPAddress as ip_address
+
+    except ImportError:
+        ip_address = None
+
+
 from sqlalchemy import types
 from sqlalchemy_utils import ImproperlyConfigured
 
 
 class IPAddressType(types.TypeDecorator):
     """
-    Changes Color objects to a string representation on the way in and
+    Changes IPAddress objects to a string representation on the way in and
-    changes them back to Color objects on the way out.
+    changes them back to IPAddress objects on the way out.
     """
+
     impl = types.Unicode(50)
 
     def __init__(self, max_length=50, *args, **kwargs):
-        if not ipaddress:
+        if not ip_address:
             raise ImproperlyConfigured(
-                "'ipaddress' package is required to use 'IPAddressType'"
+                "'ipaddr' package is required to use 'IPAddressType' "
+                "in python 2"
             )
 
         super(IPAddressType, self).__init__(*args, **kwargs)
         self.impl = types.Unicode(max_length)
 
     def process_bind_param(self, value, dialect):
-        if value:
+        return six.text_type(value) if value else None
-            return six.text_type(value)
-        return value
 
     def process_result_value(self, value, dialect):
-        if value:
+        return ip_address(value) if value else None
-            return ipaddress.ip_address(value)
-        return value
 
     def coercion_listener(self, target, value, oldvalue, initiator):
-        if (
+        return ip_address(value) if value else None
-            value is not None and
-            not isinstance(value, ipaddress.IPv4Address) and
-            not isinstance(value, ipaddress.IPv6Address)
-        ):
-            value = ipaddress.ip_address(value)
-        return value

tests/test_ip_address.py

@@ -5,7 +5,7 @@ from sqlalchemy_utils.types import ip_address
 from tests import TestCase
 
 
-@mark.skipif('ip_address.ipaddress is None')
+@mark.skipif('ip_address.ip_address is None')
 class TestIPAddressType(TestCase):
     def create_models(self):
         class Visitor(self.Base):