1 files changed, 22 insertions, 16 deletions
diff --git a/core/models/survex.py b/core/models/survex.py
index b30f231..9d2b17e 100644
--- a/core/models/survex.py
+++ b/core/models/survex.py
@@ -134,12 +134,18 @@ class SurvexBlock(models.Model):
         return True
 
     def GetPersonroles(self):
+        '''To do: excise the 'role' bit of this while retaining personrole
+        which is used in some later logic
+        
+        But apparently never used !?
+        '''
         res = [ ]
         for personrole in self.survexpersonrole_set.order_by('personexpedition'):
-            if res and res[-1]['person'] == personrole.personexpedition.person:
-                res[-1]['roles'] += ", " + str(personrole.nrole)
-            else:
-                res.append({'person':personrole.personexpedition.person, 'expeditionyear':personrole.personexpedition.expedition.year, 'roles':str(personrole.nrole)})
+            # if res and res[-1]['person'] == personrole.personexpedition.person:
+                # res[-1]['roles'] += ", " + str(personrole.nrole)
+            # else:
+                # res.append({'person':personrole.personexpedition.person, 'expeditionyear':personrole.personexpedition.expedition.year, 'roles':str(personrole.nrole)})
+            res.append({'person':personrole.personexpedition.person, 'expeditionyear':personrole.personexpedition.expedition.year})
         return res
    
     def DayIndex(self):
@@ -147,21 +153,21 @@ class SurvexBlock(models.Model):
 #
 # member of a SurvexBlock
 #
-ROLE_CHOICES = (
-        ('insts','Instruments'),
-        ('dog','Other'),
-        ('notes','Notes'),
-        ('pics','Pictures'),
-        ('tape','Tape measure'),
-        ('useless','Useless'),
-        ('helper','Helper'),
-        ('disto','Disto'),
-        ('consultant','Consultant'),
-        )
+# ROLE_CHOICES = (
+        # ('insts','Instruments'),
+        # ('dog','Other'),
+        # ('notes','Notes'),
+        # ('pics','Pictures'),
+        # ('tape','Tape measure'),
+        # ('useless','Useless'),
+        # ('helper','Helper'),
+        # ('disto','Disto'),
+        # ('consultant','Consultant'),
+        # )
 
 class SurvexPersonRole(models.Model):
     survexblock         = models.ForeignKey('SurvexBlock',on_delete=models.CASCADE)
-    nrole               = models.CharField(choices=ROLE_CHOICES, max_length=200, blank=True, null=True)
+#    nrole               = models.CharField(choices=ROLE_CHOICES, max_length=200, blank=True, null=True)
         # increasing levels of precision
     personname          = models.CharField(max_length=100)
     person              = models.ForeignKey('Person', blank=True, null=True,on_delete=models.SET_NULL)