<?php
namespace App\Event;
use Cake\Event\EventListenerInterface;
use Notifier\Utility\NotificationManager;
class FeedbackListener implements EventListenerInterface {
public function implementedEvents() {
return [
'Model.afterSave' => 'addFeedbackNotification',
];
}
public function addFeedbackNotification($event, $entity) {
if ($entity->source() == 'AkkaFeedback.Feedbacks' && $entity->isNew()) {
// Add notification
$notificationManager = NotificationManager::instance();
$notificationManager->addTemplate('newMessage', [
'title' => 'New Message Notification',
'body' => 'New message from :name'
]);
$notificationManager->addRecipientList('administrators', [7]);
$notificationManager->notify([
'recipientLists' => ['administrators'],
'template' => 'newMessage',
'vars' => [
'id' => $entity->get('id'),
'name' => $entity->get('name'),
'feedback' => $entity->get('feedback')
]
]);
}
}
}
Plus I added this to
bootstrap.php
/**
* Event listeners
*/
use App\Event\FeedbackListener;
use Cake\Event\EventManager;
$feedackListener = new FeedbackListener();
EventManager::instance()->attach($feedackListener);
Yeah, in fact this should work. However, using this code will be very expensive, because you listen to the event
Model.afterSave. So, on every model's safe you'll fire this event (thats why you used if/else). So to improve your code I would suggest creating a new event into your afterSave-method in your model (in the plugin), and listen on that event. The bootstrap looks good!
I will definitely add
afterSave to my plugin to make it easy to track its save events. The reason I went this route is because I use a plugin from another developer that does not fire any specific events and I wanted a notification on new creation as well. But all good for now, thanks for the help!
Oh, to improve your code, I would move
$notificationManager->addTemplate('newMessage', [
'title' => 'New Message Notification',
'body' => 'New message from :name'
]);to your bootstrap.php. Else the templates can't be read when showing them...
Just to add, everything else though is very clear on your doc and easy to follow..
How to install notifier without using composer
Am using an existing project.
But I strongly recommend using composer
what is the best way to check notification every second ?
@designskiller, @bobmulder is correct as it can be heavy on the server. I had the same need to check quickly for new notifications at page load. The way that it works for me is that I start checking immediately at page load, but my jquery script increments the call by 1 second after each call. Such that I get every second, every two seconds, every three seconds, etc. You can even increment by the X seconds you desire or double the interval. Like every second, two seconds, four seconds, eight seconds.
Also for me, I set max increment at 5 minutes and the process starts all over again. Just in case a user leaves the page open and walks away.
Optionally you can even stop the calls at that time
Another solution could be: https://pusher.com/...
@akkaweb if you are going to share some code that would be great help.
@designskiller the following is a jQuery example of how to increment intervals, not specifically an example of how to display your notifications. As a matter of fact, my setup was to check for something else.. not notifications. Here is my jQuery code
jQuery.noConflict();
var interval = 1000;
setTimeout(checkStatus, interval);
function checkStatus(){
jQuery.ajax({
method: "GET",
url: '/dashboard/notifications/new'
}).done(function(data){
notifications = jQuery.parseJSON(data);
notificationsUpdate(notifications);
});
// ie. every 1, 2, 3, 4, 5, 6, 7 secs, etc
interval += 1000;
// ie. double - every 1, 2, 4, 8, 16, 32 secs, etc.
// interval = interval * 2;
setTimeout(checkStatus, interval);
}
function notificationsUpdate(notifications){
// Code to handle notifications display
}