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

Fix ošetření prázdného seznamu a chybného logování registrace

Browse source 
= pád webu          = nemožnost registrace
This commit is contained in:
Jonas Havelka 2020-09-03 16:07:52 +02:00
parent 6477735914
commit 80d608eb53
1 changed files with 7 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
seminar/views/views_all.py
View file

@ -992,6 +992,8 @@ def soustredeniUcastniciExportView(request,soustredeni):
def group_by_rocnik(clanky): def group_by_rocnik(clanky):
''' Vezme zadaný seznam článků a seskupí je podle ročníku. ''' Vezme zadaný seznam článků a seskupí je podle ročníku.
Vrátí seznam seznamů článků ze stejného ročníku.''' Vrátí seznam seznamů článků ze stejného ročníku.'''
if len(clanky) == 0:
return clanky
clanky.order_by('cislo__rocnik__rocnik') clanky.order_by('cislo__rocnik__rocnik')
skupiny_clanku = [] skupiny_clanku = []
skupina = [] skupina = []
@ -1131,7 +1133,7 @@ def logoutView(request):
def prihlaska_log_gdpr_safe(logger, gdpr_logger, msg, form_data): def prihlaska_log_gdpr_safe(logger, gdpr_logger, msg, form_data):
msg = "{}, form_hash:{}".format(msg,hash(form_data)) msg = "{}, form_hash:{}".format(msg,hash(frozenset(form_data.items)))
logger.warn(msg) logger.warn(msg)
gdpr_logger.warn(msg+", form:{}".format(form_data)) gdpr_logger.warn(msg+", form:{}".format(form_data))
@ -1199,8 +1201,8 @@ def prihlaskaView(request):
if form.is_valid(): if form.is_valid():
generic_logger.info("Form valid") generic_logger.info("Form valid")
fcd = form.cleaned_data fcd = form.cleaned_data
form_hash = hash(fcd) form_hash = hash(frozenset(fcd.items()))
form_logger.info(fcd,form_hash=form_hash) form_logger.info(fcd,form_hash) # TODO takhle log nefunguje, ale ta předchozí varianta dokonce padala
with transaction.atomic(): with transaction.atomic():
u = User.objects.create_user( u = User.objects.create_user(
@ -1230,7 +1232,7 @@ def prihlaskaView(request):
else: else:
# Unknown country - log it # Unknown country - log it
msg = "Unknown country {}".format(fcd['stat_text']) msg = "Unknown country {}".format(fcd['stat_text'])
err_logger.warn(msg,form_hash=form_hash) err_logger.warn(msg,form_hash) # TODO viz výše
o.save() o.save()
o.user = u o.user = u
@ -1248,7 +1250,7 @@ def prihlaskaView(request):
else: else:
# Unknown school - log it # Unknown school - log it
msg = "Unknown school {}, {}".format(fcd['skola_nazev'],fcd['skola_adresa']) msg = "Unknown school {}, {}".format(fcd['skola_nazev'],fcd['skola_adresa'])
err_logger.warn(msg,form_hash=form_hash) err_logger.warn(msg,form_hash) # TODO viz výše
r.save() r.save()