Page MenuHomePhabricator

RCFeed: Send boolean flag indicating that target page is a redirect
Open, LowestPublic

Description

Author: conrad.irwin

Description:
This would aid all tools that need to take special action, or no action, on redirects by meaning that they don't have to load the page independantly to determine the redirect status for themselves.

Details

Reference
bz14843

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 10:09 PM
bzimport set Reference to bz14843.

I do not think that MediaWiki tags redirects in RC, like new pages, bot edits and minor edits, so the dumb IRC bot will also not know.

CANTFIX, I think.

matthew.britton wrote:

MediaWiki does tag redirects in recent changes, because the API's recent changes query shows them. So it is possible.

(In reply to comment #2)

MediaWiki does tag redirects in recent changes, because the API's recent
changes query shows them. So it is possible.

No, it doesn't. That's handled because the API queries both the recentchanges and page tables, giving it access to page_is_redirect. This information isn't _directly_ stored in the database. However, implemented it in r44454 for the IRC output in a different manner.

This doesn't work as intended; when editing existing articles, this'll pull the *previous* redirection state of the article, rather than the current state, so it's wrong whenever a page is changed into, or away from, being a redirect.

Reverted in r44456

This is about the MediaWiki recent changes IRC feed itself, isn't it?
I doubt the format will be changed, however; such info should probably wait for its reimplementation?

The IRC RC Feed is no longer in active development (considered frozen for backwards compatibility).

It is superseded by the JSON RC Feed. It seems valuable to have a 'redirect' flag there.

There is currently no 'redirect' flag in the RecentChange object itself, however we do construct such a property in the RecentChanges API query module[1].

Seems interesting to consider adding that to RCFeedFormatter[2] as well.

[1] https://github.com/wikimedia/mediawiki-core/blob/77f9979e9/includes/api/ApiQueryRecentChanges.php#L511-L514
[2] https://github.com/wikimedia/mediawiki-core/blob/77f9979e9b6/includes/rcfeed/MachineReadableRCFeedFormatter.php#L40

Krinkle lowered the priority of this task from Medium to Lowest.
Krinkle updated the task description. (Show Details)
Krinkle set Security to None.
Krinkle added a subscriber: ori.