satano
commented
8 years ago Tak ja som toto mrkol. Teda len zdrojáky zhora dolu. neskúšal som si v browseri či to funguje v praxi, ale to ti verím. :) Opravil som nejakú gramatiku v komentároch, ale aj v zdrojákoch. Slovo equipment je nepočítateľné a preto nemá množné číslo. Teda slovíčko equipments neexistuje a tak som to všade opravil. Zmenili sa názvy tried, metód aj vlastností.
Ja som to už kdesi myslím písal (alebo niečo veľmi podobné): zišlo by sa zjednotiť názvy metód v IRepository. Sú tam metódy GetItem, ale ostatné nemajú prídavok Item (Edit, Delete...). Ale toto by rozbilo Majove veci, tak by som to dal ako samostatnú issue, keď toto fšetko bude v master-i.
Inak za mňa to môže ísť do mastera.
Zapracovanie pripomienok z pull requestu #71
project.lock.json odstránený
EquipmentDummyRepository Zjednotil som to a presunul som ho do hlavného projektu.
Id v RoomEquipment Na to aby som mohol mať model RoomEquipment bez Id, som musel trošku prepracovať metódu Edit v RoomRepository. V podstate som ju zapracoval tak, ako sme o tom diskutovali u Maja v Issue #57. S tým súvisí aj to, že som zapracoval už spomínanú požiadavku, aby naše repository vracali odpojené entity.