diff options
author | Jeremy Kerr <jk@ozlabs.org> | 2011-04-19 13:09:36 +0800 |
---|---|---|
committer | Jeremy Kerr <jk@ozlabs.org> | 2011-04-19 13:09:36 +0800 |
commit | fa7c42111829420729d7f024a694196aefb299b4 (patch) | |
tree | 12f1b3be3a0fd925ad4e0d6f7c78ae224db240c2 | |
parent | c3291f5d18445cd91b540342d31d76254b32376c (diff) | |
download | patchwork-fa7c42111829420729d7f024a694196aefb299b4.tar.bz2 patchwork-fa7c42111829420729d7f024a694196aefb299b4.tar.xz |
Order: always set Order.order
If we create an Order objecty with editable = True, we don't initialise
self.order, leading to testsuite failures with django-1.3.
This change always initialises Order.order (and Order.reversed), so we
don't exit the constructor without these initalised.
Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
-rw-r--r-- | apps/patchwork/utils.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/apps/patchwork/utils.py b/apps/patchwork/utils.py index e41ffb6..5a8e4c0 100644 --- a/apps/patchwork/utils.py +++ b/apps/patchwork/utils.py @@ -49,12 +49,12 @@ class Order(object): def __init__(self, str = None, editable = False): self.reversed = False self.editable = editable + (self.order, self.reversed) = self.default_order if self.editable: return if str is None or str == '': - (self.order, self.reversed) = self.default_order return reversed = False @@ -63,7 +63,6 @@ class Order(object): reversed = True if str not in self.order_map.keys(): - (self.order, self.reversed) = self.default_order return self.order = str |