Požadavek #814
otevřený
Převést zbylé staré typy komentářů na nový typ
Přidáno uživatelem Ondřej Fibich před více než 10 roky(ů).
Aktualizováno před téměř 10 roky(ů).
Kategorie:
Členi a uživatelé
Popis
Komentáře u člena, zařízení. ... stále používají starý typ (= jedna textarea).
Cílem tohoto požadavku je předělat tyto zbylé staré typy komentářů na vláknové komentáře (např. u fin. stavu člena).
Zahrnuje generickou implementaci operací (přidání, editaci, smazání, ...) v controlleru Comments pro komentáře libovolného objektu. URL controlleru bude ve formátu /comments/<komentovaný objekt>/<id komentovaneho objektu/id existujiciho komentare>/<operace>. Při generické implementaci lze ignorovat drobečkovou navigaci, jelikož se komentáře spravují v dialozích (fallback mode bez JS bych ignoroval).
- Předmět změněn z Převést zbylé staré typy formulářů na nový typ na Převést zbylé staré typy komentářů na nový typ
- Cílová verze změněn z 1.1 na 1.2
- Kategorie nastaven na Členi a uživatelé
- Přiřazeno nastaven na David Raška
Momentálně mám nachystané nové typy komentářů u člena včetně DB upgradu pro převedení starých typů na nové.
Dál jich přidám k uživatelovi a zařízením.
Ovšem otázka je jestli má smysl dělat nové typy například u VLANu, přerušení členství, bílé listiny, tarifu člena, linky, rozhraní, a nevim kde eště.
Zas tak generické to asi nebude, jelikož u každého objektu je potřeba navíc kontrolovat například přístupové práva, povolené moduly atd. U komentářů k požadavkům a pracím se navíc posílají maily. Takže ve výsledku bude stejně potřeba v metodách mět kód rozdělený pro jednotlivé objekty.
- Stav změněn z Nový na Čeká se
OK, to se musí ještě vyřešit. Napadá mě použít konfigurační pole spolu s anonymními funkcemi, něco jako:
// ...
private static $configuration = array(
'log_queue' => array(
'access' => function () {
return Controller::instance()->acl_check_delete('Log_queues_Controller', 'comments')
},
'afterAddCallback' => function ($comment) { /* ... */ },
'afterEditCallback' => function ($comment) { /* ... */ },
'afterDeleteCallback' => function ($comment) { /* ... */ },
)
// next config records...
)
// ...
Je to novinka PHP 5.3, ale to nám nevadí, jelikož stejně musíme přejít na 5.3 (#928).
Důvodem proč jsem to dal do stavu "čeká se" je to, že si nejsem jistý zda anonymní fnce používat vedou totiž k prasení.. Co myslíš Michale?
Také k dispozici: Atom
PDF
Aktualizováno uživatelem 
Aktualizováno uživatelem