more robust handling of "name" messages

monty5811 · Oct 19, 2016 · 4215abc · 4215abc
1 parent b4948a9
commit 4215abc
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/apostello/reply.py b/apostello/reply.py
@@ -90,10 +90,12 @@ def reply_to_name(self):
         try:
             # update person's name:
             self.contact.first_name = self.sms_body.split()[1].strip()
-            self.contact.last_name = " ".join(self.sms_body.split()[2:]).strip(
-            )
-            if not self.contact.last_name:
+            last_name = " ".join(self.sms_body.split()[2:]).strip()
+            if not last_name:
                 raise ValidationError('No last name')
+            last_name = last_name.split('\n')[0]
+            last_name = last_name[0:40] # truncate last name
+            self.contact.last_name = last_name
             self.contact.save()
             # update old messages with this person's name
             async('apostello.tasks.update_msgs_name', self.contact.pk)