fixed bug in user activation

corso-python-prato / share-system-team2

Apache License 2.0

2 stars 9 forks source link

fixed bug in user activation #44

Closed generale84 closed 10 years ago

generale84 commented 10 years ago

Veniva fatta la pulizia dei pending user prima di prenderne i dati.

Bene io vado in ferie :D :D : D

Ci si vede lunedi prox ;)

coveralls commented 10 years ago

Coverage remained the same when pulling 840bab2ad498a8d11837b957602eace82b9ddfe8 on generale84:master into bef95a5de570317d694ff5a1930b3a1698aaed36 on corso-python-prato:master.

giornaledisistema commented 10 years ago

Come anticipava Iacopo, in effetti il problema era nel metodo di pulizia Users._clean_pending_users(). Il confronto nel test

if now_timestamp() - data['timestamp'] > USER_ACTIVATION_TIMEOUT]

era fatto fra tempi diversi: il tempo di USER_ACTIVATION_TIMEOUT era in secondi, quello di now_timestamp() e data['timestamp'] era il tempo in secondi moltiplicato per 10000 per eliminare la parte decimale.

Ho sistemato nella pull request #47 moltiplicando anche USER_ACTIVATION_TIMEOUT per 10000.

GendoIkari commented 10 years ago

Da verificare se questa pull request diventa effettivamente obsoleta dopo il fix di Paolo. Se e' cosi', qui si chiude.

coveralls commented 10 years ago

Coverage remained the same when pulling f9d589f941c099fd3e2635c85983dbc7c3cee8dc on generale84:master into 6389d11a538c5be95cedcc8fa9d419160f06a83a on corso-python-prato:master.

coveralls commented 10 years ago

Coverage increased (+0.02%) when pulling 36240731921a4928e823a55ea7737bacb854f8ff on generale84:master into deb2b847b580dac735263b4e40018907074cbb63 on corso-python-prato:master.