Get rid of fill_list and rather use defaultdict

src/documents/models.py

@@ -328,11 +328,8 @@ class Document(models.Model):
         if settings.PAPERLESS_FILENAME_FORMAT is not None:
             tag = defaultdict(lambda: slugify(None),
                               self.many_to_dictionary(self.tags))
-            list_length = 10
+            tags = defaultdict(lambda: slugify(None),
-            tags = self.many_to_list(self.tags)
+                               enumerate(self.many_to_list(self.tags)))
-            while True:
-                tags = Document.fill_list(tags, list_length, None)
-                try:
             path = settings.PAPERLESS_FILENAME_FORMAT.format(
                    correspondent=slugify(self.correspondent),
                    title=slugify(self.title),
@@ -340,13 +337,6 @@ class Document(models.Model):
                    added=slugify(self.added),
                    tag=tag,
                    tags=tags)
-                    break
-                except IndexError:
-                    list_length *= 10
-
-                if list_length > 1000:
-                    path = ""
-                    break
         else:
             path = ""
 

src/documents/tests/test_file_handling.py

@@ -264,7 +264,7 @@ class TestDate(TestCase):
         # Ensure that filename is properly generated
         tmp = document.source_filename
         self.assertEqual(document.generate_source_filename(),
-                         "0000001.pdf")
+                         "none-0000001.pdf")
         document.create_source_directory()
         Path(document.source_path).touch()