Last modified: 2014-11-17 10:36:44 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 T12268, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 10268 - Watchlist RSS feed should include diffs
Watchlist RSS feed should include diffs
Status: PATCH_TO_REVIEW
Product: MediaWiki
Classification: Unclassified
API (Other open bugs)
unspecified
All All
: Normal enhancement with 9 votes (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch, patch-reviewed
: 23349 (view as bug list)
Depends on:
Blocks: feeds
  Show dependency treegraph
 
Reported: 2007-06-15 08:50 UTC by Nick222
Modified: 2014-11-17 10:36 UTC (History)
12 users (show)

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


Attachments
Screenshot RSS-client (42.87 KB, image/png)
2007-06-15 12:56 UTC, Nick222
Details
Screenshot of the watchlist feed in Firefox (88.20 KB, image/png)
2007-06-15 14:14 UTC, Roan Kattouw
Details
Abilon show diff - history of Wikipedia page (71.98 KB, image/png)
2007-06-16 21:53 UTC, Nick222
Details
add diffs parameter to feedwatchlist that implements this feature (2.42 KB, patch)
2010-10-28 01:03 UTC, Ville Stadista
Details

Description Nick222 2007-06-15 08:50:27 UTC
Hello!

When I read my watchlist - I see long list with many entries and full text.

But when I open it ( http://ru.wikipedia.org/w/api.php?action=feedwatchlist ) in RSS-agregator (for example Abilon) I see very short list of messages - with entry name, author and date only.

How can I see RSS-feed the same watchlist as it is in browser?

Or it is a bug of RSS-feed http://ru.wikipedia.org/w/api.php?action=feedwatchlist ?

Thank You! :)
Comment 1 Roan Kattouw 2007-06-15 10:35:18 UTC
By default, action=feedwatchlist only returns entries changed in the last 24 hours. You can set this to 72 (maximum) using:

http://ru.wikipedia.org/w/api.php?action=feedwatchlist&hours=72


Comment 2 Nick222 2007-06-15 12:56:25 UTC
Created attachment 3778 [details]
Screenshot RSS-client
Comment 3 Nick222 2007-06-15 12:59:14 UTC
But what I see in RSS-client:

See attachment - that is all ("white" field right down corner) - only title, name and date ??

And where is content of message?
Comment 4 Roan Kattouw 2007-06-15 14:14:17 UTC
Created attachment 3779 [details]
Screenshot of the watchlist feed in Firefox

This is what I get from
http://www.mediawiki.org/w/api.php?action=feedwatchlist&hours=72

<?xml version="1.0" encoding="utf-8"?>
<?xml-stylesheet type="text/css" href="http://www.mediawiki.org/skins-1.5/common/feed.css?76"?>
<rss version="2.0" xmlns:dc="http://purl.org/dc/elements/1.1/">
	<channel>
		<title>MediaWiki - My watchlist [en]</title>
		<link>http://www.mediawiki.org/wiki/Special:Watchlist</link>
		<description>My watchlist</description>
		<language>en</language>

		<generator>MediaWiki 1.11alpha</generator>
		<lastBuildDate>Fri, 15 Jun 2007 14:06:45 GMT</lastBuildDate>
		<item>
			<title>Talk:API</title>
			<link>http://www.mediawiki.org/wiki/Talk:API</link>
			<description>/* Page watched ? */ no need (Catrope)</description>

			<pubDate>Thu, 14 Jun 2007 19:42:00 GMT</pubDate>			<dc:creator>Catrope</dc:creator>					</item>
		<item>
			<title>API</title>
			<link>http://www.mediawiki.org/wiki/API</link>
			<description>/* siteinfo / si (done) */ updating (Catrope)</description>

			<pubDate>Thu, 14 Jun 2007 17:03:14 GMT</pubDate>			<dc:creator>Catrope</dc:creator>					</item>
	</channel>
</rss>

All this shows is the name of the page, a link to it, and the user, summary and timestamp of the last edit. I also attached a screenshot of how Firefox displays this feed.

I'm diving into the code right now and will try to implement returning the diff since your last visit in the feed.
Comment 5 Nick222 2007-06-15 15:22:02 UTC
Hm... Summary - but where in this screenshot is "summary"?
Diff - Oh, yes, if I right understand You :)
Comment 6 Roan Kattouw 2007-06-16 19:38:00 UTC
The content of the page isn't supplied with the RSS feed, a link to the page is. We could change that link to point to the associated diff if you want to. IIRC, there are RSS readers (Mozilla Thunderbird's built-in RSS readers) that do show the content of the page. Unfortunately, action=feedwatchlist doesn't seem to work with Thunderbird (gives me the not logged in error). How did you configure Abilon to log in automatically?
Comment 7 Nick222 2007-06-16 21:49:41 UTC
1) As I know, RSS 2 supply content - or not?
2) About diff: for example, RSS of history of Wikipedia page show diff OK.
3) Abilon and FeedReader 3 have fileds "login" and "password" in options of each RSS-feed. Abilon show RSS-feed OK. But FeedReader 3 cannot show this RSS-feed. You can use link http://LOGIN:PASSWORD@ru.wikipedia.org/w/api.php?action=feedwatchlist&amp;hours=72 instead of fileds "login" and "password".
Comment 8 Nick222 2007-06-16 21:53:56 UTC
Created attachment 3795 [details]
Abilon show diff - history of Wikipedia page
Comment 9 Roan Kattouw 2007-06-16 22:59:37 UTC
Somehow that doesn't work with Thunderbird. Your screenshot must be of a Recent Changes feed, it's true that they do show the diffs. I'll see how the RC feed is modelled and try to make the watchlist feed the same (showing the diffs since last view, or the last diff if you've seen the last revision already).
Comment 10 Roan Kattouw 2007-06-17 20:31:04 UTC
I've tried to get the diff of the last revision into the feed, but I couldn't get it working. It should be easy (in theory) with the patch attached to bug 10297: query api.php?action=query&generator=watchlist&prop=revisions&rvprop=ids|user|comment internally and put the diff between the last two revisions in the <description> field (if the showdiff parameter is set of course). It doesn't sound difficult, but I couldn't get it done.
Comment 11 Daniel Cannon (AmiDaniel) 2007-09-20 20:32:30 UTC
Seeing as nothing's been done on this since June, I'm marking it for LATER.
Comment 12 Ville Stadista 2010-10-28 01:03:45 UTC
Created attachment 7761 [details]
add diffs parameter to feedwatchlist that implements this feature

I doesn't change getVersion() string, which should probably be changed if this is committed to SVN. 

Patch is against the latest SVN snapshot.
Comment 13 Bryan Tong Minh 2010-11-27 18:22:07 UTC
The parameter "diffs" is ambiguous. I am going to change that to something like "linktodiff"
Comment 14 Bryan Tong Minh 2010-11-27 18:29:12 UTC
I've been rereading this thread, but it seems that your patch does not address the issue raised in this bug. This bug is about including the contents of the diff, not about the diff link itself.
Comment 15 Ville Stadista 2010-11-27 19:03:18 UTC
Sorry, I should have read the bug description more carefully; the title seemed to match what I had in mind.

Can you still commit the patch? I think it is a very useful feature and I tested it by installing MediaWiki on my own computer and it should work fine. Meanwhile, I could research if I can help with the exact concern raised in this bug.
Comment 16 Bryan Tong Minh 2010-11-27 19:27:16 UTC
Done in r77366.

Unassigning for the actual feature request on this bug.

I wonder if it would be considered unacceptable for performance reasons to include the diff of every entry?
Comment 17 Ville Stadista 2010-11-27 19:43:28 UTC
After quickly looking through the MW database layout, it seems like implementing this feature in the way originally meant would require a query to the "text" table to produce the diff output; something that is not needed in my solution. AFAIK this would reduce the performance of the feedwatchlist feature, and would requrie much more extensive testing to be safe for commiting to high-traffic sites like Wikipedia. I would dare to say that without some elaborate diff caching solution the performance hit might be just too hard.

In my solution the comment and the contributor are visible in the RSS/Atom feed item, and the user only has to open the item's link in a browser to see the diff. This does not require any additional database queries when the feedwatchlist is generated unless the user opens the link, much like by opening the "diff" link in Special:Watchlist.
Comment 18 Bryan Tong Minh 2010-11-27 19:52:30 UTC
All the caching is done by the DifferenceEngine in includes/diff. There is a good chance that most (especially recent) diffs are cached. Also I see that the RC feed includes diffs. On the other hand RC is the same for all users and for a high traffic site it is pretty much guaranteed that the recent diffs are cached, whereas the watchlist feed may contain much older items. That said, I see that it is limited to 72 hours.

Conclusion: most diffs are probably cached, so including all diffs is not a too big issue. Any comments on this?
Comment 19 Ville Stadista 2010-11-27 20:13:58 UTC
Well, in that case the relevant diffs would get cached when the user first time loads his feedwatchlist in opening his browser/aggregator, and subsequent queries would either concern new recent changes or already cached diffs. Sounds doable and performance could remain acceptable.
Comment 20 p858snake 2011-04-30 00:10:05 UTC
*Bulk BZ Change: +Patch to open bugs with patches attached that are missing the keyword*
Comment 21 Sumana Harihareswara 2011-11-09 21:48:37 UTC
(Adding the "reviewed" keyword since the patch has been reviewed, and in fact committed.  Thanks, Ville Stadista!)
Comment 22 Rd232 2013-04-27 11:34:38 UTC
*** Bug 23349 has been marked as a duplicate of this bug. ***
Comment 23 Robin Pepermans (SPQRobin) 2014-05-10 00:05:00 UTC
See also Gerrit change #132589 (to use the diff link in the equivalent contributions feed) and Gerrit change #132590 (to always link to the diff in the watchlist feed).
Comment 24 Robin Pepermans (SPQRobin) 2014-06-22 17:39:04 UTC
This bug is equivalent to bug 41085 (concerning the contributions feed). These two bugs should probably be fixed in the same way.
Comment 25 Gerrit Notification Bot 2014-07-29 17:55:01 UTC
Change 150271 had a related patch set uploaded by Rohan013:
Include diffs in Watchlist feed

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

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


Navigation
Links