In order to exclude protected pages with lots on vandalism activity from trending results we need to provide a page-restrictions-change event in the Event-Platform infrastructure.
Description
Description
Details
Details
Related Changes in Gerrit:
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Declined | None | T154899 Protected pages should be excluded from trending results | |||
| Resolved | • Pchelolo | T160942 Create mediawiki.page-restrictions-change event |
Event Timeline
Comment Actions
One minor problem: normally in our events for changed properties we would have old values and new values, but the ArticleProtectionComplete hook provides only the new values, but there's no way to get old values. We can add a way to get them, but since we don't have a use-case for old values, I guess we could skip that and not add them right now. What do you think?
Comment Actions
That sounds fine for my use case. I just need to be able to know a page has become protected or unprotected.
Comment Actions
If we do that, we should probably still design the schema so that old value fields are still there, but optional.
Comment Actions
Change 343765 had a related patch set uploaded (by Ppchelko):
[mediawiki/event-schemas] Added mediawiki.page-restrictions-change schema.
Comment Actions
Change 343767 had a related patch set uploaded (by Ppchelko):
[mediawiki/extensions/EventBus] Emit mediawiki.page-restrictions-change event.
Comment Actions
Change 343765 merged by jenkins-bot:
[mediawiki/event-schemas@master] Added mediawiki.page-restrictions-change schema.
Comment Actions
Change 343767 merged by jenkins-bot:
[mediawiki/extensions/EventBus@master] Emit mediawiki.page-restrictions-change event