Fix #66 handling of names composed entirely of conjunctions

Author: derek73

docs/release_log.rst

@@ -4,6 +4,7 @@ Release Log
     - Add Dr to suffixes (#62)
     - Add the full set of Italian derivatives from "di" (#59)
     - Add parameter to specify the encoding of strings added to constants, use 'UTF-8' as fallback (#67)
+    - Fix handling of names composed entirely of conjunctions (#66)
 * 0.5.3 - June 27, 2017
     - Remove emojis from initial string by default with option to include emojis (#58)
 * 0.5.2 - March 19, 2017

nameparser/parser.py

@@ -641,7 +641,7 @@ def join_on_conjunctions(self, pieces, additional_parts_count=0):
         
         """
         length = len(pieces) + additional_parts_count
-        # don't join on conjuctions if there's only 2 parts
+        # don't join on conjunctions if there's only 2 parts
         if length < 3:
             return pieces
 
@@ -658,7 +658,7 @@ def join_on_conjunctions(self, pieces, additional_parts_count=0):
         for i, val in enumerate(conj_index):
             try:
                 if conj_index[i+1] == val+1:
-                     contiguous_conj_i += [val]
+                    contiguous_conj_i += [val]
             except IndexError:
                 pass
 
@@ -680,7 +680,11 @@ def join_on_conjunctions(self, pieces, additional_parts_count=0):
         for i in reversed(delete_i):
             # delete pieces in reverse order or the index changes on each delete
             del pieces[i]
-        
+
+        if len(pieces) == 1:
+            # if there's only one piece left, nothing left to do
+            return pieces
+
         # refresh conjunction index locations
         conj_index = [i for i, piece in enumerate(pieces) if self.is_conjunction(piece)]
 

tests.py

@@ -1244,6 +1244,10 @@ def test_conjunction_in_an_address_with_a_first_name_title(self):
         # if you want to be technical, Queen is in FIRST_NAME_TITLES
         self.m(hn.first, "Elizabeth", hn)
 
+    def test_name_is_conjunctions(self):
+        hn = HumanName("e and e")
+        self.m(hn.first, "e and e", hn)
+
 
 class ConstantsCustomization(HumanNameTestBase):