Compare commits

..

2 commits

Author SHA1 Message Date
Bastien Roy
8b09b91e4f Merge branch 'feature/166/rendre-telephone-non-obligatoire' into 'develop'
[166] Rendre le telephone non obligatoire

Closes #166

See merge request la-chariotte/la-chariotte!108
2025-01-05 13:57:01 +00:00
xmeunier
18249653f9 Prevent conflicting usage of comma in CSV files 2025-01-05 13:46:40 +01:00
2 changed files with 3 additions and 3 deletions

View file

@ -344,7 +344,7 @@ class ExportGroupOrderEmailAdressesToDownloadView(UserPassesTestMixin, generic.V
response = http.HttpResponse(content_type="text/csv")
filename = f"commande _{grouped_order.name.replace(' ', '_')}"
response["Content-Disposition"] = f'attachment; filename="{filename}.csv"'
writer = csv.writer(response)
writer = csv.writer(response, delimiter=";")
for email in email_list:
writer.writerow([email])
return response
@ -364,7 +364,7 @@ class ExportGroupedOrderToCSVView(GroupedOrderExportView):
"Content-Disposition": f'attachment; filename="{context["object"].name}-commandes"'
},
)
writer = csv.writer(response)
writer = csv.writer(response, delimiter=";")
# write headers rows
row = ["", ""]

View file

@ -1659,7 +1659,7 @@ class TestExportGroupedOrderToCSVView:
assert response.status_code == 200
content = response.content.decode("utf-8")
csv_reader = csv.reader(StringIO(content))
csv_reader = csv.reader(StringIO(content), delimiter=";")
body = list(csv_reader)
created_date = f"{timezone.now().strftime('%d/%m/%Y')}"
created_time = f"{timezone.now().strftime('%H:%M')}"