csphere-cms / cSphere

Generic PHP Web-CMS with low footprint and high extensibility
Other
17 stars 7 forks source link

Global Notification API #11

Open ebroda opened 10 years ago

ebroda commented 10 years ago

Possibility to send notifications to users on new events (like facebook, on new comments, likes, etc.)

micha-09 commented 10 years ago

evtl sogar was für den Core, mal mit Hajo reden

hajo-p commented 10 years ago

wollt ihr nur meldungen ausgeben dabei oder notifications als eigenes plugin bzw. eigene liste in der mehrere einträge sein können? für letzeres wäre ich gegen eine core integration.

micha-09 commented 10 years ago

Ich sehe Notofications als einmalige Nachricht die ausgegeben wird... und nicht gespeichert wird

ebroda commented 10 years ago

Also ich würde zumindest eine API (bzw. Interface) für Notifications bauen und was da letztendlich hinter implementiert ist: speichern und anzeigen, nur anzeigen, Emails verschicken, ... das soll dann ein Plugin klären.

Aber das wir da einheitliche Aufrufe haben, die ggf. halt in "none" landen, wenn wir kein passendes Plugin haben.

Ich denke da an so eine Funktion wie notify($userID, $plugin, $message, $userIDCauser = "0", $pluginID = "", $pluginAction = "list", $pluginActionAnchor = "", $adminAction = false) mit der ich z.B. bei einer neuen Antwort von einem anderen Nutzer ($userID_causer) in einem Forumthema ($plugin), denjenigen Nutzer ($userID) benachrichten kann, der das Thema erstellt hat. Dazu schreib ich dann eine Nachricht $message und auch direkt auf die Nachricht zugreifen kann.

Zusätzlich sollte noch zwischen Admin und User-Benachrichtigungen utnerschieden werden. Adminbenachrichtigungen sind z.B. eine neue Kontaktanfrage.

Die Methode sollte dann auch noch berücksichtigen, ob ich überhaupt in meinen Einstellungen Notifications aktiviert habe, mehr aber momentan erstmal nicht ^^

micha-09 commented 10 years ago

Evtl. Stichwort Observer Pattern...

ebroda commented 10 years ago

geht in die Richtung ... ich hab's oben mal noch ein bisschen ausgebaut