Project

General

Profile

Požadavek #1017

Nový systém upozorňování

Added by David Raška almost 5 years ago. Updated almost 5 years ago.

Status:
Nový
Priority:
Normální
Assignee:
Category:
Upozorňování
Target version:
-
Start date:
08/26/2015
Due date:
% Done:

0%

Estimated time:

Description

Současný stav systému upozorňování je neudržitelný pro budoucí změny. Obsahuje hromadu duplikujícího se kódu (send_email a send_emails, send_sms_message a send_sms_messages) a množství různě zanořených metod, přes které se upozornění aktivují.

Například implementovat historii aktivovaných upozornění (#884) za současného stavu systému by bylo téměř nemožné.

Je nutné celkově zjednodušit a zpřehlednit způsob aktivace.


Related issues

Related to Požadavek #884: [MODUL] Historie interakcí s členem/zákazníkemNový02/14/2014David Raška

Actions
Related to Požadavek #987: Upozornění na nové členy daného typuNový04/24/2015

Actions
#1

Updated by David Raška almost 5 years ago

  • Related to Požadavek #884: [MODUL] Historie interakcí s členem/zákazníkem added
#2

Updated by David Raška almost 5 years ago

  • Related to Požadavek #987: Upozornění na nové členy daného typu added
#3

Updated by Ondřej Fibich almost 5 years ago

Ano, je to kvůli tomu, že se volají různé statické metody z bůhví kama.
Pro aktivaci upozornění by se měla vytvořit servisní třída.
Zkus navrhnout její PHP rozhraní.

Zároveň buď prosím více specifický, s čím si měl problém atp.

Pokud se bude předělávat rozhraní je potřeba nadefinovat cíle, analyzovat problémy stávajícího stavu, navrhnout řešení (minimálně návrh API) a odhadnout dopady na systém (co se bude muset modifikovat a v jaké míře).

#4

Updated by David Raška almost 5 years ago

Ondřej Fibich napsal:

Zároveň buď prosím více specifický, s čím si měl problém atp.

Problém je například v tom, že aktivace zprávy (sms, e-mail, přesměrování) pro jednoho člena je potřeba uložit jako jeden záznam (aktivovaly se najednou), tudíž se musí ukládat v místě volání Notifications_Controller::notify(). Výsledný text, který se posílá v SMS, nebo E-mailem se ale tvoří až v metodách Message_Model::send_emails() a Message_Model::send_sms_messages(), což je v call stacku o 2 úrovně níž, než se musí ukládat historie. Navíc může mít některý člen bílou listinu, která se ale kontroluje v metodě notify, takže při ukládání historie vůbec nepoznám jestli členovi byla zpráva vůbec aktivována.

#5

Updated by Ondřej Fibich almost 5 years ago

David Raška napsal:

Ondřej Fibich napsal:

Zároveň buď prosím více specifický, s čím si měl problém atp.

Problém je například v tom, že aktivace zprávy (sms, e-mail, přesměrování) pro jednoho člena je potřeba uložit jako jeden záznam (aktivovaly se najednou), tudíž se musí ukládat v místě volání Notifications_Controller::notify(). Výsledný text, který se posílá v SMS, nebo E-mailem se ale tvoří až v metodách Message_Model::send_emails() a Message_Model::send_sms_messages(), což je v call stacku o 2 úrovně níž, než se musí ukládat historie. Navíc může mít některý člen bílou listinu, která se ale kontroluje v metodě notify, takže při ukládání historie vůbec nepoznám jestli členovi byla zpráva vůbec aktivována.

A které informace se budou v rámci #884 evidovat? Chtělo by ten požadavek rozšířit o výčet evidovaných prvků.

#6

Updated by Ondřej Fibich almost 5 years ago

Když kliknu na checkbox 2x, tak už nemůžu psát do polí..

#7

Updated by David Raška almost 5 years ago

Ondřej Fibich napsal:

Když kliknu na checkbox 2x, tak už nemůžu psát do polí..

Co? Jaky checkbox?

#9

Updated by David Raška almost 5 years ago

  • Assignee set to David Raška

Also available in: Atom PDF