On Sat, Jun 12, 2010 at 10:34 AM, Peter17 <[email protected]> wrote: > Hello to all! > > I have posted here [1] a proposal about adding two fields in the > interwiki table (API URL and DB name). > > One of the goals is to simplify interwiki transclusion, but those > field might be useful for other interwiki applications. > > Could you please read it and let me know about your remarks and > suggestions, on this list and/or on the talk page? > > Thanks in advance > > [1] > http://www.mediawiki.org/wiki/User:Peter17/Reasonably_efficient_interwiki_transclusion#Adding_fields_to_the_interwiki_table > > -- > Peter Potrowl > http://www.mediawiki.org/wiki/User:Peter17 > > _______________________________________________ > Wikitech-l mailing list > [email protected] > https://lists.wikimedia.org/mailman/listinfo/wikitech-l >
I like lists better, so I'll reply here :) In general, I like your approach. I've been suggesting to add column(s) like these for awhile now. Looking over your ideas, I have a few suggestions that I think might simplify it a bit: 1) iw_trans - I don't think this needs to become more than a boolean like it is. If we allow transwiki inclusion, we'll have to use a DB or API connection. Since a DB connection will always be preferable to an HTTP request to the API, it would be safe to use the existence of a db name as an indicator to use it, else fall back to the API. 2) iw_dbname / iw_api - You could probably combine these into one column. It could store a value like "dbname=abc;api=http://foo.com/etc"; which would be loaded and split when the Interwiki object is constructed. I'm more interested in the methodologies used in the Interwiki class in general as opposed to the specific use case of IW transclusion. I think our interwiki data requests can be standardized using some of the groundwork you're laying here, and that's what I'd really like to see in the long run :) -Chad _______________________________________________ Wikitech-l mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/wikitech-l
