[05:24:56] Lydia_WMDE: ping [05:35:42] Daniel_WMDE: any idea where Lydia_WMDE is? [06:15:43] jeremyb: pong [06:15:48] hey! [06:15:50] jeremyb: i am in tallinn on vacation ;-) [06:15:53] wasup? [06:15:57] not any more! [06:16:02] heh alright [06:16:06] so... [06:17:04] seems like there's a good chance that hosting this on WMDE servers is not going to to happen without legal approving it and ryan thinks legal won't approve it [06:17:17] i don't entirely understand but i just want to get it overwith already [06:17:39] Lydia_WMDE: is there any objection to hosting the same content from the main WMF cluster? [06:17:46] and then not changing DNS at all? [06:18:23] i'll do the initial commit and maybe help with updates. and you guys can submit changes in gerrit too [06:19:15] i can't guarantee they'll be approved fast (and certainly will be slower than on your own server where you have root). but it should at least be workable [06:19:22] Lydia_WMDE: what say you? [06:20:57] jeremyb: hmmm that is not my decision tbh - can you send an email to Denny? [06:21:04] i need to run for a bit [06:21:06] meeting [06:21:07] :) [06:21:39] Lydia_WMDE: meeting on vacation?! [06:21:57] Lydia_WMDE: coming to NYC before DC? or just after? [06:27:15] jeremyb: heh yes i do meetings/conferences on my vacation ;-) [06:27:22] and no unfortunately not coming to NYC [06:27:39] Lydia_WMDE: you must after? NYT? [06:27:46] Lydia_WMDE: you have mail [06:46:50] jeremyb: you're the best [06:47:38] Lydia_WMDE: i'm sleepy! [06:47:46] Lydia_WMDE: you need to stop having meetings! [06:47:55] Lydia_WMDE: and get to NYC soon! [06:48:06] Lydia_WMDE: anyway, seriously, NYT? [06:48:09] hehe [06:48:12] will try [07:14:03] more mail sent [07:14:12] where is denny? ;) [07:14:17] anyway, sleeping any sec [07:27:31] Change merged: Henning Snater; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/13475 [07:28:53] New review: Tobias Gritschacher; "tests are running, looks good to me, please fix whitespaces." [mediawiki/extensions/Wikibase] (master); V: 0 C: 0; - https://gerrit.wikimedia.org/r/13864 [08:28:29] New patchset: Tobias Gritschacher; "fixed error in ItemViewTest; description coming from dataprovider seemed to have one layer of arrays too much" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14019 [08:44:07] New patchset: Henning Snater; "made aliases widget pick up the value returned from the API" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/13878 [09:08:46] Change merged: Jeroen De Dauw; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14019 [09:23:24] New patchset: Henning Snater; "made aliases widget pick up the value returned from the API" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/13878 [09:35:08] New patchset: Henning Snater; "hiding navigation elements on item view page removing layout gap again" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14023 [09:39:52] New patchset: Jeroen De Dauw; "Fix issue caused by refactor of some days back" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14024 [09:40:25] Change merged: Jeroen De Dauw; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14024 [09:41:13] New patchset: Jens Ohlig; "Refactored Selenium tests to use code blocks instead of re-using @current_page object." [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/13864 [09:42:32] New patchset: Jeroen De Dauw; "Killed dead code" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14025 [09:45:37] Change merged: Tobias Gritschacher; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/13864 [09:57:16] New review: John Erling Blad; "Found additional problems, could be due to missing language fallbacks ..." [mediawiki/extensions/Wikibase] (master); V: 0 C: -1; - https://gerrit.wikimedia.org/r/13931 [10:00:58] Daniel_WMDE: this might be of interest to you https://gerrit.wikimedia.org/r/#/c/14023/ [10:04:10] New review: John Erling Blad; "It is language fallbacks that are the problem. With two accounts I have one that is in English and w..." [mediawiki/extensions/Wikibase] (master); V: 0 C: -1; - https://gerrit.wikimedia.org/r/13931 [10:05:06] Change abandoned: Henning Snater; "(no reason)" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14023 [10:07:36] JeroenDeDauw: i was already looking at that, yea [10:07:38] moin Silke_WMDE [10:07:50] Henning_WMDE: please find another way to hide this thing :) [10:09:50] Denny_WMDE: i'll look over the last sprints' items and ping you then [10:12:15] ok [10:13:43] Daniel_WMDE: sorry i keep disconnecting, i might miss messages [10:14:01] Daniel_WMDE: would prefer to move to skype, better protocol for shaky connections [10:14:02] not good for skyping :/ [10:14:21] Denny_WMDE2: grab a cable! [10:14:23] better than IRC [10:14:24] aah [10:14:26] yeah [10:14:34] simple solutions [10:14:37] ;) [10:14:59] New patchset: Jeroen De Dauw; "Added ItemDiffViewTest" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14027 [10:16:16] New patchset: Jeroen De Dauw; "Added missing test registration" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14028 [10:16:58] Daniel_WMDE: can't. macbook air [10:20:06] New review: Daniel Werner; "looking good" [mediawiki/extensions/Wikibase] (master); V: 1 C: 2; - https://gerrit.wikimedia.org/r/13890 [10:20:09] Change merged: Daniel Werner; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/13890 [10:20:18] Henning_WMDE: we can hide it at least for the JS version: $('#contentSub:empty').hide(); [10:25:01] Denny_WMDE2: von mir aus können wir. haste jetzt netz? [10:30:54] TobiasG_WMDE: check if it's empty, *then* hide it [10:31:04] TobiasG_WMDE: it's a container the skin may use for a variety of things [10:31:17] pretty much any navigation, indicator or warning to be shown on top of the article will be in there [10:35:00] Daniel_WMDE: sure, that's exactly what $('#contentSub:empty').hide(); is doing: check if its emty, and then hide [10:35:32] oh, missed the :empty bit, sorry :) [10:48:47] New review: Jens Ohlig; "... for great justice!" [mediawiki/extensions/Wikibase] (master); V: 1 C: 2; - https://gerrit.wikimedia.org/r/14025 [10:48:49] Change merged: Jens Ohlig; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14025 [10:49:54] New review: Jens Ohlig; "Easy enough." [mediawiki/extensions/Wikibase] (master); V: 1 C: 2; - https://gerrit.wikimedia.org/r/14028 [10:49:57] Change merged: Jens Ohlig; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14028 [10:50:18] New patchset: Daniel Werner; "Special pages display useful link labels for data item links now." [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/13931 [11:26:04] Daniel_WMDE: I just noticed that the ViewItemAction and ItemView still have redundant code and the first doesn't use the later. Looks quite messy to me. Did you plan on leaving it this way? [11:32:59] New patchset: Jens Ohlig; "UTF-8 fixing." [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14030 [11:33:00] New patchset: Jeroen De Dauw; "Changed page_id field to page_title" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14031 [11:33:47] New patchset: Jeroen De Dauw; "Changed page_id field to page_title" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14031 [11:41:53] New patchset: John Erling Blad; "Remove multilingual labels and descriptions when they are set to empty strings." [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14032 [12:50:29] https://en.wikipedia.org/wiki/Wikipedia:Wikipedia_Signpost/2012-07-02/Op-ed [12:50:32] Denny_WMDE2: ---^ [12:50:46] only browsed through it so f seems interesting [13:04:55] Denny_WMDE2: yep, i have that open too [13:09:27] New review: John Erling Blad; "See also" [mediawiki/extensions/Wikibase] (master); V: 0 C: 0; - https://gerrit.wikimedia.org/r/14032 [13:19:06] New patchset: Henning Snater; "hiding contentSub and mw-jump elements specifically on the item page removing the layout gap" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14038 [13:19:19] New patchset: Jeroen De Dauw; "Kill obsolete sql file" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14039 [13:22:27] New patchset: Jeroen De Dauw; "Remove obsolete script to populate interwikitable (since we now have such stuff populating the sites table)" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14042 [13:25:31] New patchset: Jeroen De Dauw; "Alerady removed this, but apparently we have copies of this script for redundancy? >_>" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14045 [13:26:30] New patchset: Jens Ohlig; "UTF-8 fixing. Included present? check again. Change-Id: Ia9f6c422101d76255ee559c725102fd1f74b4029" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14030 [13:28:14] New patchset: Tobias Gritschacher; "UTF-8 fixing." [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14030 [13:29:19] Jens_WMDE: please have another look at changeset https://gerrit.wikimedia.org/r/14030 ant check if it works on your machine [13:30:16] New review: Daniel Kinzler; "this section exists for a reason. you can't just hide it and hope." [mediawiki/extensions/Wikibase] (master); V: 0 C: -1; - https://gerrit.wikimedia.org/r/14038 [13:38:49] New patchset: Jens Ohlig; "UTF-8 fixing." [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14030 [13:41:09] Change merged: Tobias Gritschacher; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14030 [13:42:46] Daniel_WMDE: so what's the deal with the sitelink object? [13:42:57] Change merged: Tobias Gritschacher; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14039 [13:46:42] Change merged: Tobias Gritschacher; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14042 [13:46:59] New patchset: John Erling Blad; "Remove multilingual labels and descriptions when they are set to empty strings." [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14032 [13:47:17] Change merged: Tobias Gritschacher; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14045 [13:49:20] New review: John Erling Blad; "Patchset 2 adds deletion markers for API return from wbdeletelanguageattribute and wbsetlanguageattr..." [mediawiki/extensions/Wikibase] (master); V: 0 C: 0; - https://gerrit.wikimedia.org/r/14032 [13:50:50] JeroenDeDauw: was thinking of doing that today, maybe, or on the plane. it's not urgent, is it? [13:52:42] Daniel_WMDE: I am currently looking at getting the client to work [13:52:54] Daniel_WMDE: and it's quite possible that in the process I'll need to implement this stuff] [13:53:06] Daniel_WMDE: so I want to be sure we're not both doing it at the same time :) [13:53:43] JeroenDeDauw: hm... let me poke at it now. if I don't submit it today, you go ahead. [13:56:24] i have to share this with you [13:56:45] so i can choose north korean localization in mediawiki [13:56:49] BUT! [13:57:09] it's called 한국어 (조선) [13:57:21] New review: Daniel Kinzler; "this isn't testing much yet, but it's better than nothing..." [mediawiki/extensions/Wikibase] (master); V: 1 C: 2; - https://gerrit.wikimedia.org/r/14027 [13:57:23] Change merged: Daniel Kinzler; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14027 [13:57:24] which is how north korea is called IN SOUTH KOREA [13:57:41] hahaha :) [13:57:42] nice one [13:57:54] the it to #mediawiki-i18n ? [13:57:56] just wait until the north korean wikipedia community finds out about this... [13:58:00] *take it [13:58:35] Jens_WMDE: yea, the large and powerful norht corean wikipedia community... [13:58:48] i know, right! [13:59:08] both of them! [13:59:24] JeroenDeDauw: on a related note: the interwiki table is still ignorant of the sites table stuff, right? so if i want to use an interwiki prefix, I'd have to put it into the interwiki table *and* the sites table? [14:00:01] JeroenDeDauw: can you turn the sites table into a view when creating the sites table? would that be sane? [14:00:20] or at least provide an .sql file for doing it. [14:00:46] hm, probably better, because it would drop any info already in that table. migrating that to the sites table would be tricky. [14:00:57] JeroenDeDauw: what do you think?... [14:01:43] Daniel_WMDE: what do you need the interwiki table for at all? [14:01:47] Just use the sites table [14:10:23] JeroenDeDauw: because it would be nice to be able to link to stuff from wikitext in a consistent way. [14:10:36] hi [14:10:49] JeroenDeDauw: and it would also be nice to be able to just use a Title object inside the SiteLink object, for constructing the URL and so forth [14:11:15] in that case, the only purpose of even having a SiteLink object instead of using Title directly would be extra stuff like badges [14:11:18] hi aude! [14:11:27] what's the status of things? [14:11:38] * aude was offline for a few days [14:11:38] snafu i guess ;) [14:11:43] Daniel_WMDE: keep in mind not all sites are MediaWikis [14:11:58] Daniel_WMDE: also, we do not need to tackle the inline linking now [14:12:07] Daniel_WMDE: the idea is that in the end it will all use the sites table [14:12:22] Daniel_WMDE: doing adittional work now to make it work while we don't need it yet seems wrong [14:12:25] JeroenDeDauw: for interwiki prefixes to work with Title, the target doesn't have to be a mediawiki. [14:12:30] but yes, we don't have to do this now [14:12:51] it would just make the implementation of SiteLink simpler, and make sure it's consistent with the handlign in wikitext [14:13:00] anyway [14:13:07] JeroenDeDauw: i'll do witout and put a todo in there [14:14:57] Daniel_WMDE: I'm actually not sure that we should use one object to represent "interwiki links" and "interlanguage links" [14:15:10] do links like [[google:weather]] work? [14:15:10] 10[1] 10http://meta.wikimedia.org/wiki/google:weather [14:15:11] Badges and stuff only apply to the later [14:15:15] heh [14:15:25] with the new site link stuff/ [14:15:39] aude: there is no new site link stuff yet [14:15:47] aude: we only have the site stuff [14:15:50] New patchset: Henning Snater; "hiding contentSub and mw-jump elements specifically on the item page removing the layout gap" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/14038 [14:15:50] oh, ok [14:15:52] which will replace the interwiki table [14:16:00] but it is not used by core at all yet [14:16:05] ok [14:16:06] neither is it by the client [14:16:12] This is what I'm working on now] [14:16:13] ok [14:16:17] oh, really? [14:16:19] JeroenDeDauw: my idea was to use an object to represent links in general (Title), and build the SiteLink handling on top of that [14:16:24] It's only used by the repo ATM [14:16:50] aude: actually, you can already do that kind of thing with the existing interwiki mechanism [14:16:53] i don't see why not [14:17:24] hm... [[meatball:wiki]] [14:17:24] 10[2] 10http://meta.wikimedia.org/wiki/meatball:wiki [14:17:26] Daniel_WMDE: Title has a lot of stuff in it we don't need, and a lot of MW specific stuff... [14:17:27] Daniel_WMDE: right [14:17:39] just want to make sure things don't break [14:17:40] ah, no magic redirects for that. [14:17:46] only works with local=1 i guess [14:25:38] Denny_WMDE1: actually there is one 404 that is used. but whatever [14:26:40] Denny_WMDE1: so, how about just making all of those styles part of the HTML instead (in a