Last modified: 2014-09-06 14:19:30 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 T25524, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 23524 - Api Modules as followup to bug 14473 (Add iwlinks table to track inline interwiki link usage)
Api Modules as followup to bug 14473 (Add iwlinks table to track inline inter...
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
API (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Roan Kattouw
:
Depends on: 14473
Blocks: 70492
  Show dependency treegraph
 
Reported: 2010-05-14 11:43 UTC by Sam Reed (reedy)
Modified: 2014-09-06 14:19 UTC (History)
5 users (show)

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


Attachments

Description Sam Reed (reedy) 2010-05-14 11:43:26 UTC
<RoanKattouw> Also, we'll want API support for http://www.mediawiki.org/wiki/Special:Code/MediaWiki/65104
<RoanKattouw> It'd be trivial to expose ParserOutput::getInterwikiLinks() in action=parse
<Reedy> I'll log it as a bug and do it later :)
<RoanKattouw> Plus we'd probably want prop=interwikilinks and list=interwikibacklinks (bad name :P )
<RoanKattouw> Yeah thanks
<Reedy> yeah
<Reedy> i was thinking something linksonpage esk aswell
<RoanKattouw> Yeah that's what prop=interwikilinks would be, the equivalent of prop=links: "get me all (iw) links on this page"
<RoanKattouw> And list=(iw)backlinks is "get me all (iw) links to this page
<Reedy> mhmm
Comment 1 Sam Reed (reedy) 2010-05-14 11:44:09 UTC
Adding dependency of 14473 for lazy linking
Comment 2 Sam Reed (reedy) 2010-05-14 12:47:46 UTC
For the parse bit.. We're gonna need the the target site and the target link..


<interwikilinks>
<iw>wikibooks:test</iw>
</interwikilinks>

Seems a bit crappy/pointless - Presumably we want both bits separate. Otherwise it's just more work for the client.

Just how to present it

<iw prefix="wikibooks">test</iw>
Comment 3 Bryan Tong Minh 2010-05-14 13:07:21 UTC
We should of course also optionally output the full url as well.
Comment 4 Sam Reed (reedy) 2010-05-14 13:24:11 UTC
Hmm. Good idea. Probably doing it as a property of fullinterwikilinks, or something. If it has that, do the url too, else without full, just the prefix/target

<iw prefix="wikibooks" link="test" />
<iw prefix="wikibooks" link="test">http://en.wikibooks.org/wiki/test</iw>
Comment 5 Sam Reed (reedy) 2010-05-14 20:58:57 UTC
r66440 does prop=iwlinks

Won't commit any of the parse stuff till we decide on how we want it.

See if i can get the iwbacklinks done..
Comment 6 Sam Reed (reedy) 2010-05-14 21:38:42 UTC
More a less got a backlinks module working

iwbltitle=wikibooks:Test

or something more like
iwblprefix=wikibooks&iwbltitle=Test

The latter would make more sense, i think.
Comment 7 Roan Kattouw 2010-05-22 21:03:47 UTC
(In reply to comment #6)
> More a less got a backlinks module working
> 
> iwbltitle=wikibooks:Test
> 
> or something more like
> iwblprefix=wikibooks&iwbltitle=Test
> 
> The latter would make more sense, i think.

Yes, I would agree.

(In reply to comment #2)
> Just how to present it
> 
> <iw prefix="wikibooks">test</iw>
As this is what you seem to have done in prop=iwlinks, let's go with that throughout.

You didn't implement returning URLs (yet?), but as far as I'm concerned that can also be implemented as <iw prefix="wikibooks" url=".......">test</iw>
Comment 8 Sam Reed (reedy) 2010-05-25 12:45:11 UTC
Where do we want to put the full links? On prop=iwlinks and on the parse output? Or just one?
Comment 9 Sam Reed (reedy) 2010-05-25 13:33:34 UTC
r66872 gives &url for ApiQueryIWLinks to expand full URL
Comment 10 Sam Reed (reedy) 2010-05-25 13:48:48 UTC
r66873 does the parse bit. Bar the external link expansion (if deemed needed)

Just backlinks to finish
Comment 11 Sam Reed (reedy) 2010-05-25 21:38:23 UTC
r66891

I think this is finished...

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


Navigation
Links