extern/egroupware_official
1
0
Fork 0
mirror of https://github.com/EGroupware/egroupware.git synced 2024-11-15 12:34:33 +01:00
Code Issues Packages Projects Releases Wiki Activity

- Add a parameter to pass in the class of the notification to use so we can pass in a test one

Browse Source 
- Explicitly tell notification what type of message (plain or html) we're giving it
This commit is contained in:
nathangray 2017-10-21 10:40:43 +02:00
parent e43a0762fb
commit d3c596a443
1 changed files with 18 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
api/src/Storage/Tracking.php
View File

@ -129,6 +129,15 @@ abstract class Tracking
*/
var $notify_current_user = false;
/**
* Class to use for generating the notifications.
* Normally, just the notification class but for testing we pass in a mocked
* class
*
* @var notification_class
*/
protected $notification_class = notifications::class;
/**
* Array with error-messages if track($data,$old) returns false
*
@ -190,7 +199,7 @@ abstract class Tracking
* @param string $cf_app = null if set, custom field names get added to $field2history
* @return bo_tracking
*/
function __construct($cf_app = null)
function __construct($cf_app = null, $notification_class=false)
{
if ($cf_app)
{
@ -205,6 +214,10 @@ abstract class Tracking
}
}
}
if($notification_class)
{
$this->notification_class = $notification_class;
}
}
/**
@ -766,10 +779,11 @@ abstract class Tracking
{
// send via notification_app
try {
$notification = new notifications();
$class = $this->notification_class;
$notification = new $class();
$notification->set_receivers(array($receiver));
$notification->set_message($body_cache['text']);
$notification->set_message($body_cache['html']);
$notification->set_message($body_cache['text'], 'plain');
$notification->set_message($body_cache['html'], 'html');
$notification->set_sender($sender);
$notification->set_subject($subject);
$notification->set_links(array($link));