Last modified: 2014-06-11 22:49:41 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T22394, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 20394 - RCFeed: Expose log_id in RC feed
RCFeed: Expose log_id in RC feed
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Recent changes (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: 1.24.0 release
Assigned To: Kunal Mehta (Legoktm)
: easy
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-08-25 19:01 UTC by Gurch
Modified: 2014-06-11 22:49 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Gurch 2009-08-25 19:01:20 UTC
It would be useful if the IRC recent changes feed included the IDs of log items. Revision IDs are already there in the URLs, but for log items the URL is just [[Special:Log]].
Comment 1 Krinkle 2014-06-05 09:04:02 UTC
The IRC RC Feed is no longer in development (considered frozen for backwards compatibility).

It is superseded by the JSON RC Feed. Either way, it seems valuable to have this there.


It is already part of the RecentChange object[1], and already exposed in the RecentChanges API query module[2]. Just need to expose it in RCFeedFormatter[3] as well.


[1] https://github.com/wikimedia/mediawiki-core/blob/77f9979e/includes/changes/RecentChange.php#L47
[2] https://github.com/wikimedia/mediawiki-core/blob/77f9979e9/includes/api/ApiQueryRecentChanges.php#L527-L532
[3] https://github.com/wikimedia/mediawiki-core/blob/77f9979e9b6/includes/rcfeed/MachineReadableRCFeedFormatter.php#L40
Comment 2 Gerrit Notification Bot 2014-06-06 02:00:14 UTC
Change 137853 had a related patch set uploaded by Legoktm:
Expose log_id in RC feed

https://gerrit.wikimedia.org/r/137853
Comment 3 Ori Livneh 2014-06-06 02:46:42 UTC
What is the value of having it there?
Comment 4 Krinkle 2014-06-07 08:26:10 UTC
(In reply to Ori Livneh from comment #3)
> What is the value of having it there?

Hm.. good point. How can the log_id be utilised by a consumer? It is not part of a permalink.
Comment 5 Kunal Mehta (Legoktm) 2014-06-07 08:36:56 UTC
If you want to do a database query for whatever reason, you'd probably want the log_id. Or if you want to revdel via the API (RC adminbot watching for bad usernames), the log_id is the id parameter that you'd use.

(In reply to Krinkle from comment #4)
> Hm.. good point. How can the log_id be utilised by a consumer? It is not
> part of a permalink.

Even though it's not used in linking, it is a permanent unique id for the given log entry, which can be useful if you're storing the events for future usage.
Comment 6 Krinkle 2014-06-11 22:26:20 UTC
Okay, those two use cases sounds great.
Comment 7 Gerrit Notification Bot 2014-06-11 22:32:06 UTC
Change 137853 merged by jenkins-bot:
RCFeed: Add log_id property (exposes rc_logid attribute)

https://gerrit.wikimedia.org/r/137853

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links