2012-04-27 15:14:24 +00:00
|
|
|
<?php
|
2015-08-10 18:27:52 +00:00
|
|
|
|
|
|
|
use MediaWiki\Logger\LoggerFactory;
|
2015-10-01 13:48:52 +00:00
|
|
|
|
2013-06-12 23:18:26 +00:00
|
|
|
/**
|
2014-08-05 00:15:14 +00:00
|
|
|
* This class represents the controller for notifications
|
2013-06-12 23:18:26 +00:00
|
|
|
*/
|
2012-04-27 15:14:24 +00:00
|
|
|
class EchoNotificationController {
|
2014-08-05 00:15:14 +00:00
|
|
|
|
|
|
|
/**
|
|
|
|
* Echo event agent per wiki blacklist
|
|
|
|
*
|
|
|
|
* @var string[]
|
|
|
|
*/
|
2013-05-06 22:34:50 +00:00
|
|
|
static protected $blacklist;
|
2014-08-05 00:15:14 +00:00
|
|
|
|
|
|
|
/**
|
|
|
|
* Echo event agent per user whitelist, this overwrites $blacklist
|
|
|
|
*
|
|
|
|
* @param string[]
|
|
|
|
*/
|
2013-05-06 22:34:50 +00:00
|
|
|
static protected $userWhitelist;
|
|
|
|
|
2012-11-13 23:06:11 +00:00
|
|
|
/**
|
2016-03-09 04:50:31 +00:00
|
|
|
* Returns the count passed in, or MWEchoNotifUser::MAX_BADGE_COUNT + 1,
|
|
|
|
* whichever is less.
|
2014-08-05 00:15:14 +00:00
|
|
|
*
|
2014-08-26 18:58:34 +00:00
|
|
|
* @param int $count
|
2016-03-09 04:50:31 +00:00
|
|
|
* @return int Notification count, with ceiling applied
|
2012-11-13 23:06:11 +00:00
|
|
|
*/
|
2016-03-09 04:50:31 +00:00
|
|
|
public static function getCappedNotificationCount( $count ) {
|
|
|
|
if ( $count <= MWEchoNotifUser::MAX_BADGE_COUNT ) {
|
|
|
|
return $count;
|
2012-11-13 23:06:11 +00:00
|
|
|
} else {
|
2016-03-09 04:50:31 +00:00
|
|
|
return MWEchoNotifUser::MAX_BADGE_COUNT + 1;
|
2012-11-13 23:06:11 +00:00
|
|
|
}
|
2016-03-09 04:50:31 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Format the notification count as a string. This should only be used for an
|
|
|
|
* isolated string count, e.g. as displayed in personal tools or returned by the API.
|
|
|
|
*
|
|
|
|
* If using it in sentence context, pass the value from getCappedNotificationCount
|
|
|
|
* into a message and use PLURAL. Example: notification-bundle-header-page-linked
|
|
|
|
*
|
|
|
|
* @param int count Notification count
|
|
|
|
* @return string Formatted count, after applying cap then formatting to string
|
|
|
|
*/
|
|
|
|
public static function formatNotificationCount( $count ) {
|
|
|
|
$cappedCount = self::getCappedNotificationCount( $count );
|
2012-11-13 23:06:11 +00:00
|
|
|
|
2016-03-09 04:50:31 +00:00
|
|
|
return wfMessage( 'echo-badge-count' )->numParams( $cappedCount )->text();
|
2012-11-13 23:06:11 +00:00
|
|
|
}
|
|
|
|
|
2012-04-27 15:14:24 +00:00
|
|
|
/**
|
|
|
|
* Processes notifications for a newly-created EchoEvent
|
2012-08-31 21:50:46 +00:00
|
|
|
*
|
2014-08-05 00:15:14 +00:00
|
|
|
* @param EchoEvent $event
|
|
|
|
* @param boolean $defer Defer to job queue or not
|
2012-04-27 15:14:24 +00:00
|
|
|
*/
|
2012-06-06 07:04:28 +00:00
|
|
|
public static function notify( $event, $defer = true ) {
|
2014-08-27 18:39:53 +00:00
|
|
|
// Defer to job queue if defer to job queue is requested and
|
|
|
|
// this event should use job queue
|
|
|
|
if ( $defer && $event->getUseJobQueue() ) {
|
2013-08-30 02:05:29 +00:00
|
|
|
// defer job insertion till end of request when all primary db transactions
|
|
|
|
// have been committed
|
2015-10-01 13:48:52 +00:00
|
|
|
DeferredUpdates::addCallableUpdate( function () use ( $event ) {
|
2014-08-02 06:52:16 +00:00
|
|
|
// can't use self::, php 5.3 doesn't inherit class scope
|
2014-08-02 06:41:19 +00:00
|
|
|
EchoNotificationController::enqueueEvent( $event );
|
|
|
|
} );
|
2015-10-01 13:48:52 +00:00
|
|
|
|
2012-06-06 07:04:28 +00:00
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
2013-03-01 01:09:40 +00:00
|
|
|
// Check if the event object has valid event type. Events with invalid
|
|
|
|
// event types left in the job queue should not be processed
|
2012-11-13 23:06:11 +00:00
|
|
|
if ( !$event->isEnabledEvent() ) {
|
2013-03-01 01:09:40 +00:00
|
|
|
return;
|
2012-11-13 23:06:11 +00:00
|
|
|
}
|
|
|
|
|
2014-08-02 06:52:16 +00:00
|
|
|
$type = $event->getType();
|
|
|
|
$notifyTypes = self::getEventNotifyTypes( $type );
|
2014-09-09 22:11:53 +00:00
|
|
|
$userIds = array();
|
|
|
|
$userIdsCount = 0;
|
2014-08-02 06:52:16 +00:00
|
|
|
foreach ( self::getUsersToNotifyForEvent( $event ) as $user ) {
|
2014-09-09 22:11:53 +00:00
|
|
|
$userIds[$user->getId()] = $user->getId();
|
2014-08-02 06:52:16 +00:00
|
|
|
$userNotifyTypes = $notifyTypes;
|
2015-06-16 00:40:14 +00:00
|
|
|
// Respect the enotifminoredits preference
|
|
|
|
// @todo should this be checked somewhere else?
|
|
|
|
if ( !$user->getOption( 'enotifminoredits' ) ) {
|
|
|
|
$extra = $event->getExtra();
|
|
|
|
if ( !empty( $extra['revid'] ) ) {
|
|
|
|
$rev = Revision::newFromID( $extra['revid'], Revision::READ_LATEST );
|
|
|
|
|
|
|
|
if ( $rev->isMinor() ) {
|
|
|
|
$notifyTypes = array_diff( $notifyTypes, array( 'email' ) );
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
2015-06-01 18:36:35 +00:00
|
|
|
Hooks::run( 'EchoGetNotificationTypes', array( $user, $event, &$userNotifyTypes ) );
|
2013-03-12 22:52:00 +00:00
|
|
|
|
2014-08-02 06:52:16 +00:00
|
|
|
// types such as web, email, etc
|
|
|
|
foreach ( $userNotifyTypes as $type ) {
|
|
|
|
self::doNotification( $event, $user, $type );
|
|
|
|
}
|
2014-09-09 22:11:53 +00:00
|
|
|
|
|
|
|
$userIdsCount++;
|
|
|
|
// Process 1000 users per NotificationDeleteJob
|
|
|
|
if ( $userIdsCount > 1000 ) {
|
|
|
|
self::enqueueDeleteJob( $userIds, $event );
|
|
|
|
$userIds = array();
|
|
|
|
$userIdsCount = 0;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// process the userIds left in the array
|
|
|
|
if ( $userIds ) {
|
|
|
|
self::enqueueDeleteJob( $userIds, $event );
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Schedule a job to check and delete older notifications
|
|
|
|
*
|
|
|
|
* @param int $userIds
|
|
|
|
* @param EchoEvent $event
|
|
|
|
*/
|
|
|
|
public static function enqueueDeleteJob( array $userIds, EchoEvent $event ) {
|
|
|
|
// Do nothing if there is no user
|
|
|
|
if ( !$userIds ) {
|
|
|
|
return;
|
2014-08-02 06:52:16 +00:00
|
|
|
}
|
2014-09-09 22:11:53 +00:00
|
|
|
|
|
|
|
$job = new EchoNotificationDeleteJob(
|
|
|
|
$event->getTitle() ?: Title::newMainPage(),
|
|
|
|
array(
|
|
|
|
'userIds' => $userIds
|
|
|
|
)
|
|
|
|
);
|
|
|
|
JobQueueGroup::singleton()->push( $job );
|
2014-08-02 06:52:16 +00:00
|
|
|
}
|
2013-03-12 22:52:00 +00:00
|
|
|
|
2014-08-02 06:52:16 +00:00
|
|
|
/**
|
|
|
|
* @param string $type Event type
|
|
|
|
* @return string[] List of notification types to send for
|
|
|
|
* this event type
|
|
|
|
*/
|
|
|
|
public static function getEventNotifyTypes( $type ) {
|
|
|
|
// Get the notification types for this event, eg, web/email
|
|
|
|
global $wgEchoDefaultNotificationTypes;
|
2012-04-27 15:14:24 +00:00
|
|
|
|
2014-08-02 06:52:16 +00:00
|
|
|
$notifyTypes = $wgEchoDefaultNotificationTypes['all'];
|
|
|
|
if ( isset( $wgEchoDefaultNotificationTypes[$type] ) ) {
|
|
|
|
$notifyTypes = array_merge(
|
|
|
|
$notifyTypes,
|
|
|
|
$wgEchoDefaultNotificationTypes[$type]
|
|
|
|
);
|
2012-04-27 15:14:24 +00:00
|
|
|
}
|
2014-08-02 06:52:16 +00:00
|
|
|
|
|
|
|
return array_keys( array_filter( $notifyTypes ) );
|
2012-04-27 15:14:24 +00:00
|
|
|
}
|
|
|
|
|
2014-08-02 06:41:19 +00:00
|
|
|
/**
|
|
|
|
* Push $event onto the mediawiki job queue
|
|
|
|
*
|
|
|
|
* @param EchoEvent $event
|
|
|
|
*/
|
|
|
|
public static function enqueueEvent( EchoEvent $event ) {
|
|
|
|
$job = new EchoNotificationJob(
|
|
|
|
$event->getTitle() ?: Title::newMainPage(),
|
|
|
|
array(
|
|
|
|
'event' => $event,
|
|
|
|
'masterPos' => MWEchoDbFactory::newFromDefault()
|
|
|
|
->getMasterPosition(),
|
|
|
|
)
|
|
|
|
);
|
|
|
|
JobQueueGroup::singleton()->push( $job );
|
|
|
|
}
|
|
|
|
|
2013-05-06 22:34:50 +00:00
|
|
|
/**
|
|
|
|
* Implements blacklist per active wiki expected to be initialized
|
|
|
|
* from InitializeSettings.php
|
|
|
|
*
|
2014-08-05 00:15:14 +00:00
|
|
|
* @param EchoEvent $event The event to test for exclusion via global blacklist
|
2013-05-06 22:34:50 +00:00
|
|
|
* @return boolean True when the event agent is in the global blacklist
|
|
|
|
*/
|
|
|
|
protected static function isBlacklisted( EchoEvent $event ) {
|
|
|
|
if ( !$event->getAgent() ) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
if ( self::$blacklist === null ) {
|
|
|
|
global $wgEchoAgentBlacklist, $wgEchoOnWikiBlacklist,
|
2015-10-01 13:48:52 +00:00
|
|
|
$wgMemc;
|
2013-05-06 22:34:50 +00:00
|
|
|
|
|
|
|
self::$blacklist = new EchoContainmentSet;
|
|
|
|
self::$blacklist->addArray( $wgEchoAgentBlacklist );
|
|
|
|
if ( $wgEchoOnWikiBlacklist !== null ) {
|
|
|
|
self::$blacklist->addOnWiki(
|
|
|
|
NS_MEDIAWIKI,
|
|
|
|
$wgEchoOnWikiBlacklist,
|
|
|
|
$wgMemc,
|
2015-10-01 13:48:52 +00:00
|
|
|
wfMemcKey( "echo_on_wiki_blacklist" )
|
2013-05-06 22:34:50 +00:00
|
|
|
);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
return self::$blacklist->contains( $event->getAgent()->getName() );
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Implements per-user whitelist sourced from a user wiki page
|
|
|
|
*
|
2014-08-05 00:15:14 +00:00
|
|
|
* @param EchoEvent $event The event to test for inclusion in whitelist
|
|
|
|
* @param User $user The user that owns the whitelist
|
2013-05-06 22:34:50 +00:00
|
|
|
* @return boolean True when the event agent is in the user whitelist
|
|
|
|
*/
|
2014-08-26 18:58:34 +00:00
|
|
|
public static function isWhitelistedByUser( EchoEvent $event, User $user ) {
|
2013-05-06 22:34:50 +00:00
|
|
|
global $wgEchoPerUserWhitelistFormat, $wgMemc;
|
|
|
|
|
|
|
|
if ( $wgEchoPerUserWhitelistFormat === null || !$event->getAgent() ) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
$userId = $user->getID();
|
|
|
|
if ( $userId === 0 ) {
|
|
|
|
return false; // anonymous user
|
|
|
|
}
|
|
|
|
|
|
|
|
if ( !isset( self::$userWhitelist[$userId] ) ) {
|
|
|
|
self::$userWhitelist[$userId] = new EchoContainmentSet;
|
|
|
|
self::$userWhitelist[$userId]->addOnWiki(
|
|
|
|
NS_USER,
|
|
|
|
sprintf( $wgEchoPerUserWhitelistFormat, $user->getName() ),
|
|
|
|
$wgMemc,
|
|
|
|
wfMemcKey( "echo_on_wiki_whitelist_" . $userId )
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
return self::$userWhitelist[$userId]
|
|
|
|
->contains( $event->getAgent()->getName() );
|
|
|
|
}
|
|
|
|
|
2012-04-27 15:14:24 +00:00
|
|
|
/**
|
|
|
|
* Processes a single notification for an EchoEvent
|
|
|
|
*
|
2014-08-05 00:15:14 +00:00
|
|
|
* @param EchoEvent $event
|
|
|
|
* @param User $user The user to be notified.
|
|
|
|
* @param string $type The type of notification delivery to process, e.g. 'email'.
|
2012-05-17 00:29:37 +00:00
|
|
|
* @throws MWException
|
2012-04-27 15:14:24 +00:00
|
|
|
*/
|
|
|
|
public static function doNotification( $event, $user, $type ) {
|
|
|
|
global $wgEchoNotifiers;
|
|
|
|
|
2012-08-31 21:50:46 +00:00
|
|
|
if ( !isset( $wgEchoNotifiers[$type] ) ) {
|
2012-04-27 15:14:24 +00:00
|
|
|
throw new MWException( "Invalid notification type $type" );
|
|
|
|
}
|
|
|
|
|
2015-06-01 23:22:48 +00:00
|
|
|
// Don't send any notifications to anonymous users
|
|
|
|
if ( $user->isAnon() ) {
|
|
|
|
throw new MWException( "Cannot notify anonymous user: {$user->getName()}" );
|
2013-06-08 01:05:35 +00:00
|
|
|
}
|
|
|
|
|
2012-04-27 15:14:24 +00:00
|
|
|
call_user_func_array( $wgEchoNotifiers[$type], array( $user, $event ) );
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
2014-07-30 03:18:48 +00:00
|
|
|
* Returns an array each element of which is the result of a
|
Allow certain users to be excluded
Right now, if certain users should be excluded, that would have
to be part of the user-locators already. This is annoying because
it's hard to write "generic" user locators when you want to exclude
just a couple of people in certain cases.
In Flow, for example, we have user-locators for users watching a
board or topic. We don't want to send the notification to people
that have also been mentioned in that post (they'll get a separate
notification). We could build that exception into those
user-locators, but then we couldn't re-use them in other places...
This basically means we couldn't use EchoUserLocator::locateUsersWatchingTitle,
we would have to roll our own that also excludes mentioned users.
Instead, this lets you add 'user-filters' (that functionality
actually exists already, but is not currently exposed), which
lists users to not send the notification to, even though they could
be in a user-locator.
Bug: T125428
Change-Id: Ifa0e2d3283f57624af4c5ec264f9f66223508e83
2016-02-02 13:16:39 +00:00
|
|
|
* user-locator|user-filters attached to the event type.
|
2012-08-31 21:50:46 +00:00
|
|
|
*
|
2014-07-29 23:54:00 +00:00
|
|
|
* @param EchoEvent $event
|
Allow certain users to be excluded
Right now, if certain users should be excluded, that would have
to be part of the user-locators already. This is annoying because
it's hard to write "generic" user locators when you want to exclude
just a couple of people in certain cases.
In Flow, for example, we have user-locators for users watching a
board or topic. We don't want to send the notification to people
that have also been mentioned in that post (they'll get a separate
notification). We could build that exception into those
user-locators, but then we couldn't re-use them in other places...
This basically means we couldn't use EchoUserLocator::locateUsersWatchingTitle,
we would have to roll our own that also excludes mentioned users.
Instead, this lets you add 'user-filters' (that functionality
actually exists already, but is not currently exposed), which
lists users to not send the notification to, even though they could
be in a user-locator.
Bug: T125428
Change-Id: Ifa0e2d3283f57624af4c5ec264f9f66223508e83
2016-02-02 13:16:39 +00:00
|
|
|
* @param string $locator Either EchoAttributeManager::ATTR_LOCATORS or EchoAttributeManager::ATTR_FILTERS
|
2014-07-30 03:18:48 +00:00
|
|
|
* @return array
|
2012-04-27 15:14:24 +00:00
|
|
|
*/
|
Allow certain users to be excluded
Right now, if certain users should be excluded, that would have
to be part of the user-locators already. This is annoying because
it's hard to write "generic" user locators when you want to exclude
just a couple of people in certain cases.
In Flow, for example, we have user-locators for users watching a
board or topic. We don't want to send the notification to people
that have also been mentioned in that post (they'll get a separate
notification). We could build that exception into those
user-locators, but then we couldn't re-use them in other places...
This basically means we couldn't use EchoUserLocator::locateUsersWatchingTitle,
we would have to roll our own that also excludes mentioned users.
Instead, this lets you add 'user-filters' (that functionality
actually exists already, but is not currently exposed), which
lists users to not send the notification to, even though they could
be in a user-locator.
Bug: T125428
Change-Id: Ifa0e2d3283f57624af4c5ec264f9f66223508e83
2016-02-02 13:16:39 +00:00
|
|
|
public static function evaluateUserCallable( EchoEvent $event, $locator = EchoAttributeManager::ATTR_LOCATORS ) {
|
2014-07-29 23:54:00 +00:00
|
|
|
$attributeManager = EchoAttributeManager::newFromGlobalVars();
|
2014-07-30 03:18:48 +00:00
|
|
|
$type = $event->getType();
|
|
|
|
$result = array();
|
Allow certain users to be excluded
Right now, if certain users should be excluded, that would have
to be part of the user-locators already. This is annoying because
it's hard to write "generic" user locators when you want to exclude
just a couple of people in certain cases.
In Flow, for example, we have user-locators for users watching a
board or topic. We don't want to send the notification to people
that have also been mentioned in that post (they'll get a separate
notification). We could build that exception into those
user-locators, but then we couldn't re-use them in other places...
This basically means we couldn't use EchoUserLocator::locateUsersWatchingTitle,
we would have to roll our own that also excludes mentioned users.
Instead, this lets you add 'user-filters' (that functionality
actually exists already, but is not currently exposed), which
lists users to not send the notification to, even though they could
be in a user-locator.
Bug: T125428
Change-Id: Ifa0e2d3283f57624af4c5ec264f9f66223508e83
2016-02-02 13:16:39 +00:00
|
|
|
foreach ( $attributeManager->getUserCallable( $type, $locator ) as $callable ) {
|
2014-07-31 02:31:47 +00:00
|
|
|
// locator options can be set per-event by using an array with
|
2014-07-30 03:18:48 +00:00
|
|
|
// name as first parameter.
|
2014-07-31 02:31:47 +00:00
|
|
|
if ( is_array( $callable ) ) {
|
|
|
|
$options = $callable;
|
2014-07-30 03:18:48 +00:00
|
|
|
$spliced = array_splice( $options, 0, 1, array( $event ) );
|
|
|
|
$callable = reset( $spliced );
|
2014-07-31 02:31:47 +00:00
|
|
|
} else {
|
2014-07-30 03:18:48 +00:00
|
|
|
$options = array( $event );
|
2014-07-31 02:31:47 +00:00
|
|
|
}
|
2014-07-29 23:54:00 +00:00
|
|
|
if ( is_callable( $callable ) ) {
|
2014-07-30 03:18:48 +00:00
|
|
|
$result[] = call_user_func_array( $callable, $options );
|
2014-07-29 23:54:00 +00:00
|
|
|
} else {
|
Allow certain users to be excluded
Right now, if certain users should be excluded, that would have
to be part of the user-locators already. This is annoying because
it's hard to write "generic" user locators when you want to exclude
just a couple of people in certain cases.
In Flow, for example, we have user-locators for users watching a
board or topic. We don't want to send the notification to people
that have also been mentioned in that post (they'll get a separate
notification). We could build that exception into those
user-locators, but then we couldn't re-use them in other places...
This basically means we couldn't use EchoUserLocator::locateUsersWatchingTitle,
we would have to roll our own that also excludes mentioned users.
Instead, this lets you add 'user-filters' (that functionality
actually exists already, but is not currently exposed), which
lists users to not send the notification to, even though they could
be in a user-locator.
Bug: T125428
Change-Id: Ifa0e2d3283f57624af4c5ec264f9f66223508e83
2016-02-02 13:16:39 +00:00
|
|
|
wfDebugLog( __CLASS__, __FUNCTION__ . ": Invalid $locator returned for $type" );
|
2014-07-29 23:54:00 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2014-07-30 03:18:48 +00:00
|
|
|
return $result;
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Retrieves an array of User objects to be notified for an EchoEvent.
|
|
|
|
*
|
|
|
|
* @param EchoEvent $event
|
|
|
|
* @return Iterator values are User objects
|
|
|
|
*/
|
|
|
|
public static function getUsersToNotifyForEvent( EchoEvent $event ) {
|
|
|
|
$notify = new EchoFilteredSequentialIterator;
|
Allow certain users to be excluded
Right now, if certain users should be excluded, that would have
to be part of the user-locators already. This is annoying because
it's hard to write "generic" user locators when you want to exclude
just a couple of people in certain cases.
In Flow, for example, we have user-locators for users watching a
board or topic. We don't want to send the notification to people
that have also been mentioned in that post (they'll get a separate
notification). We could build that exception into those
user-locators, but then we couldn't re-use them in other places...
This basically means we couldn't use EchoUserLocator::locateUsersWatchingTitle,
we would have to roll our own that also excludes mentioned users.
Instead, this lets you add 'user-filters' (that functionality
actually exists already, but is not currently exposed), which
lists users to not send the notification to, even though they could
be in a user-locator.
Bug: T125428
Change-Id: Ifa0e2d3283f57624af4c5ec264f9f66223508e83
2016-02-02 13:16:39 +00:00
|
|
|
foreach ( self::evaluateUserCallable( $event, EchoAttributeManager::ATTR_LOCATORS ) as $users ) {
|
2014-07-30 03:18:48 +00:00
|
|
|
$notify->add( $users );
|
|
|
|
}
|
|
|
|
|
|
|
|
// Hook for injecting more users.
|
2014-07-29 23:54:00 +00:00
|
|
|
// @deprecated
|
|
|
|
$users = array();
|
2015-06-01 18:36:35 +00:00
|
|
|
Hooks::run( 'EchoGetDefaultNotifiedUsers', array( $event, &$users ) );
|
2014-07-30 03:18:48 +00:00
|
|
|
if ( $users ) {
|
|
|
|
$notify->add( $users );
|
2012-04-27 15:14:24 +00:00
|
|
|
}
|
|
|
|
|
Allow certain users to be excluded
Right now, if certain users should be excluded, that would have
to be part of the user-locators already. This is annoying because
it's hard to write "generic" user locators when you want to exclude
just a couple of people in certain cases.
In Flow, for example, we have user-locators for users watching a
board or topic. We don't want to send the notification to people
that have also been mentioned in that post (they'll get a separate
notification). We could build that exception into those
user-locators, but then we couldn't re-use them in other places...
This basically means we couldn't use EchoUserLocator::locateUsersWatchingTitle,
we would have to roll our own that also excludes mentioned users.
Instead, this lets you add 'user-filters' (that functionality
actually exists already, but is not currently exposed), which
lists users to not send the notification to, even though they could
be in a user-locator.
Bug: T125428
Change-Id: Ifa0e2d3283f57624af4c5ec264f9f66223508e83
2016-02-02 13:16:39 +00:00
|
|
|
// Exclude certain users
|
|
|
|
foreach ( self::evaluateUserCallable( $event, EchoAttributeManager::ATTR_FILTERS ) as $users ) {
|
|
|
|
// the result of the callback can be both an iterator or array
|
|
|
|
$users = is_array( $users ) ? $users : iterator_to_array( $users );
|
2016-02-11 13:46:18 +00:00
|
|
|
$notify->addFilter( function ( User $user ) use ( $users ) {
|
|
|
|
// we need to check if $user is in $users, but they're not
|
|
|
|
// guaranteed to be the same object, so I'll compare ids.
|
|
|
|
$userId = $user->getId();
|
|
|
|
$userIds = array_map( function ( User $user ) {
|
|
|
|
return $user->getId();
|
|
|
|
}, $users );
|
|
|
|
return !in_array( $userId, $userIds );
|
Allow certain users to be excluded
Right now, if certain users should be excluded, that would have
to be part of the user-locators already. This is annoying because
it's hard to write "generic" user locators when you want to exclude
just a couple of people in certain cases.
In Flow, for example, we have user-locators for users watching a
board or topic. We don't want to send the notification to people
that have also been mentioned in that post (they'll get a separate
notification). We could build that exception into those
user-locators, but then we couldn't re-use them in other places...
This basically means we couldn't use EchoUserLocator::locateUsersWatchingTitle,
we would have to roll our own that also excludes mentioned users.
Instead, this lets you add 'user-filters' (that functionality
actually exists already, but is not currently exposed), which
lists users to not send the notification to, even though they could
be in a user-locator.
Bug: T125428
Change-Id: Ifa0e2d3283f57624af4c5ec264f9f66223508e83
2016-02-02 13:16:39 +00:00
|
|
|
} );
|
|
|
|
}
|
|
|
|
|
2014-07-30 03:18:48 +00:00
|
|
|
// Filter non-User, anon and duplicate users
|
|
|
|
$seen = array();
|
2015-10-01 13:48:52 +00:00
|
|
|
$notify->addFilter( function ( $user ) use ( &$seen ) {
|
2014-07-30 03:18:48 +00:00
|
|
|
if ( !$user instanceof User ) {
|
|
|
|
wfDebugLog( __METHOD__, 'Expected all User instances, received:' .
|
|
|
|
( is_object( $user ) ? get_class( $user ) : gettype( $user ) )
|
|
|
|
);
|
2015-10-01 13:48:52 +00:00
|
|
|
|
2014-07-30 03:18:48 +00:00
|
|
|
return false;
|
|
|
|
}
|
|
|
|
if ( $user->isAnon() || isset( $seen[$user->getId()] ) ) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
$seen[$user->getId()] = true;
|
2015-10-01 13:48:52 +00:00
|
|
|
|
2014-07-30 03:18:48 +00:00
|
|
|
return true;
|
|
|
|
} );
|
|
|
|
|
|
|
|
// Don't notify the person who initiated the event unless the event extra says to do so
|
2013-01-26 12:14:36 +00:00
|
|
|
$extra = $event->getExtra();
|
|
|
|
if ( ( !isset( $extra['notifyAgent'] ) || !$extra['notifyAgent'] ) && $event->getAgent() ) {
|
2014-07-30 03:18:48 +00:00
|
|
|
$agentId = $event->getAgent()->getId();
|
2015-10-01 13:48:52 +00:00
|
|
|
$notify->addFilter( function ( $user ) use ( $agentId ) {
|
2014-07-30 03:18:48 +00:00
|
|
|
return $user->getId() != $agentId;
|
|
|
|
} );
|
|
|
|
}
|
|
|
|
|
|
|
|
// Apply per-wiki event blacklist and per-user whitelists
|
|
|
|
// of that blacklist.
|
|
|
|
if ( self::isBlacklisted( $event ) ) {
|
2015-10-01 13:48:52 +00:00
|
|
|
$notify->addFilter( function ( $user ) use ( $event ) {
|
2014-07-30 03:18:48 +00:00
|
|
|
// don't use self:: - PHP5.3 closures don't inherit class scope
|
|
|
|
return EchoNotificationController::isWhitelistedByUser( $event, $user );
|
|
|
|
} );
|
2012-04-27 15:14:24 +00:00
|
|
|
}
|
|
|
|
|
2014-07-30 03:18:48 +00:00
|
|
|
return $notify->getIterator();
|
2012-04-27 15:14:24 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Formats a notification
|
|
|
|
*
|
2014-08-05 00:15:14 +00:00
|
|
|
* @param EchoEvent $event The event for a notification.
|
|
|
|
* @param User $user The user to format the notification for.
|
|
|
|
* @param string $format The format to show the notification in: text, html, or email
|
|
|
|
* @param string $type The type of notification being distributed (e.g. email, web)
|
2012-11-26 22:57:28 +00:00
|
|
|
* @return string|array The formatted notification, or an array of subject
|
|
|
|
* and body (for emails), or an error message
|
2012-04-27 15:14:24 +00:00
|
|
|
*/
|
2015-08-06 22:36:05 +00:00
|
|
|
public static function formatNotification( EchoEvent $event, User $user, $format = 'text', $type = 'web' ) {
|
2012-04-27 15:14:24 +00:00
|
|
|
$eventType = $event->getType();
|
|
|
|
|
2014-02-06 23:06:54 +00:00
|
|
|
$res = '';
|
2015-08-10 18:27:52 +00:00
|
|
|
try {
|
|
|
|
$formatter = EchoNotificationFormatter::factory( $eventType );
|
|
|
|
$formatter->setOutputFormat( $format );
|
|
|
|
} catch ( InvalidArgumentException $e ) {
|
|
|
|
self::failFormatting( $event, $user );
|
2015-10-01 13:48:52 +00:00
|
|
|
|
2015-08-10 18:27:52 +00:00
|
|
|
return '';
|
|
|
|
}
|
|
|
|
set_error_handler( array( __CLASS__, 'formatterErrorHandler' ), -1 );
|
|
|
|
try {
|
|
|
|
$res = $formatter->format( $event, $user, $type );
|
|
|
|
} catch ( Exception $e ) {
|
|
|
|
$context = array(
|
|
|
|
'id' => $event->getId(),
|
|
|
|
'eventType' => $eventType,
|
|
|
|
'format' => $format,
|
|
|
|
'type' => $type,
|
|
|
|
'user' => $user ? $user->getName() : 'no user',
|
|
|
|
'exceptionName' => get_class( $e ),
|
|
|
|
'exceptionMessage' => $e->getMessage(),
|
|
|
|
);
|
|
|
|
LoggerFactory::getInstance( 'Echo' )->error( 'Error formatting notification', $context );
|
|
|
|
MWExceptionHandler::logException( $e );
|
2012-04-27 15:14:24 +00:00
|
|
|
}
|
2015-08-10 18:27:52 +00:00
|
|
|
restore_error_handler();
|
2012-04-27 15:14:24 +00:00
|
|
|
|
2014-09-08 18:55:55 +00:00
|
|
|
if ( $res === '' ) {
|
|
|
|
self::failFormatting( $event, $user );
|
|
|
|
}
|
|
|
|
|
|
|
|
return $res;
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Event has failed to format for the given user. Mark it as read so
|
|
|
|
* we do not continue to notify them about this broken event.
|
|
|
|
*
|
|
|
|
* @param EchoEvent $event
|
|
|
|
* @param User $user
|
|
|
|
*/
|
|
|
|
protected static function failFormatting( EchoEvent $event, $user ) {
|
|
|
|
// FIXME: The only issue is that the badge count won't be up to date
|
|
|
|
// till you refresh the page. Probably we could do this in the browser
|
|
|
|
// so that if the formatting is empty and the notif is unread, put it
|
2015-10-28 17:15:25 +00:00
|
|
|
// in the auto-mark-read APIs
|
|
|
|
EchoDeferredMarkAsReadUpdate::add( $event, $user );
|
2014-02-06 23:06:54 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* INTERNAL. Must be public to be callable by the php error handling methods.
|
|
|
|
*
|
2014-07-29 23:54:00 +00:00
|
|
|
* Converts E_RECOVERABLE_ERROR, such as passing null to a method expecting
|
2014-02-06 23:06:54 +00:00
|
|
|
* a non-null object, into exceptions.
|
|
|
|
*/
|
|
|
|
public static function formatterErrorHandler( $errno, $errstr, $errfile, $errline ) {
|
|
|
|
if ( $errno !== E_RECOVERABLE_ERROR ) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
2014-08-05 00:15:14 +00:00
|
|
|
throw new EchoCatchableFatalErrorException( $errno, $errstr, $errfile, $errline );
|
2012-04-27 15:14:24 +00:00
|
|
|
}
|
2012-06-08 05:33:25 +00:00
|
|
|
}
|