Cleanup a few things

money/django/fields.py

@@ -10,7 +10,7 @@
 def _currency_field_name(name): return f"{name}_currency"
 
 
-class MoneyFieldProxy(object):
+class MoneyFieldProxy:
     def __init__(self, field):
         self.field = field
         self.currency_name = _currency_field_name(self.field.name)
@@ -29,8 +29,7 @@ def __get__(self, obj, type=None):
         amount = obj.__dict__[self.field.name]
         if amount is None:
             return None
-        else:
-            return Money(amount, getattr(obj, self.currency_name))
+        return Money(amount, getattr(obj, self.currency_name))
 
 
 class CurrencyField(models.CharField):

money/exchange.py

@@ -94,7 +94,7 @@ def __getattr__(self, item):
         return getattr(self._backend, item)
 
     def __setattr__(self, key, value):
-        if key == '_backend' or key == 'backend':
+        if key in ('_backend', 'backend'):
             return super().__setattr__(key, value)
         if self._backend is None:
             raise ExchangeBackendNotSet()

money/money.py

@@ -34,6 +34,15 @@ def operator_func(self, other, *args):
     return operator_func
 
 
+def _make_class_operator(name):
+    method = getattr(Decimal, name, None)
+
+    def operator_func(self, *args):
+        return self.__class__(method(self, *args), self._currency)
+
+    return operator_func
+
+
 class Money(Decimal):
     """Class representing a monetary amount."""
 
@@ -84,8 +93,7 @@ def __eq__(self, other):
 
         if isinstance(other, Money):
             return result and other._currency == self._currency
-        else:
-            return result
+        return result
 
     def __ne__(self, other):
         return not self == other
@@ -114,23 +122,12 @@ def __ne__(self, other):
     __pow__ = _make_arithmetic_operator('__pow__')
     __rpow__ = _make_arithmetic_operator('__rpow__')
 
-    def __neg__(self):
-        return self.__class__(super().__neg__(), self._currency)
-
-    def __pos__(self):
-        return self.__class__(super().__pos__(), self._currency)
-
-    def __abs__(self):
-        return self.__class__(super().__abs__(), self._currency)
-
-    def __round__(self, n=0):
-        return self.__class__(super().__round__(n), self._currency)
-
-    def __floor__(self):
-        return self.__class__(super().__floor__(), self._currency)
-
-    def __ceil__(self):
-        return self.__class__(super().__ceil__(), self._currency)
+    __neg__ = _make_class_operator('__neg__')
+    __pos__ = _make_class_operator('__pos__')
+    __abs__ = _make_class_operator('__abs__')
+    __round__ = _make_class_operator('__round__')
+    __floor__ = _make_class_operator('__floor__')
+    __ceil__ = _make_class_operator('__ceil__')
 
     def to(self, currency):
         """Returns the equivalent money object in another currency."""
@@ -153,7 +150,7 @@ def to(self, currency):
     def format(self, locale='en_US'):
         """Returns a string of the currency formatted for the specified locale."""
 
-        return format_currency(self, self.currency.code, locale=locale).replace(u'\xa0', u' ')
+        return format_currency(self, self.currency.code, locale=locale).replace('\xa0', ' ')
 
     @classmethod
     def set_rounding_mode(cls, mode):