mam/mamweb
11
0
Fork 0
Code Issues Pull requests 8 Projects Wiki Activity Actions

verejny kontaktnicek #71

Merged
ticvac merged 10 commits from kontaktnicek_pro_vsecny into master 2024-11-12 21:15:25 +01:00
Conversation 1 Commits 10 Files changed 7 +5 -2
1 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit ad5a242f8d - Show all commits

7
soustredeni/models.py
View file

@ -18,8 +18,11 @@ from django.utils import timezone
def generate_filename_kontaktnicek(self, filename): def generate_filename_kontaktnicek(self, filename):
# generate random string # generate random string
length = 32 length = 32
file_type = filename.split('.')[-1] if len(filename.split('.')) == 1:
fname = "kontaktnicky/" + timezone.now().strftime('%Y-%m-%d-%H_%M') + "" file_type = 'lol_neumíš_tam_dát_příponu'
ledoian commented 2024-11-12 19:24:32 +01:00
Outdated
Review
Copy link

Tohle nezachvá příponu, což je spíš bug – když se dívám na soubory na Gimlim, tak chci vědět, co jsou zač.

Tohle nezachvá příponu, což je spíš bug – když se dívám na soubory na Gimlim, tak chci vědět, co jsou zač.
👍 1
else:
ledoian commented 2024-11-12 19:22:57 +01:00
Outdated
Review
Copy link

Prosím ne KOREKTURY_PDF_DIR!!

Prosím ne `KOREKTURY_PDF_DIR`!!
👍 1
ledoian commented 2024-11-12 19:25:08 +01:00
Outdated
Review
Copy link

Also: možná je snazší a bezpečnější použít pathlib než os.path.join…

Also: možná je snazší a bezpečnější použít `pathlib` než `os.path.join`…
👎 1
file_type = filename.split('.')[-1]
fname = timezone.now().strftime('%Y-%m-%d-%H_%M') + ""
fname += ''.join(secrets.choice(string.ascii_uppercase + string.digits) for _ in range(length)) fname += ''.join(secrets.choice(string.ascii_uppercase + string.digits) for _ in range(length))
fname += '.' + file_type fname += '.' + file_type
return os.path.join(settings.SOUSTREDENI_KONTAKTNICKY_DIR, fname) return os.path.join(settings.SOUSTREDENI_KONTAKTNICKY_DIR, fname)