search

fmfi-svt / anketa

Študentská anketa FMFI a iných fakúlt UK
https://anketa.uniba.sk/
Other
0 stars 1 forks source link

Default hideAllResults #213

Closed mrshu closed 8 years ago

mrshu commented 8 years ago

Aktualne ma hideAllResults "soft" default na false(https://github.com/fmfi-svt/anketa/blob/unreviewed/src/AnketaBundle/Entity/User.php#L65). Toto v aktualnom stave branch unreviewed znamena, ze novovytvoreny uzivatel vsetko vidi (a vsetci vidia jeho).

Spravne ma byt default true, teda ze novy uzivatel nevidi nic (a nikto nevidi jeho), ak nieje student alebo admin.

Zmena sa zda byt trivialna, kazdopadne moze mat dalekosiahle dosledky, ktore treba poriadne preskumat (hlavne vzhladom na potrebu migracie schemy).

mrshu commented 8 years ago

Ukazuje sa tiez, ze oproti verzii, ktora bola deploynuta v zime by doctrine chcelo spravit nasledujucu zmenu:

ALTER TABLE User CHANGE hideAllResults hideAllResults TINYINT(1) NOT NULL

Bolo by vhodne zistit, ci ju je vobec treba, a tiez, ci predchadzajuci stav (pri ktorom bol default 1) nebol postacujuci.