summaryrefslogtreecommitdiff
path: root/archaeological_operations/forms.py
diff options
context:
space:
mode:
authorÉtienne Loks <etienne.loks@iggdrasil.net>2018-08-01 12:41:44 +0200
committerÉtienne Loks <etienne.loks@iggdrasil.net>2018-08-13 18:26:03 +0200
commitb93797feaa842068f7b93676bddb7ca3e14eb356 (patch)
tree34a471d9e263947fdfc6e6f1c2287f2fd3cb411a /archaeological_operations/forms.py
parent5bd94ea3b1083607642afaf014d2663ce420febf (diff)
downloadIshtar-b93797feaa842068f7b93676bddb7ca3e14eb356.tar.bz2
Ishtar-b93797feaa842068f7b93676bddb7ca3e14eb356.zip
Fix innapropriate parcel deletion
Diffstat (limited to 'archaeological_operations/forms.py')
-rw-r--r--archaeological_operations/forms.py28
1 files changed, 14 insertions, 14 deletions
diff --git a/archaeological_operations/forms.py b/archaeological_operations/forms.py
index acce3835b..866700871 100644
--- a/archaeological_operations/forms.py
+++ b/archaeological_operations/forms.py
@@ -77,20 +77,6 @@ class ParcelForm(IshtarForm):
public_domain = forms.BooleanField(label=_(u"Public domain"),
initial=False, required=False)
- def count_valid_fields(self, data):
- if not data:
- return 0
- data = get_data_from_formset(data)
- nb = len(data)
- # remove last non relevant fields
- for idx, vals in enumerate(reversed(data[:])):
- if 'public_domain' in vals:
- break
- if 'section' in vals and 'parcel_number' in vals:
- break
- nb -= 1
- return nb
-
def __init__(self, *args, **kwargs):
towns = None
if 'data' in kwargs and 'TOWNS' in kwargs['data']:
@@ -106,6 +92,20 @@ class ParcelForm(IshtarForm):
if towns:
self.fields['town'].choices = towns
+ def count_valid_fields(self, data):
+ if not data:
+ return 0
+ data = get_data_from_formset(data)
+ nb = len(data)
+ # remove last non relevant fields
+ for idx, vals in enumerate(reversed(data[:])):
+ if 'public_domain' in vals:
+ break
+ if 'section' in vals and 'parcel_number' in vals:
+ break
+ nb -= 1
+ return nb
+
def clean(self):
"""Check required fields"""
if any(self.errors):