mam/mamweb
13
1
Fork 0
Code Issues Pull requests 12 Projects Wiki Activity Actions

Upgrade odevzdavatka #30

Merged
zelvuska merged 37 commits from upgrade_odevzdavatka into master 2023-06-19 20:48:30 +02:00
Conversation 3 Commits 37 Files changed 15 +2 -2
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit baaaa0829a - Show all commits

4
seminar/views/views_all.py
View file

@ -35,7 +35,7 @@ from django.conf import settings
import unicodedata import unicodedata
import logging import logging
import time import time
from collections.abc import Iterable from collections.abc import Sequence
from seminar.utils import aktivniResitele from seminar.utils import aktivniResitele
@ -678,7 +678,7 @@ def StavDatabazeView(request):
# Interní, nemá se nikdy objevit v urls (jinak to účastníci vytrolí) # Interní, nemá se nikdy objevit v urls (jinak to účastníci vytrolí)
def formularOKView(request, text='', dalsi_odkazy: Iterable[tuple[str, str]] = ()): def formularOKView(request, text='', dalsi_odkazy: Sequence[tuple[str, str]] = ()):
zelvuska marked this conversation as resolved Outdated
ledoian commented 2023-06-19 19:38:07 +02:00
Outdated
Review
Copy link

V zájmu bezpečnosti bych možná spíš než Iterable použil Sequence, tj. věc, kde mají věci dané pořadí (množina a slovník jsou Iterable, ale ne Sequence; asi chceme, aby to pořadí, co tady napíšeme, se dodrželo.)

Ne že by někdo aktuálně moc kontroloval typy, ale třeba jednou bude :-D

V zájmu bezpečnosti bych možná spíš než `Iterable` použil `Sequence`, tj. věc, kde mají věci dané pořadí (množina a slovník jsou Iterable, ale ne Sequence; asi chceme, aby to pořadí, co tady napíšeme, se dodrželo.) Ne že by někdo aktuálně moc kontroloval typy, ale třeba jednou bude :-D
template_name = 'seminar/formular_ok.html' template_name = 'seminar/formular_ok.html'
odkazy = list(dalsi_odkazy) + [ odkazy = list(dalsi_odkazy) + [
# (Text, odkaz) # (Text, odkaz)