* Moved most of the content from notify to getEventNotifyTypes
* Added phpunit test verifying per-event config overrides defaults
* Remove special-snowflake welcome handling with proper configuration
Change-Id: Ic1aae11d37f23f2b7b8abe3c1edaa414e29be021
timestamp is generated in MW format (YYYYMMDDHHMMSS)
for notification but it's
loaded into model in the database specific format. We just need
to convert it back to MW format, it just happens that MW
format is the same as MySQL timestamp format
Change-Id: I1f7f2aeb32b63d47e92631d6daa711d9c32e4017
The code was passing the notifyTypes in the hook for each user, but
the second user would get passed whatever made it through the first
user.
It is unlikely this is intended. Give each user a fresh set of the
configured notify types.
Change-Id: I13059d380ea54c71cd1c062fd803897f5e5ad2b8
Remove the records:
* If a page is no longer valid
* If a notification somehow has been read
* If a notification is no longer a bundle base and has not been removed
Change-Id: I282568691d6649c6e3263aea598c3bba29119a29
* cache function result in local variables
* Update the logic of generating notification date header
Change-Id: I04c3ed853076f17c819da8f27bfdb169e99b2a3a
Core titleCache doesn't do what I expect it to do, issuing
Title::newFromId( 1 ) mulitple times would issue multiple
idential queries to the database. It doesn't return what's
already in the cache.
The goal of this patch is to batch load titles via newFromIDs,
and save the number of mysql queries
Change-Id: I8fe767ac2669e67bdf7d17eecccfc0dcb6b5fc7d
The new locateUsersWatchingTitle implementation could end up returning
thousands of users, currently on enwiki there are 25 titles with more
than 10k subscribed users and aprox 550 titles with more than 1k subscribed
users.
This switches the user collection to an iterator based implementation so that
we no longer need to have the entire users list at any one time.
Change-Id: I3d3fa9328f348bb48682d3658622952ce82d3925
The special page will now not auto mark a notification as read if
it has a target url. Currently no notifications have target urls
but this will be changed in a later patch...
Change-Id: I9bd71d59391189d5d761ab5f1c84af0bc3554be0
