aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlexis Metaireau <alexis@notmyidea.org>2016-09-12 17:54:58 +0200
committerGitHub <noreply@github.com>2016-09-12 17:54:58 +0200
commit7ba9048b0642d88e1298032243aa88a4a501afab (patch)
tree2b7d0a478479e669e92e4c7d35613496d74f509a
parent30904481aab39bcb33e52ae7ae844f363c9408cf (diff)
parentc9b2d17f603bd34dc15b66c123d7b17228362b33 (diff)
downloadihatemoney-mirror-7ba9048b0642d88e1298032243aa88a4a501afab.zip
ihatemoney-mirror-7ba9048b0642d88e1298032243aa88a4a501afab.tar.gz
ihatemoney-mirror-7ba9048b0642d88e1298032243aa88a4a501afab.tar.bz2
Merge pull request #150 from fredericsureau/negative-amounts
Allow negative bill amounts
-rw-r--r--budget/forms.py4
-rw-r--r--budget/tests.py4
2 files changed, 2 insertions, 6 deletions
diff --git a/budget/forms.py b/budget/forms.py
index 7d6eb51..4b59a36 100644
--- a/budget/forms.py
+++ b/budget/forms.py
@@ -143,9 +143,7 @@ class BillForm(Form):
self.payed_for.data = self.payed_for.default
def validate_amount(self, field):
- if field.data < 0:
- field.data = abs(field.data)
- elif field.data == 0:
+ if field.data == 0:
raise ValidationError(_("Bills can't be null"))
diff --git a/budget/tests.py b/budget/tests.py
index 82465f9..eea7537 100644
--- a/budget/tests.py
+++ b/budget/tests.py
@@ -398,12 +398,10 @@ class BudgetTestCase(TestCase):
'what': u'fromage à raclette',
'payer': members_ids[0],
'payed_for': members_ids,
- # bill with a negative value should be converted to a positive
- # value
'amount': '-25'
})
bill = models.Bill.query.filter(models.Bill.date == '2011-08-12')[0]
- self.assertEqual(bill.amount, 25)
+ self.assertEqual(bill.amount, -25)
#add a bill with a comma
self.app.post("/raclette/add", data={