mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/Echo
synced 2024-11-27 17:20:40 +00:00
(bug 45591) Fix inconsistent function return type
Change-Id: Iccc4acb2bb809d0f957d73d6e8a4da3033839c1e
This commit is contained in:
parent
42dbd11d7e
commit
54c52b2a4e
|
@ -132,10 +132,10 @@ class EchoNotificationController {
|
|||
return;
|
||||
}
|
||||
|
||||
// Check if the event object still has an enabled type. The check is necessary because an extension could
|
||||
// get turned off and we may still have un-processed event type in the queue
|
||||
// Check if the event object has valid event type. Events with invalid
|
||||
// event types left in the job queue should not be processed
|
||||
if ( !$event->isEnabledEvent() ) {
|
||||
return false;
|
||||
return;
|
||||
}
|
||||
|
||||
if ( $event->getType() == 'welcome' ) { // Welcome events should only be sent to the new user, no need for subscriptions.
|
||||
|
|
Loading…
Reference in a new issue