[00:06:42] TypeError: snak.getValue is not a function [00:06:47] dependencies again, I geuss [00:09:32] Lydia_WMDE: ceiling cat is already watching you, now answer the question! :<0 [00:11:02] " TODO: This whole file along with its resource loader module should be be removed" [00:12:04] aude: thx for putting important stuff on the deployment page for the announcement. will try to help in the future [00:13:05] I am sorry for being the first one to add a quantity value to Wikidata, but it was taking to long and I was curious if it works :) [00:13:05] aude: Around? [00:13:47] dennyvrandecic_: it's ok. you can have the honor ;-) [00:14:36] ui. why did it become a statement at the top? curious [00:14:39] Lydia_WMDE: That JS bug is a little hairy [00:14:45] Q922 [00:14:57] dennyvrandecic_: bug with ordering -.- [00:15:02] unrelated to quantities [00:15:06] oh, ok [00:15:11] we need to clean up the ordering mess [00:15:15] hoo: ? [00:15:33] Lydia_WMDE: The one dennyvrandecic_ reported earlier [00:15:51] Thought it was dependencies again, but this time it's not that easy [00:16:08] meh [00:16:18] i better stop poking around in Wikidata for now ;) [00:16:20] how widespread? [00:16:29] dennyvrandecic_: yes! all your fault ;-) [00:16:38] I know :) [00:17:09] Lydia_WMDE: I'm going to build a huge http://upload.wikimedia.org/wikipedia/commons/c/cb/Replica_catapult.jpg if you don't answe my question! [00:17:26] And I'll shoot kittens with it, which will be very angy when they land on your head [00:17:55] JeroenDeDauw: yes, you are allowed to enter over9000 as a quantity value. unfortunately, you cannot do it. [00:17:59] JeroenDeDauw: don't forget that i will see you latest on saturday in a crowd of 7000 geeks [00:18:04] :P [00:18:16] also what dennyvrandecic_ said [00:19:06] * hoo rages [00:20:09] debugging things I didn't write on the live site *check* [00:20:41] huh - what did dennyvrandecic_ say? [00:23:28] 01[13Geo01] 15JeroenDeDauw tagged 060.1.2 at 06master: 02http://git.io/AlV_JA [00:26:15] hah, got the bug! [00:27:07] JeroenDeDauw: 0:17 JeroenDeDauw: yes, you are allowed to enter over9000 as a quantity value. unfortunately, you cannot do it. [00:27:50] hoo: \o/ [00:28:12] Lydia_WMDE: Not yet the fix, though :P [00:28:24] finding it is a first step ;-) [00:28:28] an important one [00:28:31] JohnFLewis: ok, I'm blind, I thought Lydia_WMDE had typed that [00:28:37] Who invented novalue snaks in the first place? :D [00:28:50] * Lydia_WMDE hands JeroenDeDauw a pair of glasses [00:29:00] * Lydia_WMDE looks at dennyvrandecic_ and markus [00:29:04] Lydia = Denny... ofc! Succession of the position! [00:29:12] hoo: wht is the problem with thi snak? [00:29:52] yah. it was us [00:30:02] JeroenDeDauw: People assume it to have a value... [00:30:04] Lydia_WMDE: I'm to bussy constructing that catapult to read IRC you know [00:30:20] JeroenDeDauw: but but but! you have your answer [00:30:25] uhm.... it is call NoValueSnak... :D [00:30:31] hoo: its name implies it does not have one? [00:30:35] lol [00:30:38] hoo: are you complaining about the UI? [00:30:46] JeroenDeDauw: Yep :P [00:31:10] hoo: I have too in the past. Assuming Lydia_WMDE remembers what I said about the UI. [00:31:19] im away [00:31:29] lol [00:31:38] hoo: who needs the UI anyway, I suggest we ditch it and concentrate on the domain model. Much more fun [00:31:47] <3 [00:32:01] JeroenDeDauw: Agreed, get rid the UI and use api :D [00:32:06] * Lydia_WMDE points at a few thousand users who need it [00:32:42] Lydia_WMDE: And? We lose a few thousand users. As long as the devs are happy :p [00:33:44] JohnFLewis: i have slightly higher ambitions for wikidata ;-) [00:33:55] just slightly but still [00:34:01] :p [00:34:51] Lydia_WMDE: I think I have a fix, but can't tests, as my testwiki is broken :/ [00:35:12] Well I am going to get off and spam Wikidata in the morning :p cya [00:35:18] hoo: ok - great - maybe JeroenDeDauw can or wait for aude to come back? [00:35:25] night JohnFLewis [00:47:52] Lydia_WMDE: Hah :) [00:48:00] Looks like my fix is going to work [00:48:04] \o/ [00:48:09] Not having a owrking local wiki aint no fun :P [00:48:09] awesome [00:48:16] yeah :/ [00:48:41] (03PS1) 10Hoo man: Don't assume Snaks have a value in wikibase.ui.scrapeFormattedValues [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/110480 [00:48:51] JohnFLewis: to me the API is not much different from the API. Both are in the evil presentation layer [00:48:59] Lydia_WMDE: Anyone around to review etc.? [00:49:19] hoo: only aude later and JeroenDeDauw now i believe [00:49:20] I can get it deployed, but need it merged first [00:49:25] JeroenDeDauw: ^ [00:49:30] arrr [00:49:31] Hot fix... [00:51:01] (03CR) 10Jeroen De Dauw: [C: 04-1] "check that type == value. Else you have the same issue for somevalue snaks" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/110480 (owner: 10Hoo man) [00:51:18] The code in that file looks suspect [00:51:24] JeroenDeDauw: Some value snaks also don#t have getValue() ? mh [00:51:28] COuld use some refactoring [00:51:42] hoo: only value snaks have a value [00:51:45] JeroenDeDauw: You're right [00:51:54] Got that wrong in my head... [00:52:24] (03PS2) 10Hoo man: Don't assume Snaks have a value in wikibase.ui.scrapeFormattedValues [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/110480 [00:53:21] JeroenDeDauw: ^ [00:55:01] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/110480 (owner: 10Hoo man) [00:55:15] (03PS1) 10Hoo man: Don't assume Snaks have a value in wikibase.ui.scrapeFormattedValues [extensions/Wikibase] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110481 [00:56:37] JeroenDeDauw: Any further concerns? [00:58:19] * hoo would like to have that merge, make the branch and deploy... [01:00:19] Lydia_WMDE: Or do you want to merge? :P [01:00:27] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/110480 (owner: 10Hoo man) [01:00:49] hoo: i'd love to but i shouldn't merge stuff i can't really sign off on [01:01:23] Also understandable... [01:01:35] * hoo will stay around a bit more... doing some Scribunto fixes [01:03:22] :) [01:04:40] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110481 (owner: 10Hoo man) [01:08:16] (03CR) 10Jeroen De Dauw: [C: 032] Don't assume Snaks have a value in wikibase.ui.scrapeFormattedValues [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/110480 (owner: 10Hoo man) [01:10:50] (03CR) 10Hoo man: [C: 032 V: 032] "Already merged on master" [extensions/Wikibase] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110481 (owner: 10Hoo man) [01:12:23] back [01:12:30] aude: :) [01:12:32] do we need a new build? [01:12:58] aude: I'm on it [01:13:23] ok [01:13:37] aude: If I run composer update it also pulls in some datavalues changes :/ [01:13:43] we need build for master and the branch (same thing) [01:13:56] i guess tomorrow we use addshore's thing for beta [01:15:54] (03PS1) 10Hoo man: Add "Don't assume Snaks have a value in wikibase.ui.scrapeFormattedValues" [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110485 [01:15:56] (03CR) 10jenkins-bot: [V: 04-1] Add "Don't assume Snaks have a value in wikibase.ui.scrapeFormattedValues" [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110485 (owner: 10Hoo man) [01:15:59] aude: ^ [01:16:40] rebase conflict... wtf [01:16:49] ooh, it submitted against amster [01:17:18] hmm [01:17:30] [travis-ci] wikimedia/mediawiki-extensions-Wikibase/master/e7f19ea : Marius Hoch The build was broken. http://travis-ci.org/wikimedia/mediawiki-extensions-Wikibase/builds/17949076 [01:17:37] (03PS1) 10Hoo man: Update Wikibase [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110486 [01:17:51] aude: ^ [01:18:25] I was able to make it not pull a new datavalues ;) [01:18:34] ok [01:19:01] (03CR) 10Aude: [C: 032] Update Wikibase [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110486 (owner: 10Hoo man) [01:19:04] looks fine [01:19:55] (03PS2) 10Hoo man: Update Wikibase [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110485 [01:20:05] same against master ^ [01:20:10] k [01:20:37] (03CR) 10Aude: [C: 032] Update Wikibase [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110485 (owner: 10Hoo man) [01:22:08] * hoo waits for Jenkins... [01:22:31] ... [01:22:45] (03Merged) 10jenkins-bot: Update Wikibase [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110486 (owner: 10Hoo man) [01:22:51] ah, there we go [01:24:17] * Lydia_WMDE finally goes to bed now [01:24:21] :* [01:24:22] Lydia_WMDE: :) [01:24:26] I should so also :P [01:24:33] Awake now... yay jetlag [01:24:39] aude: https://gerrit.wikimedia.org/r/110488 [01:24:47] hoo: go to bed even if you lie there awake [01:24:50] ;-) [01:25:00] otherwise you'll not get back to normal for way too long [01:25:11] too quick [01:25:21] aude: ... [01:25:29] Sorry, didn't know you were on that also [01:25:52] :) [01:26:20] now to find someone to deploy it [01:26:25] Yep :/ [01:26:41] We really need our shell thingies sorted completely ... [01:27:54] Reedy: ^ [01:27:54] (03Merged) 10jenkins-bot: Update Wikibase [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110485 (owner: 10Hoo man) [01:34:56] aude: Reedy probably isn't around... want to poke someone else? [01:35:25] yeah [01:35:35] seems rather quiet [01:39:12] * aude goes back to reddit [01:40:35] aude: fyi, localization update also is broken [01:40:45] in what way? [01:40:46] that's why we have those broken data type string [01:40:55] VE broke it [01:41:03] submodules... [01:41:08] it's not us... but the updater can't run through atm [01:41:17] mmm [01:41:29] http://p.defau.lt/?iO7VNWKrhZmOJwAVPgi9UA [01:42:11] and translate? [01:42:17] yes [01:42:21] sorry forgot taht [01:47:15] Away for now... [01:47:35] Hope you find someone to deploy before users go crazy :P [01:48:09] ok [01:48:52] I WANT MY TRANSLATED MESSAGES FIXED [01:48:53] :P [01:49:09] :) [01:56:52] looks like the problem is with i18n json files [02:04:58] (03PS1) 10Aude: include extension entry points instead of i18n files in config [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110494 [02:08:31] (03PS1) 10Aude: include extension entry points instead of i18n files in config [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110497 [02:08:33] (03CR) 10jenkins-bot: [V: 04-1] include extension entry points instead of i18n files in config [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110497 (owner: 10Aude) [02:08:53] (03CR) 10Aude: [C: 032] "should be fine, shall try on beta" [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110494 (owner: 10Aude) [02:10:07] (03PS2) 10Aude: Include extension entry points instead of i18n files in config [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110497 [02:11:28] (03Merged) 10jenkins-bot: include extension entry points instead of i18n files in config [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110494 (owner: 10Aude) [02:16:39] back [02:17:11] aude: Deployed yet? [02:18:03] no [02:18:36] but figured out that data values / types is not in the i18n files [02:18:41] aude: Shall I amend my patch to ship the above? [02:18:52] if both my patches merge, yes [02:19:20] (03CR) 10Aude: [C: 032] Include extension entry points instead of i18n files in config [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110497 (owner: 10Aude) [02:19:29] i can try localisation on beta [02:21:56] (03Merged) 10jenkins-bot: Include extension entry points instead of i18n files in config [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110497 (owner: 10Aude) [02:25:38] think the change is ok [02:25:59] aude: Tested on beta? [02:26:12] much as i can [02:26:26] caches are fresh so it's not rebuilding everything [02:26:36] but what it generates looks sane [02:28:08] aude: How do you rebuild the cache after all? Just using the maint. script? [02:28:39] sudo -u mwdeploy mw-update-l10n --verbose [02:28:41] on labs [02:28:53] ah :) [02:28:58] production might be slightly different [02:29:16] I'll check everything twice before running anything on production at first :P [02:29:24] I don't want to be that guy ... [02:29:29] it makes ExtensionMessages-master.php [02:29:57] result of mergeMessageFileList.php [02:30:13] should list everything... the one in production is missing data values, on labs it's there [02:31:09] probably because $wgExtensionEntryPointListFiles actually needs to have entry points, not just list i18n files (although sometimes that also is ok) [02:32:47] inconsistencies are fun -.- [02:32:53] yep [02:33:06] missing messages however don't break things fatally [02:33:13] messages are just missing... [02:33:33] having things listed in extension messages that actually can't be found can cause major issues [02:36:33] yep... [02:55:01] seems like no one is around :( [02:56:14] aude: I tried Tim... guess he's around... not sure about time though [02:56:25] ori maybe [02:59:56] Why not :P [03:01:50] aude: Going to bed now... would nice to get this deployed until European morning... before the majority of our users notices it :P [03:02:20] :/ [03:03:53] Sucks that even hot fixes take hours to deploy if no one is really around [03:04:23] At this time of the day probably only Tim is around, cause he's in Australia :P [03:04:51] he does it \o/ [03:05:19] yay [03:18:29] aude: Off now... you can verify the JS fix on the Item for "Germany" later on [03:18:44] if you can add a new statement we fixed it, if not *grbml* [03:19:00] ok [05:44:53] https://www.wikidata.org/wiki/Q6581072 (female) is now showing me the 'or' language label & description at the top [06:58:12] jayaway: yes - we try to offer you some more languages ULS thinks you might speak [06:58:17] add a babel box to your user page [06:58:25] then it will take those languages [06:59:23] anyone up for updating the news section on the main page? [07:11:46] Lydia_WMDE: I think it has a caching problem [07:12:11] because it was showing me someone elses prefer languages in 'In other languages' [07:12:49] I had to do a ?action=purge to get rid of the previous editors preferences [07:29:55] interesting [07:30:01] need to investigate [08:29:21] [13WikibaseDataModelSerialization] 15Tpt opened pull request #5: Adds beggining of Claim serialization (06master...06claim) 02http://git.io/F3dnuA [09:01:08] moin [09:01:16] hm, no Henning? [09:01:33] Lydia_WMDE: ich seh einen JS fehler auf https://www.wikidata.org/wiki/Q38 [09:01:48] in FF: TypeError: snak.getValue is not a function [09:02:23] Adrian_WMDE, Lydia_WMDE: is this known, or shall i file a bug? [09:10:24] Hi DanielK_WMDE [09:10:34] I'm the only one in office [09:11:12] again, im seeing the wrong language here : https://www.wikidata.org/wiki/Q36774 [09:11:30] should be English, Indonesian. Im seeing German as the default [09:12:50] Adrian_WMDE: everyone else is off to FOSDEM?... [09:12:57] Or off to bed [09:12:59] Adrian_WMDE: can you confirm the JS error? do you know about it? [09:13:21] * DanielK_WMDE wants to take a nap too - 6:30 is not a good time to get up. [09:13:25] DanielK_WMDE: I don't see a JS error in Iceweasel [09:13:33] 6:30 is the only time to get up [09:13:45] hehe. having kids is so much fun. [09:14:02] Adrian_WMDE: on Q38? [09:14:12] Chromium has an error for me [09:14:19] NoValueSnak::getType [09:15:09] Adrian_WMDE: that error is different from the one I reported, but simmilar: some code seems to assume all Snaks are Value snaks. [09:15:25] only ValueSnaks havew getValue and getType. SomeValueSnak and NoValueSnak doN#t. [09:15:55] i'll report it [09:15:56] DanielK_WMDE: Was there a deployment in the last few days? [09:16:30] AH, sure [09:16:34] It's Quantities [09:16:38] I'm still out of the loop, but since Lydia announced that we now have Quantities live, i'd say yes - yesterday. [09:16:50] right [09:18:23] DanielK_WMDE: I can fix the code, but I don't know the process works after that [09:18:36] *don't know how the process works [09:20:27] Adrian_WMDE: once the patch is merged, you backport (cherry-pick) it to the current deployment branch. Then ask Reedy or Chad to deploy it. Usually, Katie or Tobi do this after testing. [09:20:45] Ok [09:21:29] Adrian_WMDE: NoValueSnak::getType is PHP syntax - is this really a JS error? [09:22:22] DanielK_WMDE: Yeah, that's just me using a non-JS syntax [09:22:31] For talking to you [09:23:08] ah ok. [09:23:13] reproduced it now. [09:24:14] (03PS6) 10Adrian Lang: Support formatting commons media links via API [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/109638 [09:24:16] (03PS1) 10Adrian Lang: Support data types and output formats in API-based formatting [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/110521 [09:25:14] https://bugzilla.wikimedia.org/show_bug.cgi?id=60670 [09:25:28] this is critical, as it renders the item impossible to edit. [09:26:05] I see "" on https://www.wikidata.org/wiki/Q3181407 [09:27:59] DanielK_WMDE: I'm fixing it right now [09:28:28] Reedy, aude: you around? I'm a bit out of the loop - when did we roll out new code to wikidata.org? Did wgCacheEpoch get pocked? It should have been... [09:29:25] jayvdb: quick fix is to create that message on site. but please report this on bugzilla, so we add the message for translation on translatewiki. [09:30:59] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/109638 (owner: 10Adrian Lang) [09:31:08] jayvdb: hm... the message should be effectively empty, so it should probably be defined as "-" (nasty hack, that) [09:31:19] will do that now [09:33:12] hm... maybe i'm confused, and the key just changed, and it'S a parser cache issue.... grr... [09:33:19] DanielK_WMDE: there was a deployment yesterday late [09:33:32] and indeed there are some prolems with i18n cache [09:33:45] also with the parser cache, it seems [09:33:47] hm [09:34:32] jayvdb: purging the page should fix the "wrong language" issue. should only be needed once per page. would of course be better to fix it centrally. [09:35:20] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/110521 (owner: 10Adrian Lang) [09:37:09] DanielK_WMDE: http://tools.wmflabs.org/wm-bot/logs/index.php?start=01%2F30%2F2014&end=01%2F31%2F2014&display=%23wikidata [09:37:26] starts around 2014-01-30 23:11:47 [09:38:08] lazowik: thank you [09:39:09] 2014-01-30 15:54:02 got cache epoch setting [09:39:16] so it was at least on the radar [09:39:51] jayvdb: please keep reporting issues with apges showing the wrong language [09:40:06] collect that info somewhere in addition to posing here [09:40:10] thanks! [09:40:53] afk for a bit [09:57:39] hoi has one new property for a quantity been created ? [09:57:46] just one suffices [09:58:46] addshore: JeroenDeDauw: mediawiki/core coverage is much nicer https://integration.wikimedia.org/cover/mediawiki-core/master/php/ [09:59:03] made possible by Jan Zerebecki change https://gerrit.wikimedia.org/r/#/c/90584/ [10:02:04] * hoo waves [10:06:18] hey hoo [10:06:35] hi DanielK_WMDE... just saw your bug report [10:06:47] I fixed that tonight, not sure why it's still there [10:07:22] maybe it's broken in more moduls... [10:07:38] mh [10:07:42] hoo: It's definitely broken where I'm editing now [10:07:49] scrapeFormattedValues [10:07:50] ok ok [10:08:04] 03:37 logmsgbot: tstarling started scap: mostly no-op, did a reset to revert the wikidata changes [10:08:07] what? [10:08:25] aude: ^ [10:08:27] damn [10:08:42] DanielK_WMDE: We had to wait ages for Tim to deploy and I got to bed at 4am [10:08:47] seems like something went wrong after [10:08:52] but I was no longer around [10:09:09] Adrian_WMDE: My patch isn't live [10:09:20] hoo: It's not in master, either [10:09:46] Adrian_WMDE: the patch is in master [10:09:49] or do you mean the problem? [10:10:01] hoo: The problem I'm fixing is in master [10:10:02] I fixed it in master and for the branch tonigh [10:10:06] doh [10:10:25] You didn't fix all of it :D [10:10:47] Adrian_WMDE: yikes [10:10:51] What do you think about using snak instanceof wb.PropertyValueSnak instead of getType()? [10:10:52] which file is that? [10:11:20] scrapeFormattedValues [10:11:29] instanceof in JS can be buggy in older IE depending on where the data comes from, so I rather wanted to paly safe [10:11:33] repo/resources/wikibase.ui.scrapeFormattedValues.js [10:11:36] that one I fixed [10:11:37] Yeah [10:11:45] The mainSnak on the top [10:11:50] claim.getMainSnak().getValue(), [10:11:53] ok [10:11:56] Just saw it [10:12:47] (03PS1) 10Adrian Lang: Don't assume Snaks have a value in wikibase.ui.scrapeFormattedValues once more [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/110526 [10:12:59] GerardM-: yes, HDI [10:13:11] hoo: ^ [10:13:14] :) [10:13:17] What is HDI ? [10:13:23] https://www.wikidata.org/wiki/Property:P1081 [10:13:32] GerardM-: ^ [10:13:53] (03CR) 10Hoo man: [C: 032] Don't assume Snaks have a value in wikibase.ui.scrapeFormattedValues once more [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/110526 (owner: 10Adrian Lang) [10:14:00] Adrian_WMDE: thinking about it, all snaks should have getType() and also something like getDataType(). Only getValue() is really problematic. [10:14:02] Human Development Index [10:14:03] (03PS1) 10Hoo man: Don't assume Snaks have a value in wikibase.ui.scrapeFormattedValues once more [extensions/Wikibase] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110527 [10:14:16] (03CR) 10Hoo man: [C: 032] Don't assume Snaks have a value in wikibase.ui.scrapeFormattedValues once more [extensions/Wikibase] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110527 (owner: 10Hoo man) [10:14:17] (03Merged) 10jenkins-bot: Don't assume Snaks have a value in wikibase.ui.scrapeFormattedValues once more [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/110526 (owner: 10Adrian Lang) [10:14:22] will make a new build sonnish [10:14:24] * soonish [10:14:33] Adrian_WMDE: I also prefer instanceof checks to checks against constains. getType() is really for serialization and localization stuff. [10:14:39] (03Merged) 10jenkins-bot: Don't assume Snaks have a value in wikibase.ui.scrapeFormattedValues once more [extensions/Wikibase] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110527 (owner: 10Hoo man) [10:14:39] But I need to find out why tim reverted tonight [10:14:54] I wonder what exactly tim reverted, and why exactly [10:15:02] hoo: indeed [10:15:24] hoo: he should have sent mail about it :/ [10:15:30] https://gerrit.wikimedia.org/r/#/c/110507/ [10:16:00] hwo did that even happen [10:16:59] Wrong data-model version? [10:17:11] https://gerrit.wikimedia.org/r/110449 [10:17:13] doh [10:17:58] Tim is never ever going to deploy for us again :( [10:18:12] Already had to beg him hard to actually do that as he was the only one still around [10:18:45] hoo: DanielK_WMDE: again, IRC logs to the rescue [10:18:46] http://paste.tstarling.com/p/bhUBQv.html [10:18:53] http://tools.wmflabs.org/wm-bot/logs/index.php?start=01%2F30%2F2014&end=01%2F31%2F2014&display=%23wikimedia-operations [10:19:08] hoo: Did that commit just nuke the classmap? [10:19:11] @ 2014-01-31 03:31:51 [10:19:15] Adrian_WMDE: Not yours [10:19:21] Will sort this [10:19:21] (03CR) 10WikidataJenkins: "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/110526 (owner: 10Adrian Lang) [10:19:36] hoo: No, aude's [10:19:55] it apparently did... and addshore didn't notice [10:19:56] [travis-ci] wikimedia/mediawiki-extensions-Wikibase/master/9234c4c : Adrian Lang The build was fixed. http://travis-ci.org/wikimedia/mediawiki-extensions-Wikibase/builds/17966813 [10:20:36] oh, you got that arleady [10:20:50] ah crap [10:20:59] hoo: Well, it's just a big red bar :D [10:21:46] Adrian_WMDE: :P Didn't plan to actually do all that again today... deployment was hard enough yesterday already [10:21:58] especially as we counld hardly find someone to deploy [10:23:05] »I think every time someone asks me to deploy a really easy change because it's really really easy, it turns out to be fantastically complicated « :D [10:23:39] (03CR) 10WikidataJenkins: "Build Successful" [extensions/Wikibase] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110527 (owner: 10Hoo man) [10:23:48] Adrian_WMDE: that's life for you. [10:24:40] (03CR) 10Daniel Kinzler: "should there really be a composer.lock file in this commit?..." [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110449 (owner: 10Aude) [10:25:17] hoo: --^ [10:25:30] should be in gitignore i guess [10:25:51] err, or not? maybe i'm misunderstanding what this file does? [10:26:35] DanielK_WMDE: No, we actually want it [10:26:46] DanielK_WMDE: Maybe we commit that stuff on the deployment branches … [10:26:54] the whole thing is broken [10:27:01] don't use build.sh for doing builds... [10:27:04] -.- [10:27:18] You have to be very careful [10:27:42] if not for builds, what *can* it be used for?... [10:28:04] DanielK_WMDE: build.sh? for updating to a new master :P [10:28:13] but not for minor updates for deploys [10:28:29] http://ultimategerardm.blogspot.nl/2014/01/wikidata-day-pete-seeger-was-born.html calendard support using four queries happening simultaneously [10:28:32] better rename it, then [10:28:58] DanielK_WMDE: meh, we wont use that for the next branch anyway [10:29:06] and nobody except aude and /me touches it really [10:29:35] (03PS1) 10Hoo man: Fix autoloader [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110528 [10:29:39] hmhm [10:29:57] don't merge [10:30:09] will probably manually pull it onto beta [10:32:28] ah, it's not even broken on beta [10:33:18] (03Abandoned) 10Hoo man: Fix autoloader [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110528 (owner: 10Hoo man) [10:33:23] (03PS3) 10Adrian Lang: Use API for formatting Wikibase item links [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/110151 [10:38:01] wtf [10:38:55] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/110151 (owner: 10Adrian Lang) [10:40:37] ah, I think I know [10:41:29] ewk... that hting is fragile as glas [10:41:33] nobody touch it :D [10:42:00] (03PS1) 10Hoo man: Update Wikibase and fix autoloader [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110531 [10:42:02] (03CR) 10jenkins-bot: [V: 04-1] Update Wikibase and fix autoloader [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110531 (owner: 10Hoo man) [10:42:23] woops, submitted it against the wrong branch [10:42:55] (03Abandoned) 10Hoo man: Update Wikibase and fix autoloader [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110531 (owner: 10Hoo man) [10:43:22] hoo, it's just great to read this IRC log here. Like a really slow train crash. [10:43:22] (03PS1) 10Hoo man: Update Wikibase and fix autoloader [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110532 [10:44:09] Adrian_WMDE: Trust me, I know what I'm doing [10:44:10] mostly [10:45:51] (03PS1) 10Hoo man: Update Wikibase [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110533 [10:47:44] (03PS2) 10Hoo man: Update Wikibase and fix autoloader [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110533 [10:48:06] (03CR) 10Hoo man: [C: 032] Update Wikibase and fix autoloader [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110533 (owner: 10Hoo man) [10:48:10] Let's try that on beta... [10:52:56] (03Merged) 10jenkins-bot: Update Wikibase and fix autoloader [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110533 (owner: 10Hoo man) [11:08:40] DanielK_WMDE: Want to approve https://gerrit.wikimedia.org/r/110532 [11:09:05] I'll get that deployed then [11:10:37] GerardM-: very interesting [11:11:05] :) the calendar thing pyb ? [11:11:15] GerardM-: yes [11:11:34] (03PS11) 10Adrian Lang: Replace wb.fetchedEntities with wb.store.EntityStore [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/107554 [11:11:59] (03CR) 10jenkins-bot: [V: 04-1] Replace wb.fetchedEntities with wb.store.EntityStore [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/107554 (owner: 10Adrian Lang) [11:17:18] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/107554 (owner: 10Adrian Lang) [11:19:57] (03PS12) 10Adrian Lang: Replace wb.fetchedEntities with wb.store.EntityStore [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/107554 [11:20:32] DanielK_WMDE: Afraid of that? I don't really want to self merge something for production.. [11:24:03] http://tools.wmflabs.org/reasonator/test/?q=Q213 Reasonator supports quantities [11:24:59] (03CR) 10Daniel Kinzler: [C: 031] "The code looks fine, but I'm not an expert on the UI stuff. Also, needs selenium or qunit testing." [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110532 (owner: 10Hoo man) [11:25:22] DanielK_WMDE: This is only about the build, not the code itself [11:25:29] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/107554 (owner: 10Adrian Lang) [11:25:34] the UI change is already par of Wikibase [11:27:18] but not integration-tested for the branch, right? [11:27:31] i don't even know how i would verify that the build itself is ok [11:27:42] DanielK_WMDE: Install it on a wiki, but nobody does that [11:27:58] Just look whether it looks sane :P [11:28:09] that's how we did it yesterday... [11:28:24] see where that got us ;) [11:28:39] hoo: that patch adds a whole bunch of stuff to the classmap. why is that? [11:28:40] DanielK_WMDE: :P That wasn't my fault... didn't review that one [11:28:57] DanielK_WMDE: Because I used -o on composer dump-autoload [11:29:02] optimize, that is [11:29:08] to have the full class map in there [11:29:36] that will also fix the breakage Tim stumbled upon yesterday [11:36:15] hoo: we really need full testing for this stuff :/ [11:36:33] DanielK_WMDE: Yeah :/ [11:37:23] Don't we have a labs instance where we could test that with Selenium or so? [11:38:07] on betalabs, yes [11:38:19] that's the whole thing why beta is there, right? [11:38:24] http://wikidata.beta.wmflabs.org/wiki/Q8042 [11:38:27] no idea what the status is. Tobi_WMDE_mobi or hashar might know [11:38:28] Yes and no [11:38:35] Beta runs master [11:38:54] hi [11:39:00] hey :) [11:39:03] then it should be tested on test.* [11:39:14] if beta runs master, it should be called alpha :) [11:39:20] true [11:39:28] hashar: Any chance I can run a non-master on beta to test something? [11:39:36] nop [11:39:44] ok [11:39:45] beta uses the master branch of core and extensions [11:39:49] and does a reset --hard [11:39:53] to make sure there is no live hack :/ [11:39:57] k [11:40:16] maybe one day we will find a way to deploy custom code [11:40:20] DanielK_WMDE: Beta is currently running with master which has almost the same auto loader [11:40:23] such as using a 'staging' branch [11:40:37] and attempt to merge that 'staging' branch on top of master [11:40:43] so I guess we're good to go [11:40:49] hashar: mh [11:41:22] hashar: It would be good enough if I could change the submodule version like we have in production [11:41:34] we should be able to test it on our own labs instance. [11:41:45] DanielK_WMDE: Don't have access to tht, sorry [11:41:46] Tobi_WMDE_mobi: should we revive wikidata-test-repo.wikimedia.de for this? [11:41:56] * aude waves [11:42:00] hey aude [11:42:04] aude :) [11:42:12] DanielK_WMDE: we have wikidata-dev.wmflabs.org [11:42:15] aude: merge merge merge https://gerrit.wikimedia.org/r/110532 [11:42:15] i think the problem is maybe localisation update needs Wikidata.php [11:42:28] autoload.php is there [11:42:29] aude: can you check this bug fix? https://gerrit.wikimedia.org/r/#/c/110532/1 [11:42:31] aude: You broke the autoloader because you not used -p [11:42:34] * -o [11:42:44] aude: there seems to be no good way to test this build [11:42:50] sorry, 146738500 [11:42:54] and that then relid on psr-4 loading for DataModel [11:43:01] but that doesn't work in production [11:43:07] * relied [11:43:26] DanielK_WMDE: jenkins will test [11:43:30] meehh. http://wikidata-dev-repo.wmflabs.org/wiki/Main_Page [11:43:31] 2) can test on beta [11:43:48] aude: beta runs master, we have just been over that with hashar [11:43:51] aude: Already have that on beta (master, though... but that's nearly identical) [11:44:12] Tobi_WMDE_mobi: Can you get it to run there? [11:44:22] aude: jenkins doesn't seem to run any meaningful tests... or will it only do that for the gateway job? [11:44:35] it does testextensions... yes [11:44:39] DanielK_WMDE: It runs the phpunits [11:44:41] I'm not sure what is currently deployed there [11:44:51] Tobi_WMDE_mobi: /srv/mediawiki/extensions/Wikibase/repo/includes/specials/SpecialNewEntity.php [11:44:52] aude knows [11:44:53] the old wikibase [11:44:55] it's trivial to have Wikidata installed [11:45:01] yep [11:45:03] aude: i didn't think testextensions includes wikibase repo. does it? [11:45:12] DanielK_WMDE: Over there it does [11:45:12] hashar: --^ [11:45:21] over where? [11:45:27] it's like a totally different Extension [11:45:33] inside of the Wikdata extension [11:45:52] hm, right, i guess [11:46:00] without classs map, it still works [11:46:04] not ideal [11:46:20] why does that not work in production? [11:46:20] problem i think is maybe autoload.php missing [11:46:26] it does work [11:46:35] just not in loalisation update apprently [11:46:38] apparently* [11:46:41] orrr [11:46:55] worked fine for wikibase etc. [11:46:59] not for data values [11:47:06] bit odd [11:47:20] the WMF Jenkins does not run any tests nowadays [11:47:23] just linting [11:47:41] *sigh* [11:47:54] am i the only one who is utterly confused about what gets tested when and where and why? [11:48:13] https://integration.wikimedia.org/ci/job/mwext-Wikidata-testextensions-master/114/console [11:48:15] Tobi_WMDE_mobi: would be nice to have an overview of that somewhere... [11:48:18] looks like test to me [11:48:20] tests* [11:48:39] might be just wikibase, but should do all ideally [11:48:43] DanielK_WMDE: hashar's confusing Wikidata and Wikbiase [11:48:58] easily confusing [11:49:56] yep [11:50:34] (03CR) 10Aude: [C: 032] Update Wikibase and fix autoloader [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110532 (owner: 10Hoo man) [11:50:38] alright, tried it [11:50:43] a potential issue is that the Wikidata changes made on wmf branches are tested using MediaWiki master branch [11:50:51] when it should use the mediawiki wmf branch [11:51:05] hashar: that's always an issue, though usually not a problem [11:51:09] hashar: not that critical [11:51:15] the problem always was wikibase branch, data values master [11:51:21] with the build, that's eliminated [11:51:22] aude: Will you or shall I update core wmf11? [11:51:24] * wmf12 [11:51:27] hoo: go ahead [11:51:32] :) [11:51:43] it's good ot tackle the two issues (js , i18n) separately [11:51:50] this patch should be fine [11:53:06] (03Merged) 10jenkins-bot: Update Wikibase and fix autoloader [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110532 (owner: 10Hoo man) [11:54:37] next thing is to try to reproduce the localisation cache issue on beta [11:54:45] see if including Wikidata.php helps [11:56:11] aude: I tried that before, but I can't sudo -u mwdeploy [11:56:20] probably all we are missing is http://pastie.org/8685261 (data types) [11:56:51] bad, but something we can live with maybe a couple hours [11:57:08] Sorry, user hoo is not allowed to execute '/usr/local/bin/mw-update-l10n --verbose' as mwdeploy on i-00000390.pmtpa.wmflabs. [11:57:09] * aude needs to pack, eat etc. and then see if still have time (or time at the airport later) [11:57:24] hoo: i did try and the cache was fresh [11:57:34] it says i can use --force [11:57:41] but didn't like that param [11:58:06] on beta it's fine: http://wikidata.beta.wmflabs.org/wiki/Property:P496 [11:58:11] ooo [11:58:31] Reedy: https://gerrit.wikimedia.org/r/110536 [11:58:36] localisation runs as the first wiki in wikiversions [11:58:47] on beta, that might be aawiki (which has wikibase) [11:58:55] on production it might be aawikibooks [11:59:15] then more important to get the extension list right to load everything that wikibooks otherwise does not have [11:59:40] in beta you can specify --wiki enwikibooks or something [11:59:56] aude: Well, I can't at all [12:00:07] can't sudo into mwdeploy [12:00:16] maybe hashar [12:00:51] let me give you mwdeploy access [12:00:56] that is a mess [12:01:52] hashar: i always have to sudo as root [12:01:57] and then sudo as mwdeploy [12:01:59] seems wrong [12:02:05] ah yeah [12:02:42] alias mwdeploy='sudo su --login --shell /bin/bash mwdeploy' [12:02:44] that is what I have [12:02:53] ok [12:03:01] https://wikitech.wikimedia.org/wiki/Special:NovaSudoer does not let us grant sudo rights for system user accounts [12:03:02] :( [12:04:29] going via root it works [12:04:39] didn't even think I had root there :P [12:04:56] hoo: if you have nda, then think ou can have [12:04:59] you* [12:05:04] I do :P [12:05:10] there might be ssl cert [12:05:15] s and such that are private [12:06:53] mh... maybe we would ahve to add aawikibooks or so to test that [12:07:32] mw-update-i10n can't be given a wiki to use [12:10:38] mwExtVerDbSets=$($BINDIR/mwversionsinuse --extended --withdb) || die [12:10:54] hoo: you can try with something else [12:11:07] hoo@deployment-bastion:/a/common$ mwversionsinuse --extended --withdb [12:11:07] master=aawiki [12:11:14] you can also do parts of the script like rebuildLocalisationCache.php [12:11:24] aude: What does taht give in production? ...=aawikibooks or so? [12:11:30] aawikibooks [12:11:34] ok [12:11:40] * aude broke that before [12:11:46] because of that [12:13:34] Warning: Class 'Wikibase\DataModel\SiteLink' not found in /data/project/apache/common-local/php-master/extensions/Wikidata/vendor/wikibase/data-model/WikibaseDataModel.php on line 69 [12:13:52] when running on enwikibooks on beta [12:15:42] ah, that's because of the class_alias calls in there [12:16:00] l10n update is maybe not loading the autoloader [12:19:13] aude: ^ [12:54:30] hoo: did that patch of yours already get dployed? [12:58:38] addshore: Nope [13:04:36] yay [13:05:45] (03PS1) 10Hoo man: Add Wikidata.php to the extension list [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110544 [13:05:55] aude: DanielK_WMDE: ^ [13:06:16] (03PS1) 10Hoo man: Add Wikidata.php to the extension list [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110545 [13:07:43] addshore: ^ [14:14:32] addshore: can you explain to me why hoo's patch is needed? i'm still confused about the localization cache stuff. [14:18:55] DanielK_WMDE: I dont fully understand the localisation cache stuff either :/ [15:59:42] DanielK_WMDE: it just needs require_once( __DIR__ . '/vendor/autoload.php' ); [16:00:15] but.... i think maybe we need a different entry point for this purpose [16:01:38] nope, looks fine [16:02:18] (03CR) 10Aude: [C: 032] Add Wikidata.php to the extension list [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110544 (owner: 10Hoo man) [16:04:53] (03Merged) 10jenkins-bot: Add Wikidata.php to the extension list [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110544 (owner: 10Hoo man) [16:13:50] aude: why do we need to include the top level entry point *and* all the separate entry points for the libraries/extensions? [16:16:09] it might not be needed but then we need to set the switch variables all to true [16:16:28] it's like enabling client + repo at same time [16:19:14] so require_once( _DIR__ . '/path/to/client' ) and repo and autoload.php [16:24:36] (03PS1) 10Aude: add entry point specific for localisation cache update [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110585 [16:24:38] (03CR) 10jenkins-bot: [V: 04-1] add entry point specific for localisation cache update [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110585 (owner: 10Aude) [16:24:42] rage [16:25:48] (03PS2) 10Aude: add entry point specific for localisation cache update [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110585 [16:26:16] hoo|away: ^ what do you think? [16:35:29] [13WikibaseDataModelSerialization] 15JeroenDeDauw comment on pull request #5 144a1611e: Not quite a unit test if you pass in an actual SnakSerializer and have real snak serializations in the provider below. The test itself is perhaps not bad, though it is a component test rather than a unit one. I suggest stubbing out the SnakSerializer. 02http://git.io/xL6wFQ [16:36:02] [13WikibaseDataModelSerialization] 15JeroenDeDauw comment on pull request #5 144a1611e: Ping @addshore. Did you not recently change some stuff to the current implementation that is not reflected here yet? 02http://git.io/ts0KfA [16:59:05] back in a few hours [18:34:50] (03CR) 10Hoo man: [C: 032] add entry point specific for localisation cache update [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110585 (owner: 10Aude) [18:37:27] (03Merged) 10jenkins-bot: add entry point specific for localisation cache update [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/110585 (owner: 10Aude) [19:12:39] aude: Ping me when online [20:04:51] [13WikibaseDataModelSerialization] 15Tpt comment on pull request #5 14d4f71b8: Fixed 02http://git.io/acm_TQ [20:12:09] somehow item pages with many statements dont work properly anymore. links to other items are missing and i cannot edit those pages. [20:12:55] for example Q72 [20:13:29] under Ubuntu with firefox and chromium [20:15:24] uew: Known [20:15:39] We've fixed that yesterday, but didn't yet manage to deploy the fix [20:15:52] ah ok [20:16:28] DanielK_WMDE: Around? [20:33:28] (03CR) 10Jhall: [C: 031] cucumber.yml is no longer needed [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/110357 (owner: 10Zfilipin) [20:44:11] hoo: back but don't have great internet connection [20:44:21] aude: Ok [20:44:30] might disappear any time [20:44:30] I merged your change and tested it on beta [20:44:33] ok [20:44:33] yikes [20:44:45] Why do we have extension credits in there? [20:44:54] so it appears in special version [20:45:15] don't know if it's important for localisation... esp it's not translated yet [20:45:16] but that's only supposed to load for i10n update, right? [20:45:28] ok [20:45:29] we might want it translated [20:45:47] might be complicated though and not worth it [20:46:14] (03CR) 10Hoo man: [C: 032 V: 032] Add Wikidata.php to the extension list [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110545 (owner: 10Hoo man) [20:46:17] aude: k [20:46:27] I just want to get stuff deployed soon [20:46:32] yes [20:46:45] that merge above is only a lazy way to resolve the rebase conflict for the cherry pick :P [20:46:52] ok [20:47:29] aude: Or shall I not backport that entry point change to the Wikidata extension branch? [20:47:35] Both worked fine on beta [20:47:50] and I don't really care, just want to have something we can have deployed [20:47:58] worth trying and i think is ok [20:48:16] hmmm [20:48:25] (03PS1) 10Hoo man: add entry point specific for localisation cache update [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110626 [20:48:39] i do wonder if repo gets loaded w/o settings [20:48:48] if that can cause a problem (doubt it) [20:49:26] aude: I had no problem running the l10n update for both enwikibooks and aawiki on beta... just had to hack the mw10n script slightly [20:49:36] ok, good [20:49:38] * mw-update-l10n [20:49:56] (03CR) 10Hoo man: [C: 032] "Per IRC discussion with aude" [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110626 (owner: 10Hoo man) [20:50:22] Once that is in, I'll renew the core branch patch and then I'll try to find someone to deploy :) [20:50:30] ok [20:50:37] People are complaining all day, so I finally want this in.. [20:50:44] +1 [20:51:17] Tim's revert yesterday was a bit overcautious, he could have just ignored the Warnings [20:51:32] those were just in the l10n updater ... :/ [20:51:49] too many warnings [20:52:04] Well, all from the same source [20:52:32] take no chance with localisation [20:52:42] mh? [20:52:52] can break things in nasty ways [20:53:11] then he has to fix it [20:53:13] Sure, but that error couldn't :P But of course Tim didn't know that [20:53:26] doesn't matrter [20:53:29] scary enough [20:53:36] that's just because we do things in an entry point which we shouldn't, probably [20:53:51] (03Merged) 10jenkins-bot: add entry point specific for localisation cache update [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110626 (owner: 10Hoo man) [20:53:55] yep [20:54:04] there we go :) [20:54:21] yay [20:56:04] maybe Reedy is around [20:56:22] Quite late [20:56:29] Ori's active in -operations [20:56:48] sure, ask him [21:00:47] aude: I'm not sure I told you, but I autoload optimized branches this afternoon to get the autoloaders into a consistent state [21:01:06] Daniel was worried and so was I, so we rather wanted to have them complete instead of relying on PSR-4 stuff [21:01:10] ok [21:01:28] agree [21:01:40] if you can find the js patch, great [21:01:43] might take a while for me [21:02:12] aude: https://gerrit.wikimedia.org/r/#/c/110532/1/extensions/Wikibase/repo/resources/wikibase.ui.scrapeFormattedValues.js [21:02:25] got them [21:02:32] Adrian did that... basically what I've already done yesterday, but I missed that spot [21:02:47] k [21:03:01] autoload ones, probably can't find easily [21:03:15] aude: Same change set [21:03:37] ok [21:03:42] hoo: did you see https://gerrit.wikimedia.org/r/#/c/109785/2 ? [21:04:13] * hoo looks [21:04:22] neat :) [21:05:58] (03PS1) 10Ori.livneh: Revert "Update Wikibase and fix autoloader" [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110630 [21:06:18] (03CR) 10Hoo man: [C: 04-2] Revert "Update Wikibase and fix autoloader" [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110630 (owner: 10Ori.livneh) [21:08:35] (03PS2) 10Ori.livneh: Revert "Update Wikibase and fix autoloader" [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110630 [21:13:50] (03PS3) 10Ori.livneh: Partially revert I4e395394a, keeping only JavaScript part [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110630 [21:14:23] (03CR) 10Hoo man: [C: 032] "Yay for restoring past flaws \o/" [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110630 (owner: 10Ori.livneh) [21:19:03] (03Merged) 10jenkins-bot: Partially revert I4e395394a, keeping only JavaScript part [extensions/Wikidata] (mw1.23-wmf11) - 10https://gerrit.wikimedia.org/r/110630 (owner: 10Ori.livneh) [21:31:37] * aude gotta run to other terminal in airport [21:32:06] later [21:32:52] cu [21:48:21] aude: Just for the log: The JS fixes work (verified them) and so the l10n ones [21:49:12] (03CR) 10Jeroen De Dauw: Support formatting commons media links via API (031 comment) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/109638 (owner: 10Adrian Lang) [21:49:39] + do [21:50:06] (03CR) 10Jeroen De Dauw: [C: 04-1] "Feel free to ignore the -1 if I'm misunderstanding the ID thing" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/109638 (owner: 10Adrian Lang) [21:51:47] (03CR) 10Jeroen De Dauw: [C: 031] Add i18n support in MwTimeIsoFormatter [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/109685 (owner: 10Addshore)