From aa6e96e54d6a29ef5a8ea5478457cc63515ec456 Mon Sep 17 00:00:00 2001
From: jonaswinkler <jonas.winkler@jpwinkler.de>
Date: Mon, 28 Dec 2020 13:33:58 +0100
Subject: [PATCH] fix pycodestyle
---
src/documents/bulk_edit.py | 6 +++---
src/documents/serialisers.py | 12 ++++++++----
2 files changed, 11 insertions(+), 7 deletions(-)
diff --git a/src/documents/bulk_edit.py b/src/documents/bulk_edit.py
index 18168a164..c0c80a795 100644
--- a/src/documents/bulk_edit.py
+++ b/src/documents/bulk_edit.py
@@ -85,9 +85,9 @@ def modify_tags(doc_ids, add_tags, remove_tags):
tag_id__in=remove_tags,
).delete()
- DocumentTagRelationship.objects.bulk_create([
- DocumentTagRelationship(
- document_id=doc, tag_id=tag) for (doc,tag) in itertools.product(affected_docs, add_tags)
+ DocumentTagRelationship.objects.bulk_create([DocumentTagRelationship(
+ document_id=doc, tag_id=tag) for (doc, tag) in itertools.product(
+ affected_docs, add_tags)
], ignore_conflicts=True)
async_task(
diff --git a/src/documents/serialisers.py b/src/documents/serialisers.py
index 1f33b957c..66f5f883f 100644
--- a/src/documents/serialisers.py
+++ b/src/documents/serialisers.py
@@ -230,17 +230,20 @@ class BulkEditSerializer(serializers.Serializer):
if not type(documents) == list:
raise serializers.ValidationError(f"{name} must be a list")
if not all([type(i) == int for i in documents]):
- raise serializers.ValidationError(f"{name} must be a list of integers")
+ raise serializers.ValidationError(
+ f"{name} must be a list of integers")
count = Document.objects.filter(id__in=documents).count()
if not count == len(documents):
raise serializers.ValidationError(
- f"Some documents in {name} don't exist or were specified twice.")
+ f"Some documents in {name} don't exist or were "
+ f"specified twice.")
def _validate_tag_id_list(self, tags, name="tags"):
if not type(tags) == list:
raise serializers.ValidationError(f"{name} must be a list")
if not all([type(i) == int for i in tags]):
- raise serializers.ValidationError(f"{name} must be a list of integers")
+ raise serializers.ValidationError(
+ f"{name} must be a list of integers")
count = Tag.objects.filter(id__in=tags).count()
if not count == len(tags):
raise serializers.ValidationError(
@@ -310,7 +313,8 @@ class BulkEditSerializer(serializers.Serializer):
raise serializers.ValidationError("add_tags not specified")
if "remove_tags" in parameters:
- self._validate_tag_id_list(parameters['remove_tags'], "remove_tags")
+ self._validate_tag_id_list(parameters['remove_tags'],
+ "remove_tags")
else:
raise serializers.ValidationError("remove_tags not specified")