paradizelost/paperless-ngx
1
0
Fork 0
mirror of https://github.com/paperless-ngx/paperless-ngx.git synced 2025-05-01 11:19:32 -05:00
Code Issues Packages Projects Releases Wiki Activity

Fix #384: duplicate tags due to case insensitivity

Browse Source
This commit is contained in:
Daniel Quinn 2018-09-02 20:48:51 +01:00
parent ef0b33e72e
commit 284ff69539
2 changed files with 19 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/documents/models.py
View File

@ -442,8 +442,10 @@ class FileInfo:
def _get_tags(cls, tags): def _get_tags(cls, tags):
r = [] r = []
for t in tags.split(","): for t in tags.split(","):
r.append( r.append(Tag.objects.get_or_create(
Tag.objects.get_or_create(slug=t, defaults={"name": t})[0]) slug=t.lower(),
defaults={"name": t}
)[0])
return tuple(r) return tuple(r)
@classmethod @classmethod

16
src/documents/tests/test_consumer.py
View File

@ -3,7 +3,7 @@ from unittest import mock
from tempfile import TemporaryDirectory from tempfile import TemporaryDirectory
from ..consumer import Consumer from ..consumer import Consumer
from ..models import FileInfo from ..models import FileInfo, Tag
class TestConsumer(TestCase): class TestConsumer(TestCase):
@ -190,6 +190,20 @@ class TestAttributes(TestCase):
() ()
) )
def test_case_insensitive_tag_creation(self):
"""
Tags should be detected and created as lower case.
:return:
"""
path = "Title - Correspondent - tAg1,TAG2.pdf"
self.assertEqual(len(FileInfo.from_path(path).tags), 2)
path = "Title - Correspondent - tag1,tag2.pdf"
self.assertEqual(len(FileInfo.from_path(path).tags), 2)
self.assertEqual(Tag.objects.all().count(), 2)
class TestFieldPermutations(TestCase): class TestFieldPermutations(TestCase):