Last modified: 2013-08-22 14:39:16 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 T49219, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 47219 - Empty Wikidata entries showing in RSS feed on recent changes in wikipedia
Empty Wikidata entries showing in RSS feed on recent changes in wikipedia
Status: VERIFIED FIXED
Product: Wikimedia
Classification: Unclassified
Wikidata (Other open bugs)
wmf-deployment
All All
: High major with 2 votes (vote)
: ---
Assigned To: Wikidata bugs
http://bs.wikipedia.org/wiki/Posebno:...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-04-14 15:44 UTC by Edin
Modified: 2013-08-22 14:39 UTC (History)
4 users (show)

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


Attachments
Example of an empty change, representing a change on Wikidata (148.28 KB, image/jpeg)
2013-04-14 15:47 UTC, Edin
Details

Description Edin 2013-04-14 15:44:51 UTC
On the RSS feed on recent changes I get empty edits, which are actually changes that are made on Wikidata:

  http://bs.wikipedia.org/wiki/Posebno:NedavneIzmjene?feed=rss

For example I see now an edit on the arctile "Ruta" where the feed shows a change of revision with the following link to view the changes:

 http://bs.wikipedia.org/w/index.php?title=Ruta&diff=2075570&oldid=2075570

It is obviously the same revision. Moreover, the change is shown to be made at "15:29, 14 april 2013", which is exactly the same time and user that is shown as the latest revision on Wikidata (at this moment):

  http://www.wikidata.org/w/index.php?title=Q25062&oldid=25100268

I'm not sure whether this is actually a real bug or some "feature", but it doesn't seem to serve any purpose since my RSS client is being spammed with this useless information. I'm also not sure when this started to occur, but I guess after the roll out of version 1.22wmf1. With the previous version(s) I was not experiencing any of these issues.
Comment 1 Edin 2013-04-14 15:47:18 UTC
Created attachment 12102 [details]
Example of an empty change, representing a change on Wikidata
Comment 2 tm112 2013-04-28 13:01:12 UTC
I confirm that this happens. And I'm almost sure it was also before 1.22wmf1. And I confirm that it's annoying. But that should be easy to fix, it's in the ApiFeedWatchlist.php.

These feed items should be easy to find as 'revid' and 'old_revid' are equal for them. But then there are two solutions:
1) To mark them as Wikidata edits – this should be easy as they have no comment so that is a good place to use for a notice, e.g. "(change on Wikidata)".
2) To extract them completely (do not show them). I'm not sure how. Maybe a simple if/else statement – function createFeedItem() returns new FeedItem. But I'm not a PHP specialist and I don't know if a code (in short):
function {
 if (revid==old_revid) {… return new FeedItem(…)}
 else {}
}
would work.
Comment 3 tm112 2013-04-28 13:09:46 UTC
I see also the 'notificationtimestamp' is empty and the useid is '0'.
I don't know which is safier and cheaper to use when fixing the bug.

P.S. Of course there shouldn't be else{} in my comment above ;)
Comment 4 tm112 2013-04-28 13:20:21 UTC
Hm, we can exclude them but… some may still want to see it. Nevertheless that's also a kind of information.
Comment 5 Edin 2013-04-28 16:42:12 UTC
I don't think anyone wants to see this. The information on revision changes on items on Wikidata should be completely separate from Wikipedia, in terms of RSS feeds on Wikipedia. And even if someone wants to see this, it does not add any relevant information except that something has been changed (the changes are empty). However, if this is to keep for the RSS feed, then in my opinion it should be optional with the ability to activate it by using an additional flag. But certainly not as default.
Comment 6 tm112 2013-04-28 22:51:44 UTC
I committed If22827129b04d423711f921307dc820d4840d9d1 as a fix. It simply introduces a new value ('changed') for the param 'show'.
//Unfortunately I have no idea how to check how it works on my local wiki/vagrant, cause I don't have wikidata here ;)
Comment 7 Gerrit Notification Bot 2013-04-30 17:06:43 UTC
https://gerrit.wikimedia.org/r/61286 (Gerrit Change If22827129b04d423711f921307dc820d4840d9d1) | change APPROVED and MERGED [by jenkins-bot]
Comment 8 Andre Klapper 2013-05-14 09:23:48 UTC
Gerrit patch got merged, can this bug report get closed as RESOLVED FIXED or is there anything else to do?
Comment 9 Andre Klapper 2013-05-17 13:00:06 UTC
(In reply to comment #8)
> Gerrit patch got merged, can this bug report get closed as RESOLVED FIXED or
> is there anything else to do?

I assume that means that the problem has been FIXED. If it is not, please reopen.

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


Navigation
Links