[00:50:11] (03CR) 10Jeroen De Dauw: Refactor Fingerprint code out of EntityView (031 comment) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153958 (owner: 10Bene) [00:50:16] (03CR) 10Jeroen De Dauw: [C: 032] Refactor Fingerprint code out of EntityView [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153958 (owner: 10Bene) [01:02:23] [travis-ci] wikimedia/mediawiki-extensions-Wikibase/master/446fd61 : Jeroen De Dauw The build was broken. http://travis-ci.org/wikimedia/mediawiki-extensions-Wikibase/builds/32597812 [01:15:57] (03PS1) 10Hoo man: Fix FingerprintViewTest for non-English wikis [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154226 [01:16:02] JeroenDeDauw: ^ [01:16:16] or aude if still around [01:30:31] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154226 (owner: 10Hoo man) [01:30:48] Thiemo_WMDE: ^ [01:31:08] leaving now [07:58:15] Lydia_WMDE: Hi, are there any plans for making a DCAT-AP descritption for Wikidata? [07:59:03] Or rather, have it been discussed at all? [07:59:03] Ainali: what is that and what is it good for? ;-) [07:59:14] no [07:59:30] It is a standard for description of LOD datasets [07:59:41] ok [08:00:26] not a priority atm tbh [08:00:52] but if you want to work on it write down some info and send it to the mailing list for discussion [08:01:02] including what would need to be done [08:01:05] We are thinking on making an application to Swedish Innovation Agency to do an investigation wether it would be good to use it for us [08:01:45] We could probably get around €10000 for an investigation [08:01:52] i'd start with an email to the mailing list [08:01:54] heh [08:02:02] ok... [08:02:19] i have a few things that are imho more important that we could do with 10000 :D [08:02:25] I bet :) [08:02:28] anyway - need to go take a shower [08:02:30] back in a bit [08:21:16] (03CR) 10Thiemo Mättig (WMDE): [C: 032] Fix FingerprintViewTest for non-English wikis [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154226 (owner: 10Hoo man) [08:21:24] (03Merged) 10jenkins-bot: Fix FingerprintViewTest for non-English wikis [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154226 (owner: 10Hoo man) [08:37:13] [travis-ci] wikimedia/mediawiki-extensions-Wikibase/master/c2f0730 : Marius Hoch The build was fixed. http://travis-ci.org/wikimedia/mediawiki-extensions-Wikibase/builds/32616417 [09:00:00] (03CR) 10Thiemo Mättig (WMDE): "Whoo, what happened to this patch? I spend quite some time refactoring this code (when it was in EntityView). I was happy to see you left " (037 comments) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153958 (owner: 10Bene) [09:00:30] (03PS1) 10Thiemo Mättig (WMDE): Revert "Refactor Fingerprint code out of EntityView" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154241 [09:00:37] (03CR) 10jenkins-bot: [V: 04-1] Revert "Refactor Fingerprint code out of EntityView" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154241 (owner: 10Thiemo Mättig (WMDE)) [09:14:20] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154241 (owner: 10Thiemo Mättig (WMDE)) [09:14:26] test.wikidata is down. not showing items [09:25:19] not all pages. example: https://test.wikidata.org/wiki/Q344 I get Unknown column 'epp_redirect_target' in 'field list' in SqlEntityInfoBuilder::getPageInfoForType [10:00:36] (03PS1) 10WikidataBuilder: New Wikidata Build - 15/08/2014 10:00 [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/154244 [10:19:45] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/154244 (owner: 10WikidataBuilder) [10:21:27] (03PS1) 10Thiemo Mättig (WMDE): Fix SqlEntityInfoBuilder(Factory) constructor parameters [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154250 [10:23:47] (03PS2) 10Thiemo Mättig (WMDE): Revert "Refactor Fingerprint code out of EntityView" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154241 [10:24:31] aude: can I merge https://gerrit.wikimedia.org/r/#/c/154244/ ? [10:24:46] Tobi_WMDE_SWE: sure [10:25:21] aude: beta is still broken I guess? [10:25:25] yes [10:25:44] * aude has raged about finding logs for the errors [10:25:47] (03CR) 10Tobias Gritschacher: [C: 032] New Wikidata Build - 15/08/2014 10:00 [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/154244 (owner: 10WikidataBuilder) [10:27:32] [13WikibaseDataModel] 15Benestar comment on commit 14398774b: @JeroenDeDauw do you have any idea why the tests are failing here? Apparently some serialization does not work although I didn't touch the code at all. 02http://git.io/xE_XuA [10:34:26] (03Merged) 10jenkins-bot: New Wikidata Build - 15/08/2014 10:00 [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/154244 (owner: 10WikidataBuilder) [10:37:25] (03CR) 10Aude: [C: 031] "waiting for jenkins" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154250 (owner: 10Thiemo Mättig (WMDE)) [10:38:33] (03PS1) 10Aude: Fix SqlEntityInfoBuilder(Factory) constructor parameters [extensions/Wikibase] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154251 [10:43:50] (03PS1) 10Thiemo Mättig (WMDE): Refactor Fingerprint code out of EntityView [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154252 [10:47:01] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154250 (owner: 10Thiemo Mättig (WMDE)) [10:51:41] (03CR) 10Aude: [C: 032] Fix SqlEntityInfoBuilder(Factory) constructor parameters [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154250 (owner: 10Thiemo Mättig (WMDE)) [10:51:47] (03Merged) 10jenkins-bot: Fix SqlEntityInfoBuilder(Factory) constructor parameters [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154250 (owner: 10Thiemo Mättig (WMDE)) [11:00:08] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154241 (owner: 10Thiemo Mättig (WMDE)) [11:08:56] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154251 (owner: 10Aude) [11:17:55] (03CR) 10WikidataJenkins: "Build Failed" [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/154244 (owner: 10WikidataBuilder) [11:17:59] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154252 (owner: 10Thiemo Mättig (WMDE)) [11:28:39] (03CR) 10Tobias Gritschacher: [C: 032] Make hide/show changes toggle respect showExternalRecentChanges setting [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/148732 (https://bugzilla.wikimedia.org/68461) (owner: 10Aude) [11:28:47] (03Merged) 10jenkins-bot: Make hide/show changes toggle respect showExternalRecentChanges setting [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/148732 (https://bugzilla.wikimedia.org/68461) (owner: 10Aude) [12:12:23] (03CR) 10Bene: [C: 032] Revert "Refactor Fingerprint code out of EntityView" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154241 (owner: 10Thiemo Mättig (WMDE)) [12:12:33] (03Merged) 10jenkins-bot: Revert "Refactor Fingerprint code out of EntityView" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154241 (owner: 10Thiemo Mättig (WMDE)) [12:32:34] aude: \o/ https://test.wikidata.org/wiki/Q486 [12:32:46] Lydia_WMDE: our first badges are life --^ [12:32:53] though not really nice styled yet [12:33:09] \o/ [12:33:25] benestar: how do we configure them to make them nice? [12:33:34] we have the wikidata.org git repo there [12:36:01] aude: they should be nice then... [12:36:26] the setting is correct, the badges have the wb-badge-goodarticle class [12:39:17] (03PS1) 10Aude: Re-add $evilDataValueMap global, still used in DataModel 0.8.2 [extensions/Wikibase] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154259 (https://bugzilla.wikimedia.org/69542) [12:39:20] evil! [12:39:31] (03CR) 10Addshore: [C: 04-1] "So long as sitelinks stay part of wikibase (and this this repo) this script should stay part of wikibase (and thus this repo)" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/145765 (https://bugzilla.wikimedia.org/42207) (owner: 10Hoo man) [12:40:19] (03CR) 10Addshore: [C: 032] Fix phpdoc [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153697 (owner: 10Jeroen De Dauw) [12:40:27] (03Merged) 10jenkins-bot: Fix phpdoc [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153697 (owner: 10Jeroen De Dauw) [12:40:46] benestar: i felt so bad for the revert. now you merged it. ;-) [12:41:08] (03CR) 10Addshore: [C: 032] Remove dead line [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153703 (owner: 10Jeroen De Dauw) [12:41:08] urgh [12:41:10] aude: the styling is there but the default style overrides the styles in wikidata.org [12:41:12] aude: we might fix that by adding !important but that isn't nice :/ [12:41:17] (03Merged) 10jenkins-bot: Remove dead line [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153703 (owner: 10Jeroen De Dauw) [12:41:20] Thiemo_WMDE: there were obviously errors [12:41:27] * benestar should not submit patches after 1 am :P [12:41:42] i'm working on it currently. adding more tests first. will let you know. [12:41:52] Thiemo_WMDE: hey, I'm working on that, too [12:41:54] (03PS2) 10Addshore: Remove unused vars [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153711 (owner: 10Jeroen De Dauw) [12:42:01] please not conflicts ... [12:42:11] (03PS2) 10Addshore: Remove some presumably dead code [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153712 (owner: 10Jeroen De Dauw) [12:42:32] i already resubmitted your patch and i'm currently adding more tests. thats what i do. [12:42:39] (03PS1) 10Aude: Re-add $evilDataValueMap global, still used in DataModel 0.8.2 [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154260 (https://bugzilla.wikimedia.org/69542) [12:42:50] wow, Henning has a lot of patches up! [12:43:27] addshore: merge! [12:43:38] merge the evil patch [12:43:48] benestar: merge! :) [12:44:09] (03CR) 10Addshore: [C: 04-1] [wip] ApiConventionsTest (031 comment) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/151659 (https://bugzilla.wikimedia.org/67407) (owner: 10Lucie Kaffee) [12:44:37] (03Abandoned) 10Addshore: Jenkins job validation (DO NOT SUBMIT) [extensions/WikibaseSolr] - 10https://gerrit.wikimedia.org/r/152697 (owner: 10Hashar) [12:44:40] (03Abandoned) 10Addshore: Jenkins job validation (DO NOT SUBMIT) [extensions/WikibaseRepo] - 10https://gerrit.wikimedia.org/r/152696 (owner: 10Hashar) [12:44:44] (03Abandoned) 10Addshore: Jenkins job validation (DO NOT SUBMIT) [extensions/WikibaseLib] - 10https://gerrit.wikimedia.org/r/152694 (owner: 10Hashar) [12:44:47] (03Abandoned) 10Addshore: Jenkins job validation (DO NOT SUBMIT) [extensions/WikibaseClient] - 10https://gerrit.wikimedia.org/r/152693 (owner: 10Hashar) [12:44:52] Thiemo_WMDE: wtf??? [12:44:55] argh [12:45:27] Thiemo_WMDE: I'm also adding tests... [12:45:32] how can this lead to a good end? [12:45:40] hm... [12:45:43] sorry. [12:45:56] i will upload what i have and do something else. [12:46:02] Thiemo_WMDE: can we remerge my patch? [12:46:10] which? [12:46:16] the reverted one [12:46:18] (03CR) 10Aude: [C: 032] Fix SqlEntityInfoBuilder(Factory) constructor parameters [extensions/Wikibase] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154251 (owner: 10Aude) [12:46:24] (03Merged) 10jenkins-bot: Fix SqlEntityInfoBuilder(Factory) constructor parameters [extensions/Wikibase] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154251 (owner: 10Aude) [12:46:27] Thiemo_WMDE: so I can upload my patch set up to there [12:46:29] (03PS4) 10Addshore: Refactor MonthNameUnlocalizer [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/148122 (owner: 10Thiemo Mättig (WMDE)) [12:46:37] and then you can rebase your change if my patch gets merge [12:46:39] d [12:46:55] sorry, dont understand. let me quickly finish this and upload it. [12:47:04] omg [12:47:09] don't merge my patch yet [12:47:25] * aude needs to fix the ascii art [12:47:58] (03CR) 10Addshore: [C: 031] "Waiting on WDJenkins to run" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/148122 (owner: 10Thiemo Mättig (WMDE)) [12:49:09] (03PS2) 10Aude: Re-add $evilDataValueMap global, still used in DataModel 0.8.2 [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154260 (https://bugzilla.wikimedia.org/69542) [12:49:16] (03CR) 10Addshore: [C: 032] Fix class ref [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153710 (owner: 10Jeroen De Dauw) [12:49:25] (03Merged) 10jenkins-bot: Fix class ref [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153710 (owner: 10Jeroen De Dauw) [12:49:34] https://gerrit.wikimedia.org/r/#/c/154260/2/lib/WikibaseLib.php [12:49:59] (03CR) 10Addshore: [C: 032] Minor cleanup in SnaksFinder [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153698 (owner: 10Jeroen De Dauw) [12:50:09] (03Merged) 10jenkins-bot: Minor cleanup in SnaksFinder [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153698 (owner: 10Jeroen De Dauw) [12:50:24] (03PS2) 10Aude: Re-add $evilDataValueMap global, still used in DataModel 0.8.2 [extensions/Wikibase] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154259 (https://bugzilla.wikimedia.org/69542) [12:50:37] (03CR) 10Addshore: [C: 032] Improve phpdoc in ChangeHandlerTest and improve class ref [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153699 (owner: 10Jeroen De Dauw) [12:50:45] (03Merged) 10jenkins-bot: Improve phpdoc in ChangeHandlerTest and improve class ref [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153699 (owner: 10Jeroen De Dauw) [12:51:41] (03PS2) 10Addshore: Introducing ApiHelperFactory. [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/151598 (owner: 10Daniel Kinzler) [12:51:47] (03PS4) 10Addshore: Introducing MergeItemsInteractor [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/151145 (https://bugzilla.wikimedia.org/68730) (owner: 10Daniel Kinzler) [12:52:16] much spam :> [12:53:53] (03PS2) 10Thiemo Mättig (WMDE): Refactor Fingerprint code out of EntityView [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154252 [12:55:23] benestar: https://gerrit.wikimedia.org/r/#/c/154252/ I added only tests, did not changed the class (only cosmetic changes, you can ignore them). [12:57:15] Thiemo_WMDE: ok, I will just work on my patch [12:57:26] we can then rebase yours and apply your changes [12:57:40] although I've already rewritten my code again ;) [12:58:00] * benestar didn't expect you to fix his patches :P [13:00:48] Thiemo_WMDE: argh, I cannot push a patchset to the closed patch... [13:00:56] :O [13:01:02] remove the changeid from the commit message [13:01:20] addshore: will try [13:03:26] (03PS1) 10Bene: Refactor Fingerprint code out of EntityView [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154261 [13:03:33] addshore: thanks :) [13:03:38] Thiemo_WMDE: you may want to review --^ [13:03:39] no worries :) [13:05:01] https://test.wikidata.org/w/api.php?action=wbparsevalue&format=json&parser=monolingualtext&values=fd&options=%7B%22valuelang%22%3A%22%22%7D [13:05:08] [travis-ci] wikimedia/mediawiki-extensions-Wikibase/master/4f5ee96 : jenkins-bot The build has errored. http://travis-ci.org/wikimedia/mediawiki-extensions-Wikibase/builds/32631036 [13:05:40] (03PS3) 10Thiemo Mättig (WMDE): Re-add $evilDataValueMap global, still used in DataModel 0.8.2 [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154260 (https://bugzilla.wikimedia.org/69542) (owner: 10Aude) [13:08:19] (03CR) 10Tobias Gritschacher: [C: 04-1] "This test is testing all API modules, not just the Wikibase ones. As this test is in the Wikibase repository I think it should only check " [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/151659 (https://bugzilla.wikimedia.org/67407) (owner: 10Lucie Kaffee) [13:09:37] (03PS2) 10Bene: Refactor Fingerprint code out of EntityView [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154261 [13:09:57] I feel like [[User:ERDINC]] might not become an administrator [13:10:38] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154259 (https://bugzilla.wikimedia.org/69542) (owner: 10Aude) [13:11:41] wut? [13:11:43] http://snag.gy/dYGh0.jpg [13:11:45] aude: --^ [13:12:15] is that only me and my local testrepo? [13:12:16] benestar: that's what my evil patch fixes [13:12:22] oh, ok :D [13:37:05] aude: hey, unfouded accusation of being evil! You evil person! [13:40:14] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153711 (owner: 10Jeroen De Dauw) [13:43:35] [13WikibaseDataModel] 15JeroenDeDauw comment on commit 14398774b: I do not know why it is failing - I'm not seeing this locally. 02http://git.io/ckdvTA [13:46:20] [13WikibaseDataModel] 15Benestar comment on commit 14398774b: So maybe travis is broken? 02http://git.io/iiDYhA [13:46:47] (03PS4) 10Aude: Re-add $evilDataValueMap global, still used in DataModel 0.8.2 [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154260 (https://bugzilla.wikimedia.org/69542) [13:52:28] (03PS5) 10Aude: Re-add $evilDataValueMap global, still used in DataModel 0.8.2 [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154260 (https://bugzilla.wikimedia.org/69542) [13:53:06] Tobi_WMDE_SWE: merge https://gerrit.wikimedia.org/r/#/c/154260/ ! [13:53:43] aude: I wanted Thiemo_WMDE to confirm it, as on my system I don't see the issue [13:53:49] [13WikibaseDataModel] 15JeroenDeDauw created 06test (+1 new commit): 02http://git.io/FI3ZgA [13:53:49] 13WikibaseDataModel/06test 14af51157 15jeroendedauw: Test [13:53:49] I don't have this setting set [13:54:13] benestar: want to review ? [13:54:39] does https://gerrit.wikimedia.org/r/#/c/154260/ fix the invalid stuff [13:54:41] ? [13:54:45] aude: checking [13:54:50] thanks [13:55:02] benestar: I will have a look at the failing thing on 0.9.x [13:55:02] lol https://gerrit.wikimedia.org/r/#/c/154259/1..2/lib/WikibaseLib.php [13:55:02] * aude wants to be ready when greg is around [13:55:09] heh [13:55:10] JeroenDeDauw: thanks :) [13:55:18] :O [13:56:16] [travis-ci] wmde/WikibaseDataModel/test/af51157 : jeroendedauw The build failed. http://travis-ci.org/wmde/WikibaseDataModel/builds/32636270 [13:56:30] aude: heh... [13:56:34] is wdjenkins dead or something? seems to be taking a very long time :< [13:56:57] oh dear [13:57:03] addshore: no, someone just only rebased lot's of stuff ;) [13:57:14] Tobi_WMDE_SWE: hoo told me you could create another server running jenkins [13:57:23] wanna do? [13:57:25] aude: do you know what actually ends up invoking that code from data model? (talking about https://gerrit.wikimedia.org/r/#/c/154260/) [13:57:36] SnakObject [13:57:43] benestar: there is one, but when we tried last we failed to get servers to communicate [13:57:55] :/ [13:58:03] Tobi_WMDE_SWE: *cant wait to move back to wmf jenkins* ;p [13:58:07] Tobi_WMDE_SWE: and what about give the current server more power? [13:58:19] benestar: guess that would not help [13:58:21] or what addshore says if that worked [13:58:22] Superjenkins? [13:58:34] our tests suck [13:58:36] benestar: Tobi_WMDE_SWE that wouldnt really work as we are still limited to running 1 of each job at a time [13:58:39] mediawiki tests suck [13:58:48] https://github.com/wmde/WikibaseDataModel/blob/0.8.2/src/Snak/SnakObject.php#L169 [13:59:05] https://github.com/wmde/WikibaseDataModel/blob/0.8.2/src/Claim/Claim.php#L206 [13:59:07] unit tests generally should not take more than a fraction of a second [13:59:21] but since we ahve to write stuff into DB that takes time [13:59:23] aude: yes, but what code in Wikibase.git... [13:59:43] https://github.com/wmde/WikibaseDataModel/blob/0.8.2/src/Entity/Entity.php#L718 unstub claims [13:59:50] JeroenDeDauw: none [14:00:06] https://github.com/wmde/WikibaseDataModel/blob/0.8.2/src/Entity/Entity.php#L705 getClaims [14:00:17] or very indirect [14:00:24] in ways that are not present in 1.0 [14:00:51] $entity->getAllSnaks() [14:00:51] Right [14:00:52] (03CR) 10Addshore: [C: 032] Remove unused vars [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153711 (owner: 10Jeroen De Dauw) [14:00:57] I forgot about this unstubbing part [14:01:00] (03Merged) 10jenkins-bot: Remove unused vars [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153711 (owner: 10Jeroen De Dauw) [14:01:08] So that indeed makes it not possible to remove the global for now [14:01:13] sadly not [14:01:13] >_> evil spagethi code [14:01:27] aude: thanks for looking at this! [14:01:56] glad it was something easy [14:02:03] that will go away again soon [14:02:14] EntityView.php:$entityIds = $refFinder->findSnakLinks( $entity->getAllSnaks() ); [14:02:28] and a couple other places we get all snaks [14:02:42] [13WikibaseDataModel] 15JeroenDeDauw created 06rmtest (+1 new commit): 02http://git.io/XrixPw [14:02:42] 13WikibaseDataModel/06rmtest 1485836b9 15jeroendedauw: Remove test that no longer works with the latest dependencies [14:03:58] [13WikibaseDataModel] 15JeroenDeDauw opened pull request #141: Remove test that no longer works with the latest dependencies (060.9.x...06rmtest) 02http://git.io/hNjxAg [14:04:49] (03CR) 10Bene: [C: 032] "tested locally" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154260 (https://bugzilla.wikimedia.org/69542) (owner: 10Aude) [14:04:57] (03Merged) 10jenkins-bot: Re-add $evilDataValueMap global, still used in DataModel 0.8.2 [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154260 (https://bugzilla.wikimedia.org/69542) (owner: 10Aude) [14:05:00] [13WikibaseDataModel] 15JeroenDeDauw comment on commit 14398774b: The failure is also on 0.8.x. https://travis-ci.org/wmde/WikibaseDataModel/jobs/32636276... 02http://git.io/Aqv6bg [14:05:05] [travis-ci] wmde/WikibaseDataModel/rmtest/85836b9 : jeroendedauw The build passed. http://travis-ci.org/wmde/WikibaseDataModel/builds/32637091 [14:05:23] (03CR) 10Jeroen De Dauw: [C: 031] Re-add $evilDataValueMap global, still used in DataModel 0.8.2 [extensions/Wikibase] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154259 (https://bugzilla.wikimedia.org/69542) (owner: 10Aude) [14:05:39] (03CR) 10Jeroen De Dauw: "Can I +2 on deployment branches?" [extensions/Wikibase] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154259 (https://bugzilla.wikimedia.org/69542) (owner: 10Aude) [14:07:06] [13WikibaseDataModel] 15JeroenDeDauw 04deleted 06test at 14af51157: 02http://git.io/9DKJYg [14:07:45] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153712 (owner: 10Jeroen De Dauw) [14:07:58] benestar: https://github.com/wmde/WikibaseDataModel/pull/141 [14:09:13] [13WikibaseDataModel] 15JeroenDeDauw pushed 1 new commit to 06master: 02http://git.io/UmJwqg [14:09:13] 13WikibaseDataModel/06master 14d732945 15Jeroen De Dauw: Change now in 0.9.0 [14:10:10] [13WikibaseDataModel] 15Benestar pushed 1 new commit to 060.9.x: 02http://git.io/XMQITw [14:10:10] 13WikibaseDataModel/060.9.x 14ff744e9 15Bene*: Merge pull request #141 from wmde/rmtest... [14:10:21] [13WikibaseDataModel] 15Benestar 04deleted 06rmtest at 1485836b9: 02http://git.io/eWbX1A [14:10:34] JeroenDeDauw: do we have to write this into the release notes [14:10:36] ? [14:11:42] [13WikibaseDataModel] 15Benestar comment on commit 14398774b: Thanks, I've merged the pull request. 02http://git.io/UBbp6A [14:12:13] Thiemo_WMDE: did you already have a look into my new FingerprintView? [14:12:53] benestar: write what in the release notes? [14:12:54] [travis-ci] wmde/WikibaseDataModel/0.9.x/ff744e9 : Bene* The build was fixed. http://travis-ci.org/wmde/WikibaseDataModel/builds/32637704 [14:13:40] JeroenDeDauw: never mind, just realized it makes no sense to write test changes into the release notes [14:17:18] [13WikibaseDataModel] 15JeroenDeDauw created 06fpadd (+1 new commit): 02http://git.io/GVXraQ [14:17:18] 13WikibaseDataModel/06fpadd 1400f6c04 15jeroendedauw: Add hasLabel, hasDescription and hasAliasGroup to Fingerprint [14:17:24] benestar: You also wanted this? https://github.com/wmde/WikibaseDataModel/pull/142 [14:17:28] [13WikibaseDataModel] 15JeroenDeDauw opened pull request #142: Add hasLabel, hasDescription and hasAliasGroup to Fingerprint (060.9.x...06fpadd) 02http://git.io/yCc7bA [14:24:15] benestar: huh, so apparently the thing does not get rebased... [14:24:43] which ting? [14:24:46] *thing [14:24:52] https://github.com/wmde/WikibaseDataModel/pull/142 [14:24:57] This will go when it's merged [14:25:32] benestar: we will roll out the client part of badges the weeek after next, right? so one week after the repo part [14:25:43] [13WikibaseDataModel] 15JeroenDeDauw pushed 1 new commit to 06fpadd: 02http://git.io/A32nzw [14:25:43] 13WikibaseDataModel/06fpadd 147dab90d 15jeroendedauw: Merge branch '0.9.x' of github.com:wmde/WikibaseDataModel into 0.9.x [14:25:57] Lydia_WMDE: yes, should be ok [14:26:15] :( [14:26:15] yay [14:26:21] although we still have to fix sth [14:26:21] (03CR) 10WikidataJenkins: "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154260 (https://bugzilla.wikimedia.org/69542) (owner: 10Aude) [14:26:23] :P [14:26:23] benestar: is that also in the current configs? [14:26:29] what? [14:26:46] Lydia_WMDE: the icons are not displayed on repo because the default icons override the special icons [14:27:06] hah [14:27:08] ok [14:27:10] on my local machine it worked but apperently the order of how they get loaded is different on production [14:27:16] and we should not rely on any ordering [14:27:28] so for now we can just add a !important hack [14:27:42] k [14:27:44] but maybe there are better ways to do that? [14:27:53] probably [14:28:14] benestar: if that commit looks good to you, I can proceed with tagging 0.9 [14:28:35] * benestar is looking [14:29:00] * JeroenDeDauw will be back in an hour [14:29:19] Lunch, toilet, ... [14:29:21] (03CR) 10Aude: "@jeroen yes, please merge" [extensions/Wikibase] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154259 (https://bugzilla.wikimedia.org/69542) (owner: 10Aude) [14:29:27] JeroenDeDauw: already done ;) [14:29:27] [13WikibaseDataModel] 15Benestar closed pull request #142: Add hasLabel, hasDescription and hasAliasGroup to Fingerprint (060.9.x...06fpadd) 02http://git.io/yCc7bA [14:30:05] [travis-ci] wmde/WikibaseDataModel/fpadd/7dab90d : jeroendedauw The build has errored. http://travis-ci.org/wmde/WikibaseDataModel/builds/32639420 [14:30:22] wut? [14:34:42] Wut [14:34:45] Wat [14:34:52] Wot [14:35:11] Thiemo_WMDE: Lydia_WMDE https://bugzilla.wikimedia.org/show_bug.cgi?id=69608 [14:35:26] don't know if this is related to why i get no suggestions [14:39:22] https://test.wikidata.org/wiki/Property:P247 and https://test.wikidata.org/wiki/Property:P248 [14:39:25] same label [14:40:18] :O [14:40:44] evil anon editor https://test.wikidata.org/wiki/Special:Contributions/87.138.110.76 ;) [14:43:37] (03CR) 10Aude: [C: 032] Re-add $evilDataValueMap global, still used in DataModel 0.8.2 [extensions/Wikibase] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154259 (https://bugzilla.wikimedia.org/69542) (owner: 10Aude) [14:43:44] (03Merged) 10jenkins-bot: Re-add $evilDataValueMap global, still used in DataModel 0.8.2 [extensions/Wikibase] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154259 (https://bugzilla.wikimedia.org/69542) (owner: 10Aude) [14:44:30] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/148122 (owner: 10Thiemo Mättig (WMDE)) [14:49:18] aude: how can that happen? [14:49:26] did someone remove the constraints? [14:49:31] probably the slaves are not updated [14:49:46] it might be checking a slave or cache for a label and doesn't find it yet [14:49:50] benestar: it can happen when they are created at the same time like here [14:50:05] :/ [14:50:09] evil indeed [14:50:24] however, not that tragical because property creation is limited on wikidata itself [14:50:30] aude: should I delete one of them? [14:50:39] benestar: if you want [14:51:02] aude: well, if it doesn't harm I will just let it there [14:51:06] ok [14:51:16] however, I think it will break things when accessing a property by its label on client [14:51:23] not sure if that is already implemented though [14:51:26] might be good to delete one if we want people to tes stuff [14:51:27] test* [14:51:37] and not confuse folks [14:52:10] aude: is accessing by label already implemented? [14:52:22] yes [14:52:34] so what happends on testwikipedia? [14:52:42] and how does it work? [14:52:55] * aude tries [14:53:04] oh wait, can't [14:53:27] editing claims is broken until we deploy bug fixes [14:55:30] (03PS1) 10Aude: Update Wikidata, fix database error and display of snak values [extensions/Wikidata] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154274 [14:55:54] aude: ups [14:55:56] https://test.wikidata.org/wiki/Q413 [14:56:22] :( [14:56:34] * aude waits for jenkins then goes to find greg [14:56:40] aude: seems like almost everything is broken on testwikidata atm [14:56:56] yep [14:59:21] aude: and now entirely broken [14:59:26] MediaWiki internal error. [14:59:27] Exception caught inside exception handler. [14:59:29] Set $wgShowExceptionDetails = true; at the bottom of LocalSettings.php to show detailed debugging information. [14:59:35] white page ... [15:00:50] that's the db error [15:01:09] might depend on which apache server it is [15:02:22] (03CR) 10WikidataJenkins: "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154260 (https://bugzilla.wikimedia.org/69542) (owner: 10Aude) [15:03:06] Thiemo_WMDE: are you still at the office? [15:03:11] (03CR) 10Aude: [C: 032] Update Wikidata, fix database error and display of snak values [extensions/Wikidata] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154274 (owner: 10Aude) [15:03:32] benestar: yes. sorry, to many bugs, to many mails to write. [15:03:47] Thiemo_WMDE: ok, don't worry [15:04:15] just want to annoy you with my fingerprintview :P [15:11:54] (03CR) 10WikidataJenkins: "Build Successful" [extensions/Wikibase] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154259 (https://bugzilla.wikimedia.org/69542) (owner: 10Aude) [15:13:31] (03Merged) 10jenkins-bot: Update Wikidata, fix database error and display of snak values [extensions/Wikidata] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154274 (owner: 10Aude) [15:17:05] (03CR) 10WikidataJenkins: "Build Successful" [extensions/Wikidata] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154274 (owner: 10Aude) [15:20:45] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/151598 (owner: 10Daniel Kinzler) [15:22:16] (03PS1) 10Aude: Handle MonolingualTextParser errors for values with language not set [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154277 (https://bugzilla.wikimedia.org/69608) [15:23:06] (03PS1) 10Aude: Handle MonolingualTextParser errors for values with language not set [extensions/Wikibase] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154279 (https://bugzilla.wikimedia.org/69608) [15:23:34] Thiemo_WMDE: Tobi_WMDE_SWE ^ [15:32:07] [13WikibaseDataModel] 15JeroenDeDauw pushed 1 new commit to 06master: 02http://git.io/4-NZaQ [15:32:07] 13WikibaseDataModel/06master 140680b3e 15jeroendedauw: Add hasLabel, hasDescription and hasAliasGroup to Fingerprint... [15:36:30] [13WikibaseDataModel] 15JeroenDeDauw pushed 1 new commit to 060.9.x: 02http://git.io/KsioXg [15:36:30] 13WikibaseDataModel/060.9.x 14843c363 15jeroendedauw: 0.9 release [15:36:37] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/151145 (https://bugzilla.wikimedia.org/68730) (owner: 10Daniel Kinzler) [15:37:46] benestar|afk: can you do a sanity check on https://github.com/wmde/WikibaseDataModel/tree/0.9.x ? To me it looks ready for the 0.9 tag now [15:39:03] JeroenDeDauw: +1 [15:44:49] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154252 (owner: 10Thiemo Mättig (WMDE)) [15:45:07] [13WikidataBuilder] 15Benestar closed pull request #24: Add Wikidata.org extension to build (06master...06badges) 02http://git.io/pNC1rg [15:51:00] JeroenDeDauw: will there be cases possible where an entity has claims but no statements? [15:51:16] do we want to support such kind of things`? [15:51:50] [13WikibaseDataModel] 15JeroenDeDauw tagged 060.9.0 at 060.9.x: 02http://git.io/-Cn-0Q [15:52:31] benestar: I do not know if we will have such cases, though having code that prevents those is likely bad [15:52:42] benestar: for where are you asking about this? [15:53:06] JeroenDeDauw: I'm currently refactoring the ClaimsView [15:53:12] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154261 (owner: 10Bene) [15:53:42] so I'm not sure how we should handle statements inside of that [15:54:01] benestar: does it currently not handle statements? [15:54:13] Right now we do not have claims that are not statements... [15:54:27] benestar: did you see the thing about changing the inheritance of the two? [15:54:37] [travis-ci] wmde/WikibaseDataModel/0.9.0/843c363 : jeroendedauw The build passed. http://travis-ci.org/wmde/WikibaseDataModel/builds/32647220 [15:54:47] JeroenDeDauw: it currently handles statements by overriding a method in ItemView [15:54:56] so it only affects the section heading but anyways [15:55:07] this is kind of an issue we have to be aware of [15:55:15] meh [15:55:26] "handles statements by overriding a method in ItemView" [15:55:28] bad [15:56:16] Right now statements might be cliams, but that does not mean a list of statements is a list of claims [15:56:54] benestar: I want to change Statement, and not have it inherit Claim, but rather use composition [15:57:24] Which means it's not good to have a Claim type hint somewhere and feed it a statement, as that would need to be changed [15:58:57] JeroenDeDauw: well, this mainly affects ClaimHtmlGenerator which currently checks if the claim is a statement [15:59:25] but will therefore also affect the ClaimsView [15:59:32] because we maybe rather need a StatementsView [16:00:37] JeroenDeDauw: also it is not possible to typehint agains Claim[] ... [16:00:53] so why don't we use a ClaimList in items? [16:01:36] (03CR) 10WikidataJenkins: "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154260 (https://bugzilla.wikimedia.org/69542) (owner: 10Aude) [16:05:47] benestar: not entirely in php indeed, though you can have Claim[] in the phpdoc ofc [16:06:09] I'd not call that typehint though :P [16:06:20] or call it type*hint* [16:06:52] benestar: it's not very nice to type hint against ClaimList, since this is a concrete implementation [16:07:07] oh, thought it was an interface [16:07:11] Type hinting against Iterator would be better [16:07:15] Though same problem there [16:07:20] PHP does not have generics... [16:07:23] JeroenDeDauw: so is it decided to use composition rather than inheritance in Statements? [16:07:33] JeroenDeDauw: I think a future version will get them? [16:07:55] benestar: this has not been decided yet. It might turn out to be to much work to get rid of the inheritance [16:08:11] benestar: yeah maybe, but that does not help us, esp since WMF is still running PHP 5.3 for fucks sake [16:08:39] lol :P [16:08:43] benestar: also keep in mind that Item has a list of Statement, not of Claim [16:08:48] Which reminds me... [16:08:59] I know but that is all a bit smelly [16:09:57] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154261 (owner: 10Bene) [16:10:58] benestar: what is? [16:11:46] JeroenDeDauw: the inheritance between claims and statements [16:12:05] [13WikibaseDataModel] 15JeroenDeDauw created 06rmca (+1 new commit): 02http://git.io/y6VxEw [16:12:05] 13WikibaseDataModel/06rmca 14d971ec5 15jeroendedauw: Remove not used ClaimAggregate [16:12:14] benestar: ^ [16:12:18] because this code smell forces us to have inheritance overall the whole code base [16:13:49] benestar: yeah, this made me realize it's usefull to already have a getter that pretends there is composition going on [16:13:56] Will add this now already for DM 1.0 [16:14:30] :) [16:14:40] JeroenDeDauw: you evil murderer :P [16:14:55] killed 151 lines in one commit [16:15:00] [travis-ci] wmde/WikibaseDataModel/rmca/d971ec5 : jeroendedauw The build passed. http://travis-ci.org/wmde/WikibaseDataModel/builds/32649068 [16:16:46] JeroenDeDauw: btw I think https://github.com/wmde/WikibaseDataModel/blob/master/src/Entity/Entity.php#L457 should also be deprecated [16:16:55] because getClaims etc are deprecated [16:17:53] (03CR) 10WikidataJenkins: "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154260 (https://bugzilla.wikimedia.org/69542) (owner: 10Aude) [16:17:57] benestar: indeed, feel free to submit a patch! [16:18:06] ok :) [16:18:08] benestar: I suggest you do not modify the rel notes in the PR though [16:18:12] Else we'll have conflicts [16:18:16] Just do that afterwards [16:18:39] JeroenDeDauw: and then directly push to master? [16:19:26] benestar: better [16:19:26] [13WikibaseDataModel] 15JeroenDeDauw pushed 1 new commit to 06rmca: 02http://git.io/hf8L1Q [16:19:26] 13WikibaseDataModel/06rmca 14909a889 15jeroendedauw: Add getStatements to Item and deprecated getClaims [16:19:35] benestar: https://github.com/wmde/WikibaseDataModel/edit/master/RELEASE-NOTES.md [16:20:23] benestar: I added some lines now so I did not purely kill them: https://github.com/wmde/WikibaseDataModel/pull/143 [16:22:04] [13WikibaseDataModel] 15JeroenDeDauw created 06Statement (+1 new commit): 02http://git.io/dAPkLg [16:22:04] 13WikibaseDataModel/06Statement 14ff49e2c 15jeroendedauw: Make fields in Statement private [16:22:14] [13WikibaseDataModel] 15JeroenDeDauw opened pull request #144: Make fields in Statement private (06master...06Statement) 02http://git.io/VmTuMw [16:23:34] class StatementTest extends ClaimTest [16:23:36] :( [16:26:10] (03CR) 10WikidataJenkins: "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154260 (https://bugzilla.wikimedia.org/69542) (owner: 10Aude) [16:26:42] JeroenDeDauw: that's what I mean :/ [16:27:43] [13WikibaseDataModel] 15JeroenDeDauw created 06getclaim (+1 new commit): 02http://git.io/JN-LnA [16:27:43] 13WikibaseDataModel/06getclaim 143fd2807 15jeroendedauw: Add getClaim method to Statement... [16:27:51] benestar: then you must like this ^ [16:27:58] [13WikibaseDataModel] 15JeroenDeDauw opened pull request #145: Add getClaim method to Statement (06master...06getclaim) 02http://git.io/ssd3pA [16:28:15] JeroenDeDauw: <3 [16:31:16] [travis-ci] wmde/WikibaseDataModel/Statement/ff49e2c : jeroendedauw The build passed. http://travis-ci.org/wmde/WikibaseDataModel/builds/32649960 [16:33:55] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154277 (https://bugzilla.wikimedia.org/69608) (owner: 10Aude) [16:36:56] [travis-ci] wmde/WikibaseDataModel/getclaim/3fd2807 : jeroendedauw The build passed. http://travis-ci.org/wmde/WikibaseDataModel/builds/32650375 [16:37:50] [13WikibaseDataModel] 15Benestar created 06deprecate (+1 new commit): 02http://git.io/WtU1xQ [16:37:50] 13WikibaseDataModel/06deprecate 1436ad5c7 15bene: Deprecated Entity::newClaim and Entity::getAllSnaks [16:38:10] [13WikibaseDataModel] 15Benestar opened pull request #146: Deprecated Entity::newClaim and Entity::getAllSnaks (06master...06deprecate) 02http://git.io/98qhcQ [16:45:59] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] (mw1.24-wmf17) - 10https://gerrit.wikimedia.org/r/154279 (https://bugzilla.wikimedia.org/69608) (owner: 10Aude) [16:46:02] benestar: did you see this piece of awesome? http://www.bn2vs.com:8000/components [16:46:21] Lydia_WMDE: yu are now officially sad we did not do this earlier http://www.bn2vs.com:8000/components [16:46:32] no, whats that? [16:46:32] Even though it does not have a cornify button yet [16:46:51] benestar: it's the WIP version of our new awesome site [16:47:07] Right now this is on mediawiki.org: http://wikiba.se/ [16:47:24] benestar: you can add stuff here https://github.com/wmde/Wikiba.se [16:47:31] JeroenDeDauw: cool :) [16:47:34] but do we need this? :P [16:48:03] JeroenDeDauw: we need a footer for this [16:48:56] "need"? We don't die if we don't have it. We lose out on stuff if we don't though. [16:49:15] benestar: not sure how you'd add a footer [16:49:20] Jackob has been creating this thing [16:49:25] it's kinda cut off at the bottom [16:49:29] Though it just uses Sculpin [16:49:40] also such a fancy website should use HTML5 tags imo :) [16:49:53] benestar: file all of the things! https://github.com/wmde/Wikiba.se/issues [16:49:58] Or better yet, make PRs ;p [16:50:04] Now we can actually do such fancy stuff [16:50:21] On mediawiki.org we can't even get rid of the totally unrelated main menu [16:52:12] * benestar first refactors ClaimsView [16:52:25] benestar: merge all of the things? https://github.com/wmde/WikibaseDataModel/pulls/assigned/Benestar [16:52:54] [13WikibaseDataModel] 15JeroenDeDauw closed pull request #146: Deprecated Entity::newClaim and Entity::getAllSnaks (06master...06deprecate) 02http://git.io/98qhcQ [16:54:26] (03CR) 10Jeroen De Dauw: "Sires are a MediaWiki concept. Why does this script, which only deals with sites, need to be in Wikibase? Anything else that depends on th" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/145765 (https://bugzilla.wikimedia.org/42207) (owner: 10Hoo man) [16:57:28] JeroenDeDauw: Thiemo_WMDE will kill me :P [16:57:47] he told me to wait at least some hours before merging something [16:58:29] benestar: don't worry, I'll kill Thiemo_WMDE first [16:58:39] xD [16:59:23] [13WikibaseDataModel] 15Benestar pushed 1 new commit to 06master: 02http://git.io/VACQQw [16:59:23] 13WikibaseDataModel/06master 142d5cc9c 15Bene*: Merge pull request #144 from wmde/Statement... [16:59:27] benestar: that's a silly policy IMO. It slows you down and enourages both big commits and long commit chains [16:59:34] [13WikibaseDataModel] 15Benestar 04deleted 06Statement at 14ff49e2c: 02http://git.io/5qm9pw [16:59:45] this one was trivial enough that Thiemo_WMDE cannot complain :) [16:59:47] In some cases it's good to wait for review by specific people [17:00:04] Though if you wait on no one in particular for all commits, that's just silly [17:00:48] If one qualified person reviewed it, it's generally fine [17:01:15] And if it is not, then waiting for a few hours is a totally arbitrary measure that is not guaranteed to help at all [17:01:24] What if no one notices it in those hours? [17:01:33] It's much better to then poke the relevant people [17:01:38] [13WikibaseDataModel] 15Benestar comment on pull request #145 143fd2807: why not `return $this`? 02http://git.io/7T3nVA [17:02:06] Thiemo_WMDE: ^ proof you are a silly person! :) [17:04:10] [13WikibaseDataModel] 15Benestar comment on pull request #143 14909a889: renaming `$entity` to `$item` would help avoiding confusion 02http://git.io/zpuj6w [17:05:43] benestar: well, a lot of other stuff should also be changed in that test class... [17:06:10] JeroenDeDauw: so maybe I'll do it in a follow up [17:06:15] [13WikibaseDataModel] 15JeroenDeDauw pushed 1 new commit to 06rmca: 02http://git.io/PRWBsw [17:06:15] 13WikibaseDataModel/06rmca 14100e9e1 15jeroendedauw: Some naming imporvements [17:06:41] benestar: this comment makes no sense to me https://github.com/wmde/WikibaseDataModel/pull/145#discussion_r16302815 [17:06:51] benestar: no need, already done [17:07:18] oh [17:07:20] I get it [17:07:59] JeroenDeDauw: statement extends claim so we can just return ourselves [17:08:51] [13WikibaseDataModel] 15JeroenDeDauw comment on pull request #145 143fd2807: Because then it's still an instance of `Statement`, and some code right now might be doing `if ( $claim instanceof Statement )`. You are right that `return $this` will suffice for a lot of cases. In any case, this is a temporary implementation. 02http://git.io/T0KHXA [17:09:01] makes more sense imo because we want to return an existing object [17:09:21] benestar: why? [17:09:36] JeroenDeDauw: than we can modify the object and the changes get visible [17:10:01] or is a claim unmutable? [17:11:25] why doesn't PHP support typecasting? [17:11:36] otherwise we could just do return (Claim) $this; [17:11:40] and everything would be fine [17:11:48] benestar: right, I did consider the "by value" part [17:12:28] so that changes do *not* get visible in the statement itself? [17:12:48] That is the behaviour of the current implementation yes [17:12:57] I'm mostly concerned about this since it will likely change [17:13:05] When we change the thing to actually use composition [17:13:12] And this is a breaking change [17:13:24] So using what you suggested fixes that [17:13:39] Though I'm concerned about the instanceof checks part [17:13:41] ... [17:14:13] JeroenDeDauw: that's what I meant [17:14:32] Ok, I suspect just returning this will end up being less of a hassle in the future [17:14:34] so maybe we should just eliminate all instanceof Statement checks? [17:14:57] JeroenDeDauw: btw, getClaim is not used at the moment anyways so it cannot conflict with some old code [17:15:14] and for new code we have to take extra care and avoid any instanceof checks [17:15:57] benestar: I know there is no break now, but there would be one in the future [17:16:24] yeah, and therefore I'm in favour of return $this; [17:19:00] benestar: ^ [17:19:07] ugh [17:19:09] [13WikibaseDataModel] 15JeroenDeDauw 04force-pushed 06getclaim from 143fd2807 to 14e024b43: 02http://git.io/5nQLkg [17:19:09] 13WikibaseDataModel/06getclaim 14e024b43 15jeroendedauw: Add getClaim method to Statement... [17:19:11] ^ [17:20:24] benestar: so merge https://github.com/wmde/WikibaseDataModel/pull/143 ? [17:20:37] Then I can update the rel notes for your change which was already merged [17:23:18] [13WikibaseDataModel] 15Benestar comment on pull request #143 14100e9e1: since 1.0 02http://git.io/ETGpew [17:23:25] JeroenDeDauw: :P [17:23:27] --^ [17:24:36] [13WikibaseDataModel] 15Benestar comment on pull request #143 14100e9e1: also why did you kill the `@since 0.3` tag? 02http://git.io/--unVw [17:25:26] [13WikibaseDataModel] 15JeroenDeDauw comment on pull request #143 14100e9e1: ??... 02http://git.io/kcvkeA [17:25:27] benestar: you got confused? [17:25:36] [13WikibaseDataModel] 15Benestar pushed 1 new commit to 06master: 02http://git.io/eV0QmQ [17:25:36] 13WikibaseDataModel/06master 14488cfc1 15Bene*: Merge pull request #145 from wmde/getclaim... [17:26:42] Lydia_WMDE: ping [17:26:56] aude: meep [17:27:17] to enable redirects, i forgot that we also need to change a setting after we add a column [17:27:38] after begging greg for the bug fixes, i'm not sure about asking for that on a friday [17:28:10] instead, think we should enable it on test.wikidata on monday [17:28:24] then give them a week to test, and enable it on wikidata a week later, imho [17:28:46] the setting is to tell wikibase to use the column [17:29:11] if it is just about asking greg i will do that [17:29:17] hmmm [17:29:28] [13WikibaseDataModel] 15JeroenDeDauw pushed 1 new commit to 06master: 02http://git.io/OcH9EQ [17:29:28] 13WikibaseDataModel/06master 149ce2ba0 15Jeroen De Dauw: Update RELEASE-NOTES.md [17:29:29] you can [17:29:30] it'd be sad if we miss out on it just because of that imho [17:29:55] [13WikibaseDataModel] 15Benestar comment on pull request #143 14100e9e1: ...... 02http://git.io/zKJ4Fw [17:30:41] JeroenDeDauw: perhaps you got confused :P [17:30:45] will merge after dinner :) [17:30:50] [13WikibaseDataModel] 15JeroenDeDauw comment on pull request #143 14100e9e1: Meh! Screenshot or it didn't happen. Nowait.gif 02http://git.io/qAMmfg [17:30:51] benestar|afk:you misplaced the comment! [17:31:15] aude: asked [17:31:27] ok :) [17:33:08] [13WikibaseDataModel] 15JeroenDeDauw 04force-pushed 06rmca from 14100e9e1 to 14ff29b0f: 02http://git.io/Ena2JQ [17:33:08] 13WikibaseDataModel/06rmca 145406c6c 15jeroendedauw: Remove not used ClaimAggregate [17:33:08] 13WikibaseDataModel/06rmca 14a83d8db 15jeroendedauw: Add getStatements to Item and deprecated getClaims [17:33:08] 13WikibaseDataModel/06rmca 14ff29b0f 15jeroendedauw: Some naming imporvements [17:33:39] * JeroenDeDauw is off to the land of rts [17:35:38] (03PS2) 10Aude: Update / remove references to TermCache interface [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/152056 [17:35:56] easy chocolate ^ [17:40:37] aude: good to go :) [17:40:45] yay! [17:41:06] * aude wishes for a day between thursday and friday :) [17:41:25] to avoid these issues [17:41:50] JeroenDeDauw: i feel likr 70% drunk and still dont know what you are refering to. [17:46:51] column added [17:49:08] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/152056 (owner: 10Aude) [17:50:37] \o/ [17:52:10] updating config [17:54:11] Lydia_WMDE: https://gerrit.wikimedia.org/r/#/c/154292/1/wmf-config/Wikibase.php :) [17:54:46] :) [17:56:20] * aude blame Lydia_WMDE https://gerrit.wikimedia.org/r/#/c/154292/ [17:56:32] :P [17:56:38] that's what i'm here for right? [17:56:42] heh [17:57:58] doing [17:58:05] done :) [17:58:20] yaaay [17:58:40] * aude codes my bot to do redirects now [17:58:54] during the weekend.... [18:00:02] aude: sweet. ok to send out emails? [18:00:22] i think so [18:00:31] is there any documentation about redirects? [18:00:46] nope [18:00:53] i should probably add something to the email about that [18:01:02] that they can for now only be created viat the api? [18:01:04] only in https://test.wikidata.org/w/api.php [18:01:08] for wbcreateredirect [18:01:16] looks straightforward [18:01:31] * Lydia_WMDE goes and adds to email and then sends [18:01:35] k [18:01:38] * aude going home [18:01:46] laters :) [18:01:50] later! [18:01:54] and thanks for getting the test system up [18:02:01] thanks for asking gre.g [18:02:29] * aude really wish we deploy test.wikidata in the morning on thursdays so we have more time for bug fixes [18:02:32] before friday :) [18:02:42] definitely, yeah [18:02:50] doesn't work with the timezones :( [18:02:58] laters [18:03:02] hopefully getting better with next deployments when we have less stuff again [18:03:06] yep [18:03:25] what about no deploys on fridays :) [18:03:35] dennyvrandecic: shhhhhh :P [18:03:40] this was test.wikidata.org [18:04:03] * Lydia_WMDE goes and sends off emails [18:09:46] [13WikibaseDataModel] 15Benestar comment on pull request #143 14ff29b0f: y did you kill @since 0.3? I think it was useful at this place. 02http://git.io/4sBt3A [18:11:43] * aude is spammed! [18:15:44] * Lydia_WMDE hides [18:16:54] (03CR) 10Addshore: [C: 032] Refactor MonthNameUnlocalizer [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/148122 (owner: 10Thiemo Mättig (WMDE)) [18:17:02] (03Merged) 10jenkins-bot: Refactor MonthNameUnlocalizer [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/148122 (owner: 10Thiemo Mättig (WMDE)) [19:00:22] is there a page that shows when each language adopted wikidata, and their RFCs? [19:00:31] i know italian and hebrew and polish were first right? [19:00:38] are all languages using now? [19:08:31] Lydia_WMDE: most mailing lists (including wikitech-ambassadors) strip cc's from the headers when sending it out to everyone [19:08:48] legoktm: urgh seriously? [19:09:02] those people still get it, but others can't see the cc [19:09:25] also, YAY FOR BADGES :DDDD [19:09:31] notconfusing: yes all wikipedias. the news page linked on the main page of wikidata.org has the dates [19:09:36] legoktm: :D [19:09:41] https://test.wikidata.org/wiki/Special:SetSiteLink/Q296/enwiki <-- how do I unset a badge? [19:09:55] notconfusing: there were no real rfcs. just asking for the first ones [19:10:07] legoktm: deselect [19:10:10] and then save [19:10:27] oh, that's confusing [19:10:32] yeah not perfect yet [19:10:41] can there just be a third "None" option? [19:11:21] legoktm: well you can select several [19:11:31] that makes it a bit tricky [19:11:31] hm [19:11:53] oh interesting [19:12:00] how do i deselect? [19:12:46] dennyvrandecic: ctrl + click [19:13:01] oh [19:13:03] and on mobile? [19:13:04] same for selecting multiple badges [19:13:32] I wonder if we're going to tag all featured articles as good articles too, or just remove GA when it reaches FA. [19:13:37] dennyvrandecic: not sure if this is possible there tbh. this special page isn't the final solution [19:13:54] legoktm: the latter makes more sense to me ;-) [19:14:04] maybe at least write it down that you can use ctrl+click for deselect and multiselect [19:14:08] i didn't know that :) [19:14:13] good point [19:14:16] benestar: ^ [19:15:05] dennyvrandecic: in the end we want to be able to edit this on the item page directly. but hacking this in before the UI redesign was too painful and a waste of benestar's time [19:15:20] so this for now and then properly with the new UI [19:15:55] that's smart [19:16:33] Lydia_WMDE: we could also add checkboxes... [19:16:53] benestar: hmm wasn't there some reason why we didn't? [19:17:06] Lydia_WMDE: not sure ^^ [19:17:15] hmmmm [19:17:16] I think because it would look ugly with many badges [19:17:30] but atm we only have two so this wouldn't really matter [19:17:45] well i expect quite some requests right after roll-out [19:17:46] how many badges to we anticipate? [19:17:51] do we* [19:17:59] legoktm: 20ish maybe? [19:18:18] but i am sure people will come up with crazy stuff later [19:18:22] (03PS1) 10Gerrit Patch Uploader: Use consistently window.location.href instead of location.href [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154305 [19:18:24] (03CR) 10Gerrit Patch Uploader: "This commit was uploaded using the Gerrit Patch Uploader [1]." [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154305 (owner: 10Gerrit Patch Uploader) [19:19:43] Lydia_WMDE: but later there will be a nicer ui... [19:20:14] yeah [19:20:29] let's see what feedback we get over the weekend [19:20:43] and then check what to do next [19:21:06] i see wat u did there. [19:22:07] :P [19:23:58] Lydia_WMDE: I think the feedback will mostly be "häää" or "wtf" [19:24:09] multiselectfields are not very intuitive [19:24:58] hehe [19:25:44] let's see [19:26:04] i agree [19:26:23] but if we get requests for 15 more badges in the next days then we can't really go with checkboxes either [19:26:26] maybe just prepare a checkbox version for now, so it does not hold back deployment [19:26:42] oh you can. 15 checkboxes might be ugly, but still intuitive [19:27:00] hmmmm [19:27:02] since it is temporary anyway, why care too much about ugly? [19:27:12] true as well [19:27:21] benestar: want to prepare it? [19:27:26] you have the choice between confusing or ugly :) [19:27:42] tough! :P [19:27:47] aye [19:28:34] Lydia_WMDE: after I have killed EntityView maybe :P [19:28:42] :P [19:28:44] alright [19:28:54] well, I am going quite well :) [19:29:07] created a FingerprintView yesterday and a ClaimsView today [19:29:18] * benestar becomes faster :) [19:29:49] Lydia_WMDE: and that although I have to do code review no, too, because of your evilness :P [19:29:55] *now [19:30:06] :D [19:30:12] JeroenDeDauw annoyed me all the time to merge his patches ^^ [19:30:12] * Lydia_WMDE is an evil genius [19:30:20] <3 [19:30:41] which I did after annoying him all the time with comments on his patches :D [19:30:49] \o/ [19:30:58] so that we finally agreed on returning ourselves [19:31:15] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154305 (owner: 10Gerrit Patch Uploader) [19:31:16] function getClaim() { return $this; } [19:46:14] Just did this edit with my bot: https://test.wikidata.org/w/index.php?title=Q296&diff=7560&oldid=7554 [19:55:22] Anybody...like that are as smart or smarter than Lydia_WMDE (is there someone like that?)... http://i.snag.gy/zgUGt.jpg & http://i.snag.gy/vDCyH.jpg [19:55:55] Josve05a: :D [19:56:02] * Lydia_WMDE looks [19:56:11] (see, flattery works!) [19:56:16] :D [19:56:30] Josve05a: you mean the screwed up position of the edit link? [19:56:45] Yes, and the [cancel] [delete] [edit] too [19:56:46] and is this on wikidata.org or test.wikidata.org? [19:56:50] wikidata.org [19:56:56] ok [19:57:00] let me have a look there [19:57:10] I think I might be using a gadget...that causes this [19:57:20] You see those arrows at the right side [19:57:57] I had the browser zoomed out to 90% (looks better that way on all wikis), but I can confirm that it is on both 100% and 90% [19:58:31] Josve05a: can't reproduce here in chromium. Can you try logged out or in an anonymous session? [19:58:47] * Josve05a is logging out [19:59:18] no...then it is normal [19:59:34] that suggests it is indeed a gadget or similar [20:00:03] but which...urgh [20:00:09] yeah :( [20:00:19] iirc there is one to move links to another items? [20:00:29] that could be a candidate [20:00:49] I have like 60-75% of all gadgets turned on... [20:00:52] lol [20:00:54] ok... [20:01:41] Umm.... [20:01:54] * Josve05a is sceen-shotting [20:03:26] http://i.snag.gy/2lf0j.jpg [20:03:29] What is that? [20:04:40] Lydia_WMDE ^ [20:05:27] ah some admin recently added that gadget for everyone by default and it should not show up in their preferences [20:05:35] i don't remember what it does however [20:05:45] which gadget? [20:05:52] Urgh! [20:05:53] the one at the bottom of your screenshot [20:07:08] Sounds like MediaWiever...added by default... :P [20:07:33] Well, I can't find *my* bad gadget [20:12:00] Josve05a: do you have the move links one enabled? [20:12:52] I ctrl+f -ed "move". Not turned on any of those 2 gadgets [20:13:12] hmmmm [20:13:25] any user scripts? [20:13:50] I turned off "autoEdit". That seems to have fixed it [20:14:11] \o/ [20:14:35] Wooo! [20:15:10] But. who's script/gadget is that and how can it be fixed/warned that there is a problem? [20:23:12] (03PS1) 10Bene: [DNM] Refactor claims code out of EntityView [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154322 [20:28:17] Lydia_WMDE: "ah some admin recently added that gadget for everyone by default and it should not show up in their preferences" [20:28:19] wtf??? [20:34:22] Thiemo_WMDE: are you online? [20:35:52] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154322 (owner: 10Bene) [20:40:20] (03PS1) 10Bene: Decrease specificity in badge css selectors [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154325 [20:41:33] Lydia_WMDE: should I keep the code for the multiselectbox in the specialpage? [20:41:50] or did you decide to never use multiselectboxes again? [20:42:15] or did you go entirely? :P [20:42:37] benestar: i'd say kill it. if we need it again we can get it from history [20:43:04] ok, I've killed so much today already [20:43:17] one block of code more or less doesn't matter [20:43:24] * benestar is murderer no 1 [20:47:14] ^ ^ [20:48:05] Lydia_WMDE: having the checkboxes in one line or below each other? [20:48:15] below [20:50:04] Lydia_WMDE: looks a bit empty then... [20:50:27] screenshot? [20:50:57] Lydia_WMDE: both in one http://snag.gy/5yiVy.jpg [20:51:26] I like inline more [20:51:27] imho below is fine [20:51:31] :P [20:51:41] it seems clearer [20:51:55] other opinions? [20:53:01] Lydia_WMDE: both don't scale :/ http://snag.gy/rwum9.jpg [20:53:22] yeah [20:53:36] second one still seems clearer to me [20:53:59] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154325 (owner: 10Bene) [20:54:16] Lydia_WMDE: what about a two column layout? [20:54:34] that could work yeah [20:55:08] Lydia_WMDE: but is not widely supported :( [20:57:12] what happens in a not-supported browser? [20:57:22] do they just show it as one olumn? [20:57:26] column [20:59:22] [13WikibaseDataModel] 15JeroenDeDauw comment on pull request #143 14ff29b0f: I think you are confused by the diff. This method did not exist before, so how could I remove a tag from it?! 02http://git.io/qbXwXQ [20:59:31] Lydia_WMDE: http://snag.gy/YiuVw.jpg [20:59:52] yes! [21:00:01] hey :) [21:00:15] [13WikibaseDataModel] 15JeroenDeDauw comment on pull request #143 14ff29b0f: Or do you want me to not remove it from `getClaims`? I don't see why it would still need to be there. You should not use that method. 02http://git.io/vJNzQA [21:01:04] hey lazowik [21:01:06] "i feel likr 70% drunk and still dont know what you are refering to" -- Thiemo_WMDE [21:01:19] It's a Lydia_WMDE [21:01:26] Who is no longer in the office [21:01:28] muhahaha [21:02:34] it's a JeroenDeDauw! [21:02:42] -.- [21:02:49] benestar: I see you're feeling good? [21:03:05] https://www.youtube.com/watch?v=CmwRQqJsegw [21:03:13] lazowik: https://dl.dropboxusercontent.com/u/7313450/entropy/gif/aaaa.gif [21:04:06] JeroenDeDauw: we're visiting my parents friends [21:04:19] and they have ~6 kittens [21:04:37] <1 month old [21:05:27] so I can haz see real kittens! [21:06:21] lazowik: please to steal some and bring to the WMDE office [21:06:23] Lydia_WMDE: that is not css3 columns but simply elements besides each other [21:06:24] however, implementing this costs more effort [21:06:26] at least more than I can do within a week or so [21:06:47] lazowik: Der betreffende Nutzer hat das Video in deinem Land nicht zur Verfügung gestellt. [21:07:05] JeroenDeDauw: evil :O [21:07:15] benestar: sure :) [21:07:36] JeroenDeDauw: frankly, I'm quite close to Germany now [21:07:45] thought of visiting you guys :p [21:07:51] Lydia_WMDE: not quite sure what you are referring to [21:08:03] benestar: wat? I'm no of knowing german [21:08:17] benestar: just have two divs [21:08:27] benestar: as in "that's ok." [21:08:45] and put them one left, one right etc. [21:09:23] lazowik: but there are lots of evil floatings around in that crazy box [21:09:33] seems they have a big party there [21:09:38] CLEAR ALL OF THE FLOATS [21:09:39] :p [21:10:24] benestar: ah, right, I forgot that you have many videos blocked in Germany [21:10:43] How are the allowed bagdes determined now? Where is the list defined? On wiki? [21:10:48] JeroenDeDauw: http://snag.gy/sQCPa.jpg [21:11:01] maybe that makes clear what I'm wanting to tell you ... [21:11:17] JeroenDeDauw: config [21:11:24] Code RED: promote more adminbots or hurry the developers [21:11:46] benestar: mediawiki config, php file? [21:11:59] JeroenDeDauw: Wikibase config [21:12:05] yeah same thing [21:12:19] ok peeps, perapre for some aww [21:12:20] http://pl.tinypic.com/r/m7qs7c/8 [21:12:26] JeroenDeDauw: like that https://gerrit.wikimedia.org/r/#/c/149918/ [21:12:26] benestar: did you see my second comment? [21:12:32] just now [21:13:05] JeroenDeDauw: do we remove @since tags when we deprecate methods? [21:13:13] I generally do this [21:13:26] well, k [21:14:23] !merge | benestar [21:14:23] benestar: merge merge merge MERGEEEEEEEEEEEEE https://gerrit.wikimedia.org/r/#/c/$1/ [21:14:30] huh [21:14:34] why no aww? :( [21:14:54] (03PS2) 10Bene: Refactor claims code out of EntityView [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154322 [21:14:55] !m is merge merge merge MERGEEEEEEEEEEEEE [21:14:56] Key was added [21:15:03] !m | benestar [21:15:03] benestar: merge merge merge MERGEEEEEEEEEEEEE [21:15:15] JeroenDeDauw: not used anywhere in Wikibase.git or in WikibaseDataModel? [21:15:23] !m | wm-bot3 [21:15:23] wm-bot3: merge merge merge MERGEEEEEEEEEEEEE [21:15:46] !m is !m [21:15:47] You are not authorized to perform this, sorry [21:15:57] JeroenDeDauw: ^ would that cause a inf. loop? [21:16:02] !idk [21:16:04] no. [21:16:08] !foo is !bar [21:16:08] Key was added [21:16:12] !bar is !foo [21:16:12] Key was added [21:16:13] The bot can't see what it self says [21:16:14] !foo [21:16:14] !bar [21:16:17] :( [21:16:23] :(( [21:16:26] Josve05a: booooo! [21:16:29] hmm [21:16:30] del foo [21:16:33] wait a sec [21:16:33] !del foo [21:16:36] !foo del [21:16:37] Successfully removed foo [21:16:40] !bar del [21:16:41] Successfully removed bar [21:16:41] Just as !ping is !ping wouldn't be good... [21:16:45] stop spamming! [21:16:54] what? who is spamming! [21:16:59] gimme him! [21:17:08] Yeah Josve05a , how do you dare spam like this? [21:17:10] JeroenDeDauw: I'm not available during weekend so if I merge you have to fix :P [21:17:38] benestar: master of data model is not used by Wikibase.git... [21:17:52] JeroenDeDauw - I'm sorry.... ;( [21:17:54] And if you are talking about ClaimAggregate, it is not used bu either [21:18:22] [13WikibaseDataModel] 15Benestar 04deleted 06rmca at 14ff29b0f: 02http://git.io/7xMqow [21:18:27] JeroenDeDauw: here you go [21:18:41] lol, wrong order xD [21:19:02] benestar: well thank you good sir [21:19:17] JeroenDeDauw: somehow gh-wmde forgot to notify that I merged actually [21:20:46] (03PS1) 10Jeroen De Dauw: Use DataModel 0.9.x [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154339 [21:24:05] JeroenDeDauw: btw, there are only 7 failures left in http://wdjenkins.wmflabs.org/ci/job/wikibase-repo-tests/3148/console [21:25:14] benestar: fantastic! [21:25:25] there were 18 [21:27:14] JeroenDeDauw: shouldn't we delete the 0.8.x branch on WikibaseDataModel? [21:31:41] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154322 (owner: 10Bene) [21:40:01] benestar|cloud: it's not doing any harm... I suggest to remove it when we are sure we want no further 0.8.x releases [21:40:31] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154339 (owner: 10Jeroen De Dauw) [21:41:39] (03CR) 10Jeroen De Dauw: [C: 031] "Looks good. Did not check the details yet though." (032 comments) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154322 (owner: 10Bene) [21:41:55] (03PS3) 10Jeroen De Dauw: Remove some presumably dead code [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153712 [21:42:06] benestar|cloud: https://gerrit.wikimedia.org/r/#/c/154339/ [21:55:41] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/153712 (owner: 10Jeroen De Dauw) [23:03:17] (03CR) 10Jeroen De Dauw: [C: 032] Update / remove references to TermCache interface [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/152056 (owner: 10Aude) [23:04:33] (03CR) 10Jeroen De Dauw: [C: 032] Handle MonolingualTextParser errors for values with language not set [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/154277 (https://bugzilla.wikimedia.org/69608) (owner: 10Aude)