| Re: Help with notification needed |
| by goffy on 2019/1/28 20:07:20 Hi Michael see my comments at https://github.com/ggoffy/wggallery/pull/7 |
| Re: Help with notification needed |
| by Mamba on 2019/1/28 11:30:54 OK, it works for me: 1) the delete was because the Default Notification Mode was set to: "Notify me only once" But it didn't have any impact in. 2) The issue was that your file in /include was: notifications.inc.php instead of notification.inc.php In English there were no templates in: /language/english/mail_template/ In German, you've missed the template "global_alb_modify_notify.tpl" Now the notifications work for me without any problems. |
| Re: Help with notification needed |
| by Mage on 2019/1/28 9:48:00 I will look at why it does not work but it will take time. The last time I worked on notifications, it was for tdmdownlods ... It's been a long time ... |
| Re: Help with notification needed |
| by Mamba on 2019/1/28 9:26:19 I just ran some tests, and two things: 1) it looks like in file /albums.php for "globals" in triggerEvent() the item_id should be 0, i.e. le="color: #000000"><?php if ( $albNew ) { $notificationHandler = xoops_getHandler('notification'); // $notificationHandler->triggerEvent('global', $newAlbId, 'album_new'); $notificationHandler->triggerEvent('global', 0, 'album_new'); } else { $notificationHandler = xoops_getHandler('notification'); // $notificationHandler->triggerEvent('global', $albId, 'album_modify'); $notificationHandler->triggerEvent('global', 0, 'album_modify'); } 2) Somehow it seems that after each run (i.e. new album or modify album) the notification is being deleted, so the user never gets it, but I didn't figure yet out, why is it being deleted and where - I just saw in the database. I might not have the time now to finish it, but hopefully this will help you to find the problem. |
| Re: Help with notification needed |
| by Mage on 2019/1/28 7:17:29 I work on notifications on my modules. If you want, I can integrate notifications to your module. I do not know if I have the time today but tomorrow should be fine. |