Zla schema - mame aj User.department aj UserSeason.department

[svtimport]

Bug #175 imported from https://svt-devel.fmph.uniba.sk/chiliproject/issues/175 Author: Tomáš Belan Priority: Normal Target version: Velafakultne hlasovanie 2014/2015 Start date: 10/21/2013

Neviem ci to sposobila nejaka stara migracia alebo co, ale mame aj User.department aj UserSeason.department a zda sa ze su v nich rozne informacie :( mozno sa dokonca pouzivaju obe na roznych miestach? Treba s tym nieco robit.

---

Created by Tomáš Belan on 2013-10-21 at 13:10:40+02:00

• Start date set to 10/21/2013
• Priority set to Normal
• Project set to Anketa
• Subject set to Zla schema - mame aj User.department aj UserSeason.department
• Tracker set to Bug
• Status set to New
• Description set to Neviem ci to sposobila nejaka stara migracia alebo co, ale mame aj User.depar...

[svtimport]

Updated by Tomáš Belan on 2015-03-09 at 14:31:56+01:00

• Target version set to Velafakultne hlasovanie 2014/2015

[svtimport]

Updated by Tomáš Belan on 2015-03-16 at 11:27:17+01:00

Vinko vravi: UserSeason.department treba zrusit. Asociaciu ucitelov a katedier nechceme pre kazdy season zvlast. Nevadi nam, ze sa tym budu menit stare reporty.

Ale samozrejme, treba mat jednak "ktory ucitel patri ku ktorej katedre" a druhak "ktory clovek je sefom danej katedry". Je mozne, ze tie dva stlpce ktore mame teraz su tieto dve veci. Treba to preskumat.

[svtimport]

Updated by Tomáš Belan on 2015-03-17 at 18:49:29+01:00

Co som zistil:

• User.department sa CITA iba vo funkcii getTeachersForDepartment(), ktora sa pouziva pri generovani reportu - ide teda o zoznam ucitelov pracujucich pre danu katedru.
• UserSeason.department sa CITA iba vo funkciach getDepartmentReports() a getUserDepartment(), ktore obe riesia, kto vidi dany report. (Je trochu nestastne ze mame dve.) Navyse musis mat nielen spravnu hodnotu UserSeason.department, ale aj rolu ROLE_DEPARTMENT_REPORT.
• ImportRozvrhXMLCommand ZAPISUJE aj UserSeason.department aj User.department.
• ImportUcitelKatedraCommand ZAPISUJE iba User.department.

Co chceme spravit:

• Zmazat stlpec UserSeason.department. Zmenit funkcie, co ho pouzivali, aby pouzivali User.department. Tento stlpec bude mat oba vyznamy, a pristup k danym reportom sa bude riadit len cez ROLE_DEPARTMENT_REPORT.
• Este predtym overit vo vsetkych ostrych databazach, ze ten stlpec mozme mazat - teda pre kazdy UserSeason s neNULLovym department_id plati, ze dany User ma neNULLove rovnake department_id. (Co tiez implikuje, ze ziaden User nema viacere UserSeasony s roznymi department_id.) Ak tato podmienka neplati vo vseobecnosti, tak je to trochu podozrive, ale hlavne treba aby platila pre ludi s ROLE_DEPARTMENT_REPORT.