Last modified: 2012-06-17 19:22:28 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 T34895, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 32895 - response does not always contain an url attribute if llurl=1 is used (prop=langlinks)
response does not always contain an url attribute if llurl=1 is used (prop=la...
Status: RESOLVED WONTFIX
Product: MediaWiki
Classification: Unclassified
API (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
http://zh.wikipedia.org/w/api.php?act...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-12-09 10:39 UTC by merl
Modified: 2012-06-17 19:22 UTC (History)
7 users (show)

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


Attachments

Description merl 2011-12-09 10:39:26 UTC
If i am using llurl=1 on my request my parser always expect that there is an url attribute at the langlinks/ll-Tag.
I think all parsers should expect this if there is not error.
I would suggest that it is better to not display this langlink at all if the title contains invalid utf-8. This would be the same behaviour as done by the renderer. ~~~~
Comment 1 Mark A. Hershberger 2011-12-09 16:07:58 UTC
(In reply to comment #0)
> I would suggest that it is better to not display this langlink at all if the
> title contains invalid utf-8. This would be the same behaviour as done by the
> renderer. ~~~~

As you point out, those links don't show up on the sidebar.
Comment 2 Sam Reed (reedy) 2011-12-09 16:46:14 UTC
I'm not sure why they need to be always adding, we do a lot of things only added optionally, I don't see why the exception is needed here?
Comment 3 Mark A. Hershberger 2011-12-09 17:01:57 UTC
As it is, this seems like a great way to find invalid UTF-8 in the urls
Comment 4 db [inactive,noenotif] 2011-12-09 20:07:09 UTC
When the langlinks not shown in the sidebar, why storing this item in the database? action=parse does not display the items:

http://zh.wikipedia.org/w/api.php?action=parse&pageid=15669&prop=langlinks

This is a page, which is not touched since a very long time (touched="2007-12-04T09:51:12Z"). After a nulledit the invalid items are gone from the database.

Maybe let the api filter the invalid items out of the result?
Comment 5 Umherirrender 2012-06-06 15:51:06 UTC
In my opinion you should not assume, that "url" is a required param, it is optional. The module returns all entries from the database and adds extra information, when there are extra information. On an invalid title, he cannot provide extra information and that is ok. Hiding the whole output makes no sense in this case.

Supporting WONTFIX.
Comment 6 Umherirrender 2012-06-17 19:22:28 UTC
Setting WONTFIX per comment 5

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


Navigation
Links