[05:59:43] aude: I can't add years to P585, it will automatically change to January xxxx. [06:00:25] https://www.dropbox.com/s/loze6n9s919hffe/Screenshot%202014-04-09%2008.00.24.png [06:02:50] :/ [06:51:48] i cant add claims to https://www.wikidata.org/wiki/Q16666 - the [edit] and [add] links are not working [06:52:53] ive noticed it happening on a few other items too, but wasnt trying to edit so I ignored it hoping it would voodoo fix itself ;-) [07:04:36] (03CR) 10Tobias Gritschacher: "Just some further suggestions for the name of the setting. What about:" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124650 (owner: 10Thiemo Mättig (WMDE)) [07:06:40] jayvdb: there is a JS error there. we will look into it [07:14:40] Thx Lydia_WMDE [07:16:39] btw, a use case for qualifiers. I use a Wikidata clone in a business setting. If one of the claims on the property (and their effect) are not desirable in my setting, it would be nice to be able to add a qualifier in my local environment to effectively disable the undesirable effect [07:16:47] qualifiers on properties [07:21:07] im guessing bots using wikidata clone instances as caches will also benefit from being able to add qualifiers to their clone to assist their bots [07:21:53] but those use cases are not likely to be a high priority for your team ;-) [07:42:09] GerardM: You're flooding the recent changes... [07:43:01] Avoiding such a flood is kind of the entire reason the flood flag/bot flag exists. [07:43:19] ...and you don't seem to be using it [07:47:01] YairRand: I have a bot and I found that it behaves differently from my usual profile [07:47:07] edits were discarded [07:47:16] the point is to make those edits and all of them [07:48:22] GerardM: The edits don't go through? the server rejects them? [07:48:50] that happened [07:48:52] yes [07:48:55] are you using a separate account, and does it have the necessary user rights? [07:49:05] there's a cap for ordinary accounts [07:49:07] I think [07:51:32] * YairRand looks at the user rights list... [07:53:14] GerardM: The autoconfirmed user right makes the user exempt from the base rate limits. if you were trying bot edits from a non-autoconfirmed account, that could be why the edits weren't all being accepted. [07:54:18] RobotGMwikt has been running for ages [07:55:07] has millions of edits [07:55:17] oh [07:55:28] okay, then I don't know why that happened. sorry. [07:56:05] also I now have a project of adding items that will be done by the end of the day [07:56:20] I will blog about it as well [07:56:33] the next point is that automated tools are there for everyone [07:56:54] so makes using automated tools for being a bot ? [07:58:02] such automated tools should really only be used from accounts with flood or bot flags, when making so many edits [07:58:40] I kind of wish we had a special recent changes setup for wikidata to allow large groups of edits to be monitored as groups... [07:59:01] [13WikibaseInternalSerialization] 15tobijat pushed 4 new commits to 06master: 02http://git.io/4Sgi5w [07:59:01] 13WikibaseInternalSerialization/06master 14fad763c 15jeroendedauw: Make use of new setFingerprint method in DataModel [07:59:01] 13WikibaseInternalSerialization/06master 14598226c 15jeroendedauw: Rename terms to fingerprint [07:59:01] 13WikibaseInternalSerialization/06master 14ea19eb6 15jeroendedauw: Optimized imports [07:59:01] (and undone, for that matter.) [08:00:29] but for the moment, we don't, so we must make do. so, could you request a flood flag for your account before continuing the edits? [08:01:01] [13WikibaseQuery] 15adrianlang pushed 1 new commit to 06master: 02http://git.io/nVqxyA [08:01:01] 13WikibaseQuery/06master 14d951df9 15Adrian Lang: Use Wikibase's snakview for the SimpleQuery special page [08:01:01] [travis-ci] wmde/WikibaseInternalSerialization/master/b89a307 : Tobi Gritschacher The build was broken. http://travis-ci.org/wmde/WikibaseInternalSerialization/builds/22589800 [08:01:21] [13WikibaseQuery] 15adrianlang 04force-pushed 06master from 14d951df9 to 14d2d4e5f: 02http://git.io/8VNLwQ [08:01:54] [13WikibaseQuery] 15adrianlang 04force-pushed 06snakview from 146a20a9c to 14d951df9: 02http://git.io/29CD-Q [08:01:54] 13WikibaseQuery/06snakview 14d951df9 15Adrian Lang: Use Wikibase's snakview for the SimpleQuery special page [08:13:12] Yair, I am not a bot [08:13:14] Adrian_WMDE: /var/www/daniel/wikidata/extensions/Wikibase/vendor/data-values/interfaces/src/ValueValidators/Error.php [08:14:03] I have a bot as well [08:14:20] GerardM: I am aware of that. The flood flag is for non-bots doing quick automated edits from a main accoint. [08:14:50] though, theoretically, it's not much of a stretch to call Widar edits "bot edits", I would think [08:16:15] either way, a very large number of edits in quick succession need to be properly flagged. [08:17:06] otherwise, monitoring things becomes effectively impossible. [08:22:11] "Users may be granted a "flood flag" when doing repetitive, non-controversial edits or actions to avoid flooding Special:RecentChanges and related pages." [08:23:28] You can request the flag at [[Wikidata:Bureaucrats' noticeboard]], and remove the flag from yourself once the edits are done. Doesn't need to be a long process. [08:27:21] i'd disagree with that though, if it's stuff that could be controversial i dunno if it should be bot flagged [08:27:37] but still, you shouldn't run like 10 widar tabs at a time [08:41:07] YairRand it would make more sense for it to be a toggle [08:41:15] something that I can set at will [08:41:55] rschen7754 at this moment I have one session open and that is it [08:44:24] rschen7754 either I have the flood flag or I do not [08:44:32] you cannot have it both ways [08:45:59] is it stuff that could be controversial? [08:46:08] also what is uncontroversial ... that is very much dependend on people's perceptions [08:46:22] and largely I know why I do my edits [08:46:52] I have a consistent view on what brings quality to Wikidata [08:47:14] some people have contrary opinions.. that is fine with me [08:47:27] they do not have the same perspective [08:48:24] they often do not have arguments [08:49:07] this is straying from the topic somewhat, but if other people think your edits are damaging, you probably shouldn't be doing them without consensus, imo. [08:49:49] I have the opinion that what some people opine IS damaging in its own right [08:50:30] I disagree that just because some people have a negative opinion, they get their way [08:52:19] ...and that is why we have discussions, consensus, and admins that are capable of blocking upon achieving such consensus to block. I recommend against having a "If it doesn't make sense to be, I'm not going to listen to it" attitude. [08:52:41] s/to be/to me/ [08:56:56] when I notice that a request for a new property does not get acted upon because one negative voice, I find that there is no real discussion. I find mostly frustration [08:57:35] there is no consistent view on how we gain usability for Wikipedia [08:57:49] it's problematic. there aren't enough voices in those discussions. not much that can be done when only two people participate. [08:58:00] there are people who insist on sources at every stage making it impossible [08:58:33] Yair, I am happy to discuss anything as long as it is clear what the objective is [08:58:37] some people would consider unsourced data to be itself useless and making things impossible [08:58:51] they do not understand the concept of a wiki [08:59:00] wikidata has several objectives, not all of which have to do with wikimedia, btw [08:59:05] they are firmly in the Nupedia state of mind [08:59:26] and some objectives are mutually exclusive [08:59:39] quite. [08:59:50] you can not become usable for use in Wikipedia and insist on sources for everything [09:00:17] from my point of view that is a very restricted point of view [09:00:23] but it is a practical start [09:01:01] given that it is WIKI data, the nupedia mindset cannot prevail [09:01:45] (03CR) 10Thiemo Mättig (WMDE): [C: 032] (bug 62644) No snak val in ClaimModificationHelper (032 comments) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124317 (owner: 10Daniel Kinzler) [09:02:07] I don't think anyone here has a nupedia mindset exactly. wanting more statements to have sources, or requiring consensus for very large changes, doesn't make it not a wiki, imo [09:02:37] wanting more sources and thereby excluding unsourced statements is [09:02:55] I think this discussion will have to continue another time, sorry. gtg. [09:03:07] have fun and thanks :) [09:04:34] nb there are moments when sources are important [09:05:00] for instance when you compare differences between the values from sources [09:14:52] (03PS17) 10Daniel Kinzler: (bug 62644) move snak validation into ChangeOps [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/121689 [09:18:50] (03PS5) 10Daniel Kinzler: (bug 62644) snak validation in ChangeOpMainSnak. [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123860 [09:21:19] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/121689 (owner: 10Daniel Kinzler) [09:22:43] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123860 (owner: 10Daniel Kinzler) [09:28:56] I take it it is known that editing is not possible from within Wikidata ? [09:29:09] I can edit using Widar [09:40:57] (03CR) 10Tobias Gritschacher: [C: 04-1] (bug 62644) snak validation in ChangeOpMainSnak. (031 comment) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123860 (owner: 10Daniel Kinzler) [09:45:31] GerardM: did you try purging the page? [09:45:39] what page? [10:00:21] (03PS1) 10WikidataBuilder: New Wikidata Build - 09/04/2014 10:00 [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/124811 [10:03:55] aude: Did you see my message? [10:04:10] yes [10:04:20] what page did that happen on? [10:04:35] * aude can't reproduce on test.wikidata [10:04:43] GerardM: can you link to an example where this was happening please? [10:04:45] https://www.wikidata.org/wiki/Q15879883 [10:04:49] ok [10:06:03] The date format has also changed, so might have to something with that. [10:06:23] And the keyboardbug has been fixed. :DS [10:06:27] sjoerddebruin: some of the dates code changed [10:06:39] but shouldn't be any noticable difference [10:06:49] Ah. [10:07:00] sjoerddebruin: don't see any issue with that item [10:07:12] sjoerddebruin: does it still happen? [10:07:19] Going to try it again [10:07:26] https://www.dropbox.com/s/loze6n9s919hffe/Screenshot%202014-04-09%2008.00.24.png [10:07:31] what happened for sjoerddebruin [10:07:47] Still happens [10:07:58] what browser? [10:08:19] Safari on OS X. [10:08:24] ok [10:08:32] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/124811 (owner: 10WikidataBuilder) [10:08:41] Going to test in Chrome now [10:08:46] can reproduce js issues on https://www.wikidata.org/wiki/Q60 [10:09:10] (which i edited after wikiquote and before wikidata got switched to new code) [10:09:27] we bumped parser cache for wikiquote, so the issue could be with pages touched between that time [10:09:30] or viewd [10:09:33] viewed* [10:09:43] I get the same in Chrome [10:10:33] aude: could be [10:10:59] i think the number of pages might be small and tehy can be purged [10:11:03] they* [10:11:11] hoping [10:11:37] let's just see how it goes then [10:11:51] if necessary, we can bump again but normally should avoid that [10:11:59] (03PS1) 10Thiemo Mättig (WMDE): Fix "@param null $param" documentations [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124816 [10:12:28] (03CR) 10jenkins-bot: [V: 04-1] Fix "@param null $param" documentations [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124816 (owner: 10Thiemo Mättig (WMDE)) [10:14:01] (03CR) 10Addshore: [C: 032] New Wikidata Build - 09/04/2014 10:00 [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/124811 (owner: 10WikidataBuilder) [10:14:02] [13WikibaseQuery] 15adrianlang pushed 1 new commit to 06master: 02http://git.io/hOzvFg [10:14:02] 13WikibaseQuery/06master 14d97e297 15Adrian Lang: Use Wikibase's snakview for the SimpleQuery special page [10:14:37] [13WikibaseQuery] 15adrianlang 04force-pushed 06master from 14d97e297 to 14d2d4e5f: 02http://git.io/8VNLwQ [10:14:47] [13WikibaseQuery] 15adrianlang 04force-pushed 06snakview from 14d951df9 to 14d97e297: 02http://git.io/29CD-Q [10:14:47] 13WikibaseQuery/06snakview 14d97e297 15Adrian Lang: Use Wikibase's snakview for the SimpleQuery special page [10:14:55] sjoerddebruin: i can reproduce if i change my language to nl [10:15:03] Tobi_WMDE: ^ [10:15:09] Hm, I still have the bug in English so that's weird [10:15:16] hmmm [10:15:17] (03CR) 10Addshore: [C: 04-1] "Guessing the lib/includes/store/EntityLabelLookup.php files isnt meant to be here...?" (031 comment) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124816 (owner: 10Thiemo Mättig (WMDE)) [10:15:41] [travis-ci] wmde/WikibaseQuery/master/d97e297 : Adrian Lang The build has errored. http://travis-ci.org/wmde/WikibaseQuery/builds/22597668 [10:16:16] german shows ". 1935" [10:16:35] This is again something that leads to adding incorrect stuff. [10:16:55] addshore: ^ [10:16:58] ?? [10:17:04] problems with dates [10:17:07] what am I looking at? I think I came in half way through [10:17:13] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124816 (owner: 10Thiemo Mättig (WMDE)) [10:17:15] https://www.wikidata.org/wiki/Q15879883 [10:17:30] especially if i edit in de or nl (but also happens in en) [10:17:38] i enter 1935 and for german i get ". 1935" [10:17:47] in nl, i enter 1935 and get jan 1935" [10:18:13] addshore: seems that entering only a year in nl language will be parsed as january [10:18:21] Tobi_WMDE: really? O_o [10:18:28] but precision says "year" though [10:18:38] will play around in the sandbox [10:20:18] Tobi_WMDE: which language is that occouring in? [10:20:25] nl [10:20:29] german and nl [10:20:33] dutch [10:20:36] german? really? [10:20:40] different behaviors, neither i expect [10:20:48] tobi: "1935" becomes ". 1935" [10:20:53] hmmm [10:21:02] hehe [10:21:04] yes [10:21:05] true [10:21:06] its parsed at the year but then displayed with the month Jan [10:21:09] yikes!!! [10:21:14] *as [10:21:24] if you enter the year 999999 in EN it is displayed as 999999 [10:21:27] https://www.wikidata.org/wiki/Q76?uselang=ar [10:21:33] then view it in nl and its displayed as Jan 999999 [10:21:43] addshore: yes [10:21:43] shows timestamps! [10:21:45] [13Interfaces] 15adrianlang created 06augmentParseException (+1 new commit): 02http://git.io/3w69qA [10:21:45] 13Interfaces/06augmentParseException 1415fd8b3 15Adrian Lang: Add rawValue and expectedFormat arguments to ValueParsers\ParseException... [10:21:53] bad [10:21:57] precision [10:22:15] +00000002005-01-03T00:00:00Z [10:22:34] aude: the only reason if should show a timestamp is if it fails to format the value in any other way [10:22:37] its the final fallback [10:22:42] [13Interfaces] 15adrianlang opened pull request #4: Add rawValue and expectedFormat arguments to ValueParsers\ParseException (06master...06augmentParseException) 02http://git.io/44E0ug [10:22:42] ugh [10:22:47] en dates should be final fallback [10:23:05] aude: en dates already come before the fallback [10:23:12] so it should be formatted before doing the timestamp [10:23:19] super yikes [10:23:20] generally the only time I saw the timestamp when testing is with an invalid date [10:23:33] hence nothing can format it [10:23:46] obama has invalid dates? [10:23:52] * aude doubts [10:24:12] indeed I doubt [10:24:22] and if it formats with the lang en but not with another lang then something is wrong [10:24:27] * sjoerddebruin is just a user and finds the largest bugs. [10:24:40] in german, i see "Endzeitpunkt .1979" [10:24:47] err ". 1979" [10:25:07] Endzeitpunkt [10:25:15] sjoerddebruin: definitely helpful [10:25:32] * aude wasn't expecting the new code to be deployed last night (since wmf was working on localisation cache bugs) [10:25:39] aude: all of the translate keys should still be marked as DO NOT TRANSLATE [10:26:02] only month names should currently be localised, unless someone has changed that without me noticing :O [10:27:48] Guys: keep me up, if you make a bug send me the link pl0x. :) [10:28:05] ok [10:28:06] WikibaseValueFormatterBuilders::newHtmlTimeFormatter O_o [10:28:22] That's me [10:28:25] right? [10:28:28] Calendar model hint [10:28:53] (03PS1) 10Addshore: Remove unused WikibaseValueFormatterBuilders::newHtmlTimeFormatter [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124819 [10:29:16] you must be getting it from somewhere else [10:29:26] https://www.wikidata.org/wiki/Q60?uselang=ar also has timestamps [10:29:30] timestamps everywhere [10:29:57] addshore: line 85 [10:30:26] oh, dam text usages! [10:30:35] right well that could be the problem [10:30:48] these two methods to get the html formatters are only using one of the parsers [10:31:00] they should be using the TimeParser (dispatching) [10:31:06] hmmm [10:31:09] 8formatter [10:31:32] addshore: That's exactly why I said that this kind of stuff does not need unit tests, but API endpoint tests [10:31:49] both [10:31:52] Or tests by me. :P [10:32:05] aude: "need" as in "desperately need" [10:32:11] normally we would have had time to test on test.wikidata [10:32:25] Adrian_WMDE: yeah [10:32:25] wait, no as this is only formatting [10:32:27] not parsing [10:32:45] (03CR) 10Addshore: [C: 04-1] Remove unused WikibaseValueFormatterBuilders::newHtmlTimeFormatter [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124819 (owner: 10Addshore) [10:33:21] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124819 (owner: 10Addshore) [10:33:22] mhhm, I need a coffee , will brb [10:33:27] i can reproduce ". 1935" and ar timestamps on my wiki [10:35:54] (03PS2) 10Thiemo Mättig (WMDE): Fix "@param null $param" documentations [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124816 [10:38:40] right, the call to wbformatvalue returns the correct year, i.e. 3332... [10:38:51] (03CR) 10Thiemo Mättig (WMDE): [C: 032] (bug 62644) snak validation in more change ops [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123872 (owner: 10Daniel Kinzler) [10:40:24] i can reproduce "1935" => "jan 1935" [10:40:34] Hoi I can edit again [10:40:50] aude indeed, but if you click edit while in NL and watch the calls to the API through the console wbformatvalue will return only 1935 [10:40:53] which is correct [10:40:58] oh [10:41:13] the php formatted snak values come from SnakHtmlGenerator right? [10:41:31] i dont' think that's in the branch, but yes [10:41:39] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124816 (owner: 10Thiemo Mättig (WMDE)) [10:41:44] hah :P [10:41:45] claimhtmlgenerator otherwise [10:42:01] shoudl be same code, only moved to snak generator [10:44:44] * addshore wonders where the entitiy / itemview is actually constructed [10:46:06] https://test.wikidata.org/w/api.php?action=wbformatvalue&format=json&datavalue=%7B%22value%22%3A%7B%22time%22%3A%22%2B00000001944-01-01T00%3A00%3A00Z%22%2C%22timezone%22%3A0%2C%22before%22%3A0%2C%22after%22%3A0%2C%22precision%22%3A9%2C%22calendarmodel%22%3A%22http%3A%2F%2Fwww%2Ewikidata%2Eorg%2Fentity%2FQ1985727%22%7D%2C%22type%22%3A%22time%22%7D&options=%7B%7D&datatype=time&generate=text%2Fplain [10:46:12] sorry for the long link :p [10:46:26] that's what is requested, with 1944-01-01 [10:46:58] const PRECISION_YEAR = 9; [10:47:06] aude: thats fine, as there is on 1944-00-00 [10:47:18] precision 9 means it will be formatted with only the year taken into account [10:47:38] ok [10:47:57] Im digging through and got to OutputFormatSnakFormatterFactory::getSnakFormatter [10:48:06] this is where the snak formatters come from for itemview [10:48:20] (03CR) 10Thiemo Mättig (WMDE): [C: 032] (bug 62644) snak validation in ChangeOpMainSnak. (032 comments) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123860 (owner: 10Daniel Kinzler) [10:48:21] ok [10:48:32] with SnakFormatter::FORMAT_HTML_WIDGET which is 'text/html; disposition=widget' [10:50:00] the builders come from WikibaseSnakFormatterBuilders::getSnakFormatterBuildersForFormats [10:50:28] and then buildDispatchingSnakFormatter, oh so many layers [10:50:32] :o [10:51:31] WikibaseValueFormatterBuilders::buildDispatchingValueFormatter [10:52:39] where do the options come from? [10:52:53] god knows :P [10:53:25] , so at the top of WikibaseValueFormatterBuilders both a html and plain formatter are registered [10:54:10] Wikibase\Lib\WikibaseValueFormatterBuilders->newFromSpec [10:55:07] (03PS7) 10Thiemo Mättig (WMDE): (bug 62644) Snak validation in more ChangeOps [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123872 (owner: 10Daniel Kinzler) [10:55:59] mhhhm, im getting lost in code I hanvt looked at much :p [10:56:09] wo ist daniel? :) [10:56:22] :D [10:58:05] So, its not the Api, it is im some part of the code that generates the initial html ItemView ;p [10:58:25] well also happens in formatvalue api module [10:58:32] really? O_o [10:58:36] I cant reproduce that [10:58:38] see the link [10:58:55] aude: {"result":"1944"} [10:59:02] result: "jan 1944" [10:59:03] } [10:59:11] thats what it should be.... [10:59:56] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123872 (owner: 10Daniel Kinzler) [11:00:26] we can't just enter "1944"? [11:00:36] im confused [11:00:42] why do you get result jan 1944? [11:00:47] I dont.... [11:00:52] looking at the api calls [11:01:11] what call gives result: "jan 1944" } : [11:01:12] ? [11:01:14] https://test.wikidata.org/wiki/Q22?setlang=nl [11:01:29] i assume de give ". 1944" [11:01:35] what the actual .... [11:01:55] probably ar give timestamp [11:02:05] $timestamp = wfTimestamp( TS_MW, $fourDigitYearTimestamp ); [11:02:05] $localisedDate = $this->language->sprintfDate( [11:02:05] $this->getDateFormat( $precision ), [11:02:05] $timestamp [11:02:05] ); [11:02:08] got to be here then [11:02:22] https://test.wikidata.org/w/api.php?action=wbformatvalue&format=json&datavalue=%7B%22value%22%3A%7B%22time%22%3A%22%2B00000001941-01-01T00%3A00%3A00Z%22%2C%22timezone%22%3A0%2C%22before%22%3A0%2C%22after%22%3A0%2C%22precision%22%3A9%2C%22calendarmodel%22%3A%22http%3A%2F%2Fwww%2Ewikidata%2Eorg%2Fentity%2FQ1985727%22%7D%2C%22type%22%3A%22time%22%7D&options=%7B%7D&datatype=time&generate=text%2Fplain [11:02:32] .https://test.wikidata.org/w/api.php?action=wbformatvalue&format=json&datavalue=%7B%22value%22%3A%7B%22time%22%3A%22%2B00000001941-01-01T00%3A00%3A00Z%22%2C%22timezone%22%3A0%2C%22before%22%3A0%2C%22after%22%3A0%2C%22precision%22%3A9%2C%22calendarmodel%22%3A%22http%3A%2F%2Fwww%2Ewikidata%2Eorg%2Fentity%2FQ1985727%22%7D%2C%22type%22%3A%22time%22%7D&options=%7B%7D&datatype=time&generate=text%2Fplain [11:02:38] gah [11:02:39] result: ". 1941" [11:03:48] https://test.wikidata.org/w/api.php?action=wbformatvalue&format=json&datavalue=%7B%22value%22%3A%7B%22time%22%3A%22%2B00000001941-01-01T00%3A00%3A00Z%22%2C%22timezone%22%3A0%2C%22before%22%3A0%2C%22after%22%3A0%2C%22precision%22%3A9%2C%22calendarmodel%22%3A%22http%3A%2F%2Fwww%2Ewikidata%2Eorg%2Fentity%2FQ1985727%22%7D%2C%22type%22%3A%22time%22%7D&options=%7B%7D&datatype=time&generate=text%2Fplain (in arabic) [11:03:55] result: "+00000001941-01-01T00:00:00Z" [11:04:05] * addshore keeps digging [11:04:34] i assume formatting is done same way, same options in entity view / claim view [11:05:05] mhhhm, got to fix my ULS first.. [11:05:23] ok [11:05:35] * aude looks at formatvalue [11:06:37] $this->getDateFormat( $precision ) [11:06:54] when passed with a precision of 9 we get back string 'M Y' (length=3) [11:06:56] which be wrong [11:07:15] getting lunch [11:07:22] can look when get back [11:07:23] kk :) [11:16:12] (03PS1) 10Addshore: Correctly format dates for lang=nl [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124823 [11:16:39] sjoerddebruin: looks like I have found where the problem lies [11:16:59] did you open a bug? [11:17:26] No. [11:17:31] okay :) [11:17:33] the problems lies? [11:17:40] :P [11:17:55] well, a month ago we only ever formatted dates in englaish [11:17:57] *english [11:18:27] and when the new formatters and stuff were brought in there is actually 1 bit of the old code that remains, and amusingly thats where the problem is :D [11:19:23] guess I will go through and write tests for ALL of the languages now :D [11:19:31] :O [11:21:43] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124823 (owner: 10Addshore) [11:26:18] addshore: Are you creating a bug now? [11:28:22] sjoerddebruin: I will do :) [11:31:14] (03CR) 10WikidataJenkins: "Build Failed" [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/124811 (owner: 10WikidataBuilder) [11:33:02] (03CR) 10Addshore: "browser tests passed right??" [extensions/Wikidata] - 10https://gerrit.wikimedia.org/r/124811 (owner: 10WikidataBuilder) [11:39:04] sjoerddebruin: aude https://bugzilla.wikimedia.org/show_bug.cgi?id=63723 [11:39:33] (03PS2) 10Addshore: Correctly format dates for lang=nl [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124823 [11:39:37] Thanks. :) [11:41:12] (03PS1) 10Addshore: Add basic tests for formatting dates in different langs [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124824 [11:43:19] Is critical the highest importance? [11:44:25] yup [11:44:42] just establishing a list of languages its currently broken for [11:46:56] Okay. [11:46:58] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124823 (owner: 10Addshore) [11:47:02] * sjoerddebruin removes this from his bucket list [11:47:12] :) [11:50:33] "Report a critical bug" ;) [11:50:47] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124824 (owner: 10Addshore) [11:53:11] haha :P [11:53:25] looks like im going to have a busy day :D [11:55:16] [13Common] 15adrianlang created 06augmentParseException from 06master (+0 new commits): 02http://git.io/5VI0Uw [11:56:25] sjoerddebruin: 164 languagesaffected [11:56:30] Auch [11:56:34] [13Common] 15adrianlang pushed 1 new commit to 06augmentParseException: 02http://git.io/gceZGQ [11:56:34] 13Common/06augmentParseException 146e7e061 15Adrian Lang: Pass rawValue and expectedFormat arguments to ParseExceptions [11:56:57] out of 397 [11:56:59] [13Common] 15adrianlang opened pull request #8: Pass rawValue and expectedFormat arguments to ParseExceptions (06master...06augmentParseException) 02http://git.io/YJE_7g [11:57:16] :O [11:58:42] (03PS20) 10Daniel Kinzler: (bug 40157) store entity status in page_props [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/99385 [11:59:57] addshore: how's it going? [12:00:12] aude: found where the problem is, there is a patch up for NL [12:00:22] one patch for each lang? [12:00:25] just writing tests for everything then I can go through fixing them all [12:00:28] (03CR) 10Daniel Kinzler: [C: 04-1] "If the parameter defaults to null, it should be declared as foo|null, not just as foo." [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124816 (owner: 10Thiemo Mättig (WMDE)) [12:00:39] ok [12:00:49] (03CR) 10Addshore: [C: 04-1] "done merge me, followup coming" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124824 (owner: 10Addshore) [12:00:57] (03CR) 10Addshore: "dont..." [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124824 (owner: 10Addshore) [12:01:06] DONE MERGE ME° [12:02:55] :D [12:03:32] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/99385 (owner: 10Daniel Kinzler) [12:04:09] :D [12:04:24] addshore: i can merge that one? [12:04:32] looks good and works good [12:04:48] which? :P [12:04:50] NL or tests? [12:04:54] although we might not deploy just yet since we needm ore fixes [12:04:56] nl [12:05:01] yee, nl is fine [12:05:04] ok [12:05:07] Can i test addshore? [12:05:17] sjoerddebruin: not easily [12:05:20] :P [12:05:23] unless you have wikibase installed [12:05:26] Okay, I trust you. :D [12:05:38] we could deploy on beta wikidata [12:05:50] (03CR) 10Aude: [C: 032] Correctly format dates for lang=nl [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124823 (owner: 10Addshore) [12:06:14] (03Merged) 10jenkins-bot: Correctly format dates for lang=nl [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124823 (owner: 10Addshore) [12:08:58] [13Number] 15adrianlang created 06augmentParseException (+1 new commit): 02http://git.io/qAAavQ [12:08:58] 13Number/06augmentParseException 14175b374 15Adrian Lang: Pass rawValue and expectedFormat arguments to ParseExceptions... [12:09:37] [13Number] 15adrianlang opened pull request #5: Pass rawValue and expectedFormat arguments to ParseExceptions (06master...06augmentParseException) 02http://git.io/7ddvZw [12:09:52] (03CR) 10Daniel Kinzler: (bug 62644) snak validation in ChangeOpMainSnak. (032 comments) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123860 (owner: 10Daniel Kinzler) [12:10:37] [13Common] 15adrianlang 04force-pushed 06augmentParseException from 146e7e061 to 14f0f6ca3: 02http://git.io/5VI0Uw [12:10:37] 13Common/06augmentParseException 14f0f6ca3 15Adrian Lang: Pass rawValue and expectedFormat arguments to ParseExceptions... [12:12:52] (03CR) 10Daniel Kinzler: [C: 04-1] "ClaimTestHelper was renamed. This needs a manual rebase. Waiting for the base patch to be merged before i do that." [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123860 (owner: 10Daniel Kinzler) [12:18:15] (03CR) 10Thiemo Mättig (WMDE): [C: 04-1] "Oh noes. I almost hit +2 but found a new issue. :-(" (031 comment) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/121689 (owner: 10Daniel Kinzler) [12:21:08] addshore: so no test coverage of public function format( $value ) in MwTimeIsoFormatter? am i blind or right about that? [12:21:26] yes theres coverage, but only for english [12:21:35] oh [12:21:45] (03PS2) 10Addshore: Add basic tests for formatting dates in different langs [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124824 [12:21:58] all i see is testFormatDate [12:22:08] (03PS3) 10Addshore: Add tests for formatting dates in different langs [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124824 [12:22:47] hmm, aude acutally that is the case! [12:22:54] * addshore will make another patch to fix that [12:23:01] ok [12:23:05] after fixing this slightly bigger issue ;p [12:23:12] ok [12:23:25] gah, so many broken languages! [12:23:30] yup [12:23:31] i am looking at de right now [12:23:44] i don't know if the issues are all different [12:24:00] indeed, fixing one lang may well fix 5 or 6 different langs :P [12:24:07] yep [12:24:10] for de, there is a date issue reported at https://www.wikidata.org/wiki/Wikidata:Forum#Geburtsdatum_hier_1400 [12:24:19] reimann: thanks [12:24:30] easiest weay to debug it is to stick a var_dump of the tme format in to getDateFormat of mwtimeisformatter [12:24:39] and get $dateFormat [12:25:00] then see what it has in there that we are not expecting in the two little Ifs below that remove the month and day placeholders [12:25:24] exact year 1400 cannot be entered per GUI as it is considered as 14th century (lower precision) [12:25:54] (03PS4) 10Addshore: Add tests for formatting dates in different langs [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124824 [12:26:42] reimann: hmm that sounds wrong, if you set the precision in the UI it will obey it [12:27:54] so aude for DE the timeformat is string 'j. F Y' (length=6) [12:28:11] the regex '/((x\w{1})?(j|t)|d)/' doesnt account for the . [12:28:18] looking [12:28:30] '/((x\w{1})?(j\.?|t)|d)/' would likely work [12:28:40] mmm [12:29:06] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124824 (owner: 10Addshore) [12:29:12] infact it does work, but we might want to rethink using the regex and instead hard code what the tokens are for each language [12:29:23] but I can do that :) [12:29:35] i am making test for format() [12:29:41] so i can see and poke how it works [12:29:52] aude: the test for format() is essentially the same as the test that is already there [12:29:58] yep [12:29:59] just we need to construct a TimeValue object first [12:33:09] back in 5! [12:33:40] ok [12:34:25] [13Time] 15adrianlang created 06augmentParseException (+1 new commit): 02http://git.io/sFfjkA [12:34:25] 13Time/06augmentParseException 140460e77 15Adrian Lang: Pass rawValue and expectedFormat arguments to ParseExceptions... [12:34:39] [13Time] 15adrianlang opened pull request #14: Pass rawValue and expectedFormat arguments to ParseExceptions (06master...06augmentParseException) 02http://git.io/ZOYWpQ [12:35:05] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124824 (owner: 10Addshore) [12:35:26] addshore: true, sorry, I did only check the diffs. However, if you switch the extended mode to precision "year", still ". 14. century" is shown. While it is stored correctly in the database, the representation looks wrong. [12:35:49] "1400" would be expected. [12:36:42] reimann: in DE? Ill give it quick poke [12:37:01] Adrian_WMDE might also be able to look [12:37:15] addshore: lang doesnt matter [12:38:50] Hi guys. Please somebody add the line for Wikiquote into https://www.wikidata.org/wiki/MediaWiki:Gadget-MainLangFirst.js [12:38:55] [13jenkins-job-builder-config] 15tobijat created 06jbbtest (+1 new commit): 02http://git.io/__rtXQ [12:38:55] 13jenkins-job-builder-config/06jbbtest 14a9e5521 15Tobias Gritschacher: Split up yaml files [12:39:19] reimann: could you file a bug? [12:39:43] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124824 (owner: 10Addshore) [12:39:43] aude: Ill file a bug for the not test coverage of the public format function too [12:39:53] just for tracking, my list is getting long :P [12:39:53] making patch [12:40:00] ahh okay :P [12:40:11] try not to merge conflict and i am willing to rebase [12:40:17] will need expanding [12:43:01] addshore: reimann: I'm pretty sure there is a bug for this [12:43:08] i'll have a look [12:43:25] i thought this was one of the things we fixed before deploying.. [12:45:39] (03PS1) 10Aude: Add initial test for MwTimeIsoFormatter::format [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124830 [12:45:45] jenkins will say no because of bug [12:46:07] addshore: I thought so [12:46:50] Adrian_WMDE: didn't find onbe yet [12:46:54] No open bug [12:47:01] I'll look at the closed ones [12:47:40] aude, i guess all the test needs to do is make sure it calls dateFormat, which is throughly tested [12:47:57] ok [12:47:59] Mh, at least I did not open such a bug [12:48:37] or aude we stop using the interface for DateFormatter and thus make that method protected and just test the format method [12:48:50] we can think about that later [12:49:06] not too later but once we fix immediate problems [12:49:21] yep, im just putting together a patch to stop using the regexes [12:49:36] which should then make fixing the other langs much easier [12:49:46] ok [12:50:41] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124830 (owner: 10Aude) [12:51:22] Adrian_WMDE: shall i submit the bug? [12:53:43] [13jenkins-job-builder-config] 15tobijat pushed 1 new commit to 06jbbtest: 02http://git.io/pJURUw [12:53:43] 13jenkins-job-builder-config/06jbbtest 146f54b74 15Tobias Gritschacher: Removed broken junit report publisher from browsertest job [12:54:35] reimann: I guess so [12:54:54] [13jenkins-job-builder-config] 15tobijat opened pull request #7: Split up yaml files and remove broken publisher (06master...06jbbtest) 02http://git.io/z3MBRA [12:55:03] [13jenkins-job-builder-config] 15tobijat 04deleted 06jbbtest at 146f54b74: 02http://git.io/u-uenA [13:04:47] (03PS5) 10Addshore: Add tests for formatting dates in different langs [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124824 [13:06:17] addshore: how quick do you think we can get this all resolved? [13:06:27] (03PS1) 10Addshore: Stop ALWAYS usign regex to fix dateFormats [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124833 [13:06:35] patch! [13:06:38] i should be able to get most of them dont in another patch [13:06:43] ok [13:06:51] I just have to go through each one though :P [13:07:01] and theres a fair few of them :P [13:07:50] ok, so basically a mapping of language date formats to our format? [13:08:14] well, just a mapping of languagecode to the placeholder for day and placeholder for month in that lagnuage [13:08:17] * aude looking at the language classes [13:08:41] I had a look but couldnt imediatly spot anything [13:09:19] german has $defaultDateFormat = 'dmy' and $dateFormats array [13:09:29] 'j. F Y' for 'date' [13:09:51] i suppose we get '. F Y' from stripping j [13:10:16] yup [13:10:22] i have already fixed the DE ones ;p [13:10:28] ok [13:11:58] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124824 (owner: 10Addshore) [13:14:27] woah, so many skipped tests [13:14:33] yup :P [13:14:42] naturally, one per language [13:14:51] it's taking a really looong time to run, as i suppose it loads all language objects [13:15:01] ALLL of the languages :P [13:16:28] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124833 (owner: 10Addshore) [13:22:36] (03PS1) 10Addshore: DNM WIP Fix formatting in various languages [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 [13:27:17] [13Geo] 15adrianlang created 06augmentParseException (+1 new commit): 02http://git.io/wLnpSA [13:27:17] 13Geo/06augmentParseException 14ab8433d 15Adrian Lang: Pass rawValue and expectedFormat arguments to ParseExceptions... [13:27:55] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 (owner: 10Addshore) [13:29:07] [13Geo] 15adrianlang opened pull request #8: Pass rawValue and expectedFormat arguments to ParseExceptions (06master...06augmentParseException) 02http://git.io/Ix68hQ [13:40:18] hah, $localisedDate = '١٩٤١' for arabic [13:40:26] could be we don't know how to handle those [13:40:32] indeed :/ [13:40:41] let's see if we can fix :) [13:40:44] haha xD [13:41:22] (03PS2) 10Addshore: DNM WIP Fix formatting in various languages [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 [13:41:30] moreeee :D [13:41:57] good job I have a club-mate here ;p [13:42:11] the solution looks generally good [13:42:26] might not be most maintainable though [13:42:31] yup, its just time consuming going through and checking each format :/ [13:42:32] more so than regex [13:43:06] I guess the perfect solution would be to be able to get these things from Language [13:43:47] $this->language->getMonthPlaceholders or something [13:44:17] yeah [13:47:07] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 (owner: 10Addshore) [13:47:27] Thiemo_WMDE: did you catch the train yesterday? [13:51:24] [13Interfaces] 15JeroenDeDauw comment on pull request #4 1415fd8b3: I'd flip around where you put this info: put it in the constructor and getter docs, and not for the fields. After all, the people using the class will interact with the constructor and getters, and not with the private fields. And when looking at this class itself, docs at the fields are not needed if they are there for the constructor. 02http://git.io/exy_Vw [13:51:59] [13Interfaces] 15JeroenDeDauw comment on pull request #4 1415fd8b3: Wrong type doc, can be null 02http://git.io/n3ObgQ [13:52:08] [13Interfaces] 15JeroenDeDauw comment on pull request #4 1415fd8b3: missing dot 02http://git.io/fnJPgw [13:53:23] [13Interfaces] 15JeroenDeDauw comment on pull request #4 1415fd8b3: After reading these docs it is still not really clear to me what this field holds 02http://git.io/dBN8Zw [13:53:44] (03PS18) 10Thiemo Mättig (WMDE): (bug 62644) Move Snak validation into ChangeOps [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/121689 (owner: 10Daniel Kinzler) [13:55:03] (03PS6) 10Daniel Kinzler: (bug 62644) snak validation in ChangeOpMainSnak. [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123860 [13:57:02] (03PS7) 10Daniel Kinzler: (bug 62644) snak validation in ChangeOpMainSnak. [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123860 [14:00:41] (03CR) 10Tobias Gritschacher: [C: 031] (bug 40157) store entity status in page_props [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/99385 (owner: 10Daniel Kinzler) [14:02:47] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/121689 (owner: 10Daniel Kinzler) [14:04:44] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123860 (owner: 10Daniel Kinzler) [14:08:10] (03PS8) 10Daniel Kinzler: (bug 62644) Snak validation in more ChangeOps [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123872 [14:08:44] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123860 (owner: 10Daniel Kinzler) [14:09:26] (03CR) 10Daniel Kinzler: [C: 031] "Thanks for the patch Thiemo!" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/121689 (owner: 10Daniel Kinzler) [14:10:15] (03CR) 10Thiemo Mättig (WMDE): [C: 032 V: 031] (bug 62644) Move Snak validation into ChangeOps [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/121689 (owner: 10Daniel Kinzler) [14:10:17] (03PS6) 10Daniel Kinzler: (bug 62644) No snak val in ClaimModificationHelper [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124317 [14:10:40] (03Merged) 10jenkins-bot: (bug 62644) Move Snak validation into ChangeOps [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/121689 (owner: 10Daniel Kinzler) [14:11:04] (03CR) 10Thiemo Mättig (WMDE): [C: 032] (bug 62644) snak validation in ChangeOpMainSnak. [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123860 (owner: 10Daniel Kinzler) [14:11:28] (03Merged) 10jenkins-bot: (bug 62644) snak validation in ChangeOpMainSnak. [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123860 (owner: 10Daniel Kinzler) [14:11:38] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123872 (owner: 10Daniel Kinzler) [14:14:34] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124317 (owner: 10Daniel Kinzler) [14:15:12] [13WikibaseDataModel] 15thiemowmde pushed 1 new commit to 06master: 02http://git.io/txt-Yg [14:15:12] 13WikibaseDataModel/06master 14ca6e0b8 15thiemowmde: Merge pull request #42 from wmde/setlabel... [14:15:21] (03PS9) 10Daniel Kinzler: (bug 62644) Snak validation in more ChangeOps [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123872 [14:15:30] [travis-ci] wmde/WikibaseInternalSerialization/master/b89a307 : Tobi Gritschacher The build was broken. http://travis-ci.org/wmde/WikibaseInternalSerialization/builds/22589800 [14:15:33] (03PS7) 10Daniel Kinzler: (bug 62644) No snak val in ClaimModificationHelper [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124317 [14:18:47] addshore: https://bugzilla.wikimedia.org/show_bug.cgi?id=63726 is what you've been working on right? [14:19:09] no [14:19:21] im working on https://bugzilla.wikimedia.org/show_bug.cgi?id=63723 [14:19:27] ok [14:19:28] Hoi [14:19:31] hey [14:19:39] Hey. :) [14:19:45] regularly editing becomes available in WIkidata [14:20:13] the edit button (text) is not available :( [14:20:21] purge the page [14:20:24] I am trying to kill of https://www.wikidata.org/wiki/Q7417181 [14:20:29] if that doesn't help please link the item here [14:20:52] 8 April 2014 [14:20:52] It happens repeatedly [14:21:07] works after purging here [14:21:15] known issue we can't do much about unfortunately [14:21:27] will go away when the page is purged once and should not reappear [14:21:44] it persisted through one powerdown [14:21:56] it is a server-side caching thing [14:22:30] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123872 (owner: 10Daniel Kinzler) [14:23:13] ok [14:23:38] removed all my history in my browser [14:23:48] Let's try one more time =) Please somebody add highlightSiteLinksGroup( 'wikiquote', 'wikiquote' ); into https://www.wikidata.org/wiki/MediaWiki:Gadget-MainLangFirst.js [14:23:50] it works now [14:24:06] Sandy Brown RIP [14:24:24] putnik: you want an !admin [14:24:34] (poked them now so they should see it [14:24:51] Lydia_WMDE, you have sysop flag, I know it =) [14:25:09] putnik: i do but i should not be using it outside emergencies [14:25:14] not a staff action [14:25:37] getting there aude :) [14:25:53] alright [14:26:09] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124317 (owner: 10Daniel Kinzler) [14:26:18] also finding all the langcodes that are jst returning timestmaps [14:26:23] think i figured out the ar timestampes [14:26:25] such as 'anp', 'as', 'bh', 'bho', 'bn', 'bo', 'bpy', 'dz', 'gu', 'hi', 'kn', 'ks', [14:27:25] putnik: Done. [14:28:23] aude, what do you think it is? [14:28:33] sjoerddebruin, thank you! [14:29:22] addshore: i think parseFormattedNumber is needed and then formatYear can work [14:29:31] (03CR) 10Thiemo Mättig (WMDE): [C: 032] (bug 62644) Snak validation in more ChangeOps [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123872 (owner: 10Daniel Kinzler) [14:29:36] i'm poking a bit more and can have a patch [14:29:47] :) [14:29:54] (03CR) 10Thiemo Mättig (WMDE): [C: 032] (bug 62644) No snak val in ClaimModificationHelper [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124317 (owner: 10Daniel Kinzler) [14:29:58] thing is for farsi, it is expected to have farsi dates [14:30:01] (03Merged) 10jenkins-bot: (bug 62644) Snak validation in more ChangeOps [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123872 (owner: 10Daniel Kinzler) [14:30:16] for arabic, it can be either and think arwiki tends to show same dates we use [14:30:32] (03Merged) 10jenkins-bot: (bug 62644) No snak val in ClaimModificationHelper [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124317 (owner: 10Daniel Kinzler) [14:30:48] what i have always shows arabic localized dates, which i suppose is ok and better than a timestamp [14:31:04] and then figure out what controls that [14:31:44] :P [14:32:01] not better than a timestamp as I doubt you can edit it then :P [14:32:42] investigating [14:33:20] or maybe numbers can be unlocalized [14:35:27] (03CR) 10Daniel Kinzler: (bug 40157) store entity status in page_props (034 comments) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/99385 (owner: 10Daniel Kinzler) [14:49:58] (03CR) 10Jeroen De Dauw: [C: 032] "Looks fine. (The EntityContent thing is a bit more dubious now, but whatever.) Some suggestions for style improvements provided inline" (034 comments) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/99385 (owner: 10Daniel Kinzler) [14:51:08] addshore: did you bail out of the dm serialization stuff? ;p [14:51:43] aude: just 81 broken languages left, half are just returning timestmaps and half have interesting chars in their $dateFormat [14:51:55] JeroenDeDauw: yup :P got busy with other things :p [14:52:20] addshore: :/ [14:52:35] (03PS3) 10Addshore: DNM WIP Fix formatting in various languages [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 [14:52:40] aude ^^ thats what it looks like now [14:57:01] (03Abandoned) 10Addshore: Remove unused WikibaseValueFormatterBuilders::newHtmlTimeFormatter [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124819 (owner: 10Addshore) [14:59:18] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 (owner: 10Addshore) [14:59:31] right then [14:59:34] half way :D [14:59:38] yay [14:59:50] time for another club-mate ;p [15:01:57] JeroenDeDauw: today i would be interested in ordering food. how about it? [15:07:59] aude: please put a space between // and the comment [15:08:04] /Comment looks odd [15:08:07] huh [15:08:10] . //Comment [15:08:11] JeroenDeDauw: why talking to me? [15:08:19] aude: https://gerrit.wikimedia.org/r/#/c/124824/5/lib/tests/phpunit/formatters/MwTimeIsoFormatterTest.php [15:08:27] ohg [15:08:29] lol [15:08:32] addshore: ^^ [15:08:33] * addshore waves [15:08:36] :P [15:08:44] i am not reformatting the entire class [15:08:48] addshore: you are the new aude [15:08:49] can be done later.... not important [15:09:04] please do not distract addshore :) [15:10:38] so, if i choose "10,000" years precision it says "in 0 years"? [15:10:43] is that correct??? [15:13:57] in X years yes [15:14:08] aude: it's consistent, i don't know about correct :P [15:14:11] it's buggy or odd... not to worry though [15:14:15] DanielK_WMDE: indeed :P [15:14:21] addshore: doesn't make sense for "small" numbers though [15:14:29] just trying to determine if it's broken in arabi [15:14:30] c [15:14:32] small numbers? [15:14:46] like 2014, with a precision of 100000 [15:14:56] value < precision [15:15:08] |value| < precision, actually [15:21:23] DanielK_WMDE: sure, in 30 mins [15:21:39] ok [15:24:47] [13WikibaseDataModel] 15JeroenDeDauw created 06termlist (+1 new commit): 02http://git.io/RCDoyQ [15:24:47] 13WikibaseDataModel/06termlist 145886e57 15jeroendedauw: Add hasTermForLanguage and removeByLanguage to TermList [15:27:04] * aude rage at the british english in the formatters [15:27:19] :P [15:27:36] aude: change it to klingon [15:27:48] heh [15:28:15] [13WikibaseDataModel] 15JeroenDeDauw created 06DescriptionList (+1 new commit): 02http://git.io/s2M3yg [15:28:15] 13WikibaseDataModel/06DescriptionList 14534c638 15jeroendedauw: Add setDescription to DescriptionList [15:36:12] did I ever say I hate dates and time? [15:36:22] (03CR) 10Daniel Kinzler: [C: 04-1] "Soem ideas / thoughts. The only blocker is the potentially fatal use of page_content_model." (037 comments) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/120857 (owner: 10Hoo man) [15:37:16] patch coming for localised digits [15:38:24] * addshore waits for another set of tests to run [15:41:56] (03CR) 10Daniel Kinzler: [C: 032] "Looks good. Trusting the test cases to make sure this doesn't break anything." [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124647 (owner: 10Bene) [15:42:22] (03Merged) 10jenkins-bot: Fix getItemIdForLink in SiteLinkLookup to return ItemId objects [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124647 (owner: 10Bene) [15:42:26] aude: whens the next brancyh day? [15:42:34] next week? [15:42:38] yeah [15:42:52] what we could do for now for the deployed stuff is force lang to EN [15:42:58] at least then stuff wont be broken [15:43:03] [travis-ci] wmde/WikibaseDataModel/termlist/5886e57 : jeroendedauw The build passed. http://travis-ci.org/wmde/WikibaseDataModel/builds/22618882 [15:43:03] or revert [15:43:10] or revert [15:43:18] did much else big get deployed? [15:43:29] globe formatters in backend [15:43:34] i highly doubt I will fix all of the languages today [15:43:37] (no issues afai) [15:43:38] k [15:43:39] right [15:43:51] Lydia_WMDE: what do you think? Tobi_WMDE ? [15:43:51] its not a speedy process :P [15:44:08] put wikibase back on wmf19 so users are not affected [15:44:24] I dont understand why this has only come up? [15:44:29] are we sure this isnt also in wmf19? [15:45:01] I wront the code ages ago but I guess the frontend has only just started using it in this deployed branch? [15:45:12] i am sure [15:45:16] well not [15:45:31] but we only started using backend time and globe formatters in wmf21 [15:45:41] okay :) [15:45:47] let me get my patch up and then ... [15:47:28] (03PS1) 10Aude: Normalise digits in time formatter [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124868 [15:48:41] that's best together on top of more tests and fixing the month handling [15:48:54] yup, looks nice :) [15:52:40] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124868 (owner: 10Aude) [15:53:18] (03PS4) 10Addshore: DNM WIP Fix formatting in various languages [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 [15:53:28] urgh, getting into all of the interesting dates with interesting characters [15:54:08] aude: I could probably remove all localization for Years right? [15:54:29] i.e. in 'sr', 'sr-ec', 'sr-el', 'hr' instead of returning '2014.' just return 2014 ? [15:55:12] for noe, i suppose [15:55:15] now* [15:55:26] * aude wonders what denny would say [15:56:15] dennyvrandecic: ? ;p [15:56:31] yeah [15:56:33] that way I can get rid of the $monthPlaceholders [15:56:39] for now, simple but i have no idea how hr dates should be [15:56:44] alright [15:56:55] well, we need them for arabic [15:57:09] ? [15:57:09] maybe [15:57:21] arabic will still just have 'Y' for the year though..? [15:57:23] ar' => 'xg', [15:57:28] everything else I have come accross does [15:57:29] i guess [15:57:33] we can try [15:57:53] * aude doesn't fully understand what "xg" does [15:58:01] well, its the month ;p [15:58:06] right [15:58:08] but some sort of specialness [15:58:31] yeah, makes sense [15:59:47] hmm [15:59:59] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 (owner: 10Addshore) [16:00:06] I might even have a better idea :P [16:00:20] ok [16:00:24] ? [16:01:08] [13WikibaseDataModel] 15thiemowmde pushed 1 new commit to 06master: 02http://git.io/Bfuq1Q [16:01:08] 13WikibaseDataModel/06master 1486124d3 15thiemowmde: Merge pull request #43 from wmde/termlist... [16:01:30] well, if language->getDefaultDateFormat only ever uses 'mdy' just in different orders we can get what we want there [16:01:44] and not have to worry about anything once it has all the special stuff in it [16:02:15] hmmm, ok [16:02:38] *tests* [16:02:58] hmm, I see many fails in my console :P maybe this wont work xD [16:03:03] :( [16:06:18] !isset( $this->dateFormatStrings[$type][$pref] where $pref = 'dmy' [16:06:21] gragh, stupic class [16:09:43] [13WikibaseDataModel] 15JeroenDeDauw created 06aliaslist (+1 new commit): 02http://git.io/qvqWqA [16:09:43] 13WikibaseDataModel/06aliaslist 14a7b3546 15jeroendedauw: Added setGroup and removeByLanguage to AliasGroupList [16:15:43] [13WikibaseDataModel] 15thiemowmde pushed 1 new commit to 06master: 02http://git.io/CHq71A [16:15:43] 13WikibaseDataModel/06master 14b87dbe7 15thiemowmde: Merge pull request #45 from wmde/aliaslist... [16:16:20] [13WikibaseDataModel] 15thiemowmde pushed 1 new commit to 06master: 02http://git.io/_4BBlg [16:16:20] 13WikibaseDataModel/06master 14c0b7a32 15thiemowmde: Merge pull request #44 from wmde/DescriptionList... [16:17:09] aude: Im going to put your change in the middle of my chain (just after the patchset that adds the tests with lots of languages skipped) [16:17:18] [13WikibaseDataModel] 15JeroenDeDauw created 06noop (+1 new commit): 02http://git.io/gdnvNA [16:17:18] 13WikibaseDataModel/06noop 14373ea4c 15jeroendedauw: TermList::removeByLanguage no does no-op when no term is found [16:17:24] if thats cool with you :) [16:17:38] addshore: ok [16:17:53] [13WikibaseDataModel] 15thiemowmde pushed 1 new commit to 06master: 02http://git.io/dT-SJg [16:17:53] 13WikibaseDataModel/06master 14bea4de6 15thiemowmde: Merge pull request #46 from wmde/noop... [16:17:53] wikidata is back on wmf19 of wikibase / wikidata build [16:18:02] okay :) [16:18:09] hopefully no caching problems [16:18:30] people might need to purge pages if dates are formatted wrong [16:19:04] Q76 looks good when purged [16:27:21] [13WikibaseDataModel] 15JeroenDeDauw created 06noempty (+2 new commits): 02http://git.io/NOrM5A [16:27:21] 13WikibaseDataModel/06noempty 143251948 15jeroendedauw: Filter empty groups out in AliasGroupList constructor [16:27:21] 13WikibaseDataModel/06noempty 14883a9dd 15jeroendedauw: Filter empty groups out in AliasGroupList::setGroup [16:27:42] [13WikibaseDataModel] 15JeroenDeDauw opened pull request #47: Noempty (06master...06noempty) 02http://git.io/w252SQ [16:29:04] aude: your patch fixes 25 more languages :) [16:29:12] wait we branch not next week [16:29:14] week after [16:29:21] because of easter / holidays [16:29:37] https://www.mediawiki.org/wiki/Wikidata_deployment [16:31:07] (03PS2) 10Addshore: Normalise digits in time formatter [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124868 (owner: 10Aude) [16:31:19] ? [16:33:25] [13WikibaseDataModel] 15JeroenDeDauw created 06termc (+1 new commit): 02http://git.io/Du96oQ [16:33:25] 13WikibaseDataModel/06termc 14562471c 15jeroendedauw: Term implements Comparable [16:33:35] [13WikibaseDataModel] 15JeroenDeDauw opened pull request #48: Term implements Comparable (06master...06termc) 02http://git.io/ilKUBg [16:34:44] sjoerddekok: we switched back to previous version of wikibase etc. [16:34:58] Ah. :O [16:35:00] that means we can take time / care to fix time formatting issues [16:35:09] :) [16:35:14] (03PS3) 10Thiemo Mättig (WMDE): [WIP] Fix "@param null $param" documentations [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124816 [16:35:17] unfortunately, the issue with the entity selector is back :( [16:35:23] Ugh [16:35:46] i don't know if that is something we can backport [16:35:53] have to pick lesser of evils [16:35:54] Can you add something about it to the project chat? [16:35:57] Thiemo_WMDE: review https://github.com/wmde/WikibaseDataModel/pull/47 ? I want to do some more stuff in AliasGroup [16:36:03] um, sure [16:36:10] i'll let lydia do [16:36:16] There is a question: https://www.wikidata.org/wiki/Wikidata:Project_chat#Time_localisation_bug [16:36:35] basically, there are issues in various non-english languages [16:36:40] various issues* [16:36:48] Not only dates? [16:36:53] only dates [16:36:55] Ah [16:37:14] folks will be able to test on beta wikidata [16:39:40] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124868 (owner: 10Aude) [16:42:26] why is trim such a fail [16:42:41] left a note [16:43:03] why is there no goodTrim() :P [16:43:10] heh [16:43:30] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124816 (owner: 10Thiemo Mättig (WMDE)) [16:43:45] string(6) "‏ Y" [16:43:56] why you no trim spaces trim [16:46:20] why doesnt the regex even strip it :< [16:47:03] seems now is ok time to go home / eat [16:47:08] i'll be on later [16:48:26] daniel food arrive = time to go :) [16:48:50] :) [16:59:52] (03PS5) 10Addshore: DNM WIP Fix formatting in various languages [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 [17:07:27] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 (owner: 10Addshore) [17:07:38] (03PS10) 10Bene: Support getting single sitelinks with ClientSiteLinkLookup [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123449 [17:07:43] woo! that means 24 languages left [17:10:57] [travis-ci] wmde/WikibaseDataModel/aliaslist/a7b3546 : jeroendedauw The build has errored. http://travis-ci.org/wmde/WikibaseDataModel/builds/22622435 [17:12:28] (03PS11) 10Bene: Support getting single sitelinks with ClientSiteLinkLookup [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123449 [17:15:48] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123449 (owner: 10Bene) [17:19:30] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123449 (owner: 10Bene) [17:28:59] [13WikibaseDataModel] 15JeroenDeDauw pushed 1 new commit to 06noempty: 02http://git.io/UdC32Q [17:28:59] 13WikibaseDataModel/06noempty 14766cf76 15jeroendedauw: AliasGroup implements Comparable [17:34:59] Thiemo_WMDE: all problems fixed. It works now :D https://gerrit.wikimedia.org/r/#/c/123449/ [17:35:04] and does not have to handle with numeric ids at all \o/ [17:35:07] also @ JeroenDeDauw [17:39:11] (03PS6) 10Addshore: DNM WIP Fix formatting in various languages [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 [17:40:37] aude: ^^ 7 left [17:40:46] im off for a few hours now though :) [17:47:36] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 (owner: 10Addshore) [17:48:37] Lydia_WMDE: aaaaaaaaaaa, y u still meeting [17:50:12] JeroenDeDauw: hopefully over soonish [17:53:08] [13WikibaseDataModel] 15JeroenDeDauw pushed 3 new commits to 06noempty: 02http://git.io/_E8p8g [17:53:08] 13WikibaseDataModel/06noempty 1433bb666 15jeroendedauw: Remove duplicate aliases in AliasGroup [17:53:08] 13WikibaseDataModel/06noempty 14dff5fc2 15jeroendedauw: Make AliasGroup Countable [17:53:08] 13WikibaseDataModel/06noempty 1493f0ec5 15jeroendedauw: Filter out empty aliases in AliasGroup [18:01:37] addshore: great [18:02:21] (03CR) 10Thiemo Mättig (WMDE): [C: 04-1] "On a really good way, thanks a lot." (034 comments) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123449 (owner: 10Bene) [18:07:31] Thiemo_WMDE: so I should readd the sitelinks to the test function? [18:08:36] addshore: aude: still having the question? [18:10:48] [13WikibaseDataModel] 15JeroenDeDauw pushed 1 new commit to 06noempty: 02http://git.io/cJRrog [18:10:48] 13WikibaseDataModel/06noempty 146017488 15jeroendedauw: Trim aliases in AliasGroup [18:15:23] (03CR) 10Jeroen De Dauw: Support getting single sitelinks with ClientSiteLinkLookup (031 comment) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123449 (owner: 10Bene) [18:15:49] benestar: your commit wins the bikeshed of the week award :) [18:15:58] dennyvrandecic: not really but we are working on localised date formatting [18:16:17] for a year in croatian, seems to be formatted '2014.' [18:16:36] was wondering if to simplify, if trimming the dot would be ok [18:16:54] it is not wonderful, but ok [18:17:00] but turns out, we can't do that way [18:17:06] so no trimming [18:17:35] ok. where is this to be displayed anyway? [18:17:44] formatting of all dates [18:17:49] for snaks [18:17:51] JeroenDeDauw: ok, now I'm confused... [18:18:09] should I depend on Title? [18:18:12] oh, don't we just use mediawikis time formatting? [18:18:21] or should I not depend on Title and rename it to pageName? [18:18:33] we are, but have to transform between our timestamp and the mw one [18:20:35] (03CR) 10Bene: "Oh guys, you confuse me..." (032 comments) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123449 (owner: 10Bene) [18:21:23] aude: ? ... it's ok, I don't have to understand it :) [18:21:29] aude: thx! [18:21:39] dennyvrandecic: sure [18:22:22] it's just so many details and we're going beyond what mediawiki does to support things like millions of years [18:22:28] [13Diff] 15JeroenDeDauw created 06testspsr4 (+1 new commit): 02http://git.io/EYdrAA [18:22:28] 13Diff/06testspsr4 14f1d314d 15jeroendedauw: Make the tests PSR-4 compliant and load them only in the PHPUnit bootstrap [18:22:42] and running in to limitations of php etc [18:23:06] ah, ok, that makes sense [18:23:23] one idea is to package this on its own [18:23:32] +1 [18:23:44] I guess formatting dates in this wider range could be quite a useful thing for more apps [18:24:15] definitely [18:24:43] a lot of what mediawiki does for dates should be a separate component + what we do to extend it [18:25:17] yes [18:28:31] (03CR) 10Jeroen De Dauw: Support getting single sitelinks with ClientSiteLinkLookup (031 comment) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123449 (owner: 10Bene) [18:29:40] (03PS12) 10Bene: Support getting single sitelinks with ClientSiteLinkLookup [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123449 [18:31:16] hoo|away: suppose you are really away, but if not https://gerrit.wikimedia.org/r/#/c/124897/ [18:31:43] i can merge myself in a few minutes (i'm sure no one mindes) [18:31:45] minds* [18:31:47] I'm not really away... [18:31:52] wrong nick :P [18:31:59] ok [18:32:02] hoo: Tarnung aufgeflogen :P [18:32:26] in case you missed, we are back on mw1.23-wmf19 and will stay on it until next deploy [18:32:39] due to time formating issues [18:32:43] Ok... and the core branch hasn't yet been made?! [18:33:22] it's wednesday [18:33:28] normally that is done on thursday [18:33:34] * aude checks [18:33:34] Oh crap, right [18:33:55] Forget that ... embarrassing [18:34:03] it's ok :) [18:34:27] no new core branch yet [18:34:52] Ok, approved :) [18:35:09] thanks [18:35:14] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123449 (owner: 10Bene) [18:35:39] rather people not complain about self merging [18:37:31] (03PS13) 10Bene: Support getting single sitelinks with ClientSiteLinkLookup [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123449 [18:43:26] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/123449 (owner: 10Bene) [18:45:31] [travis-ci] wmde/WikibaseDataModel/noop/373ea4c : jeroendedauw The build has errored. http://travis-ci.org/wmde/WikibaseDataModel/builds/22623016 [19:08:11] (03PS4) 10Thiemo Mättig (WMDE): Fix "@param null $param" documentations [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124816 [19:13:44] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124816 (owner: 10Thiemo Mättig (WMDE)) [19:14:36] (03CR) 10Aude: "looks good generally but noticed a typo" (031 comment) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124816 (owner: 10Thiemo Mättig (WMDE)) [19:25:57] [13Diff] 15JeroenDeDauw created 06psr4 (+1 new commit): 02http://git.io/8jptJw [19:25:57] 13Diff/06psr4 14d372995 15jeroendedauw: Make Diff PSR-4 compliant [19:26:07] [13Diff] 15JeroenDeDauw opened pull request #8: Make Diff PSR-4 compliant (06master...06psr4) 02http://git.io/ZDZgVg [19:26:17] [13Diff] 15JeroenDeDauw 04deleted 06testspsr4 at 14f1d314d: 02http://git.io/8RXgzw [19:26:26] (03PS3) 10Thiemo Mättig (WMDE): Rename option 'displayBadges' to 'badgeClassNames' [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124650 [19:28:52] (03CR) 10Thiemo Mättig (WMDE): Fix "@param null $param" documentations (031 comment) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124816 (owner: 10Thiemo Mättig (WMDE)) [19:28:58] [13Diff] 15JeroenDeDauw pushed 1 new commit to 06psr4: 02http://git.io/UNSpeg [19:28:58] 13Diff/06psr4 146ea2838 15jeroendedauw: Add branch alias [19:29:15] (03PS5) 10Thiemo Mättig (WMDE): Fix "@param null $param" documentations [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124816 [19:30:20] addshore: you want to merge https://github.com/wmde/Diff/pull/8 [19:31:12] Thiemo_WMDE: didn't you want this thing? https://github.com/wmde/WikibaseDataModel/pull/48 [19:32:48] [13WikibaseDataModel] 15thiemowmde comment on pull request #48 14562471c: Order is mixed. ;-) 02http://git.io/xfwGUA [19:33:08] [13WikibaseDataModel] 15thiemowmde pushed 1 new commit to 06master: 02http://git.io/XIDErw [19:33:08] 13WikibaseDataModel/06master 148f4dda6 15thiemowmde: Merge pull request #48 from wmde/termc... [19:34:22] Thiemo_WMDE: order is mixed?! huh? [19:34:34] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124650 (owner: 10Thiemo Mättig (WMDE)) [19:34:57] JeroenDeDauw: all places it the class use the order languageCode > text. [19:35:24] comparing the languageCode first is 0.0001ms faster ;) [19:36:50] Thiemo_WMDE: really? I would think you more often have a lang code match than a text match ;p [19:37:10] oh, true. [19:37:42] Thiemo_WMDE: more similar goodies: https://github.com/wmde/WikibaseDataModel/pull/47/ [19:37:59] JeroenDeDauw: processing https://github.com/wmde/Diff/pull/8/files now... [19:38:07] haha [19:38:17] That took me quite some time [19:38:21] Doing very boring stuff [19:38:34] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124816 (owner: 10Thiemo Mättig (WMDE)) [19:39:28] [13Diff] 15thiemowmde closed pull request #8: Make Diff PSR-4 compliant (06master...06psr4) 02http://git.io/ZDZgVg [19:45:49] [13WikibaseDataModel] 15thiemowmde comment on pull request #47 146017488: Uh, really? So many things magically done in a constructor? Will the other classes (Label and Description) do something similar? 02http://git.io/kp2QgQ [19:47:10] (03CR) 10WikidataJenkins: "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124982 (owner: 10L10n-bot) [19:50:05] [13WikibaseDataModel] 15thiemowmde comment on pull request #47 146017488: What if a language exists and I try to call setGroup for that language, but my new group is empty? This will be silently ignored and the old non-empty group will still be there. Either remove the old group or throw an exception if you don't want to accept empty groups. 02http://git.io/YxssLA [19:56:30] [travis-ci] wmde/WikibaseDataModel/termc/562471c : jeroendedauw The build has errored. http://travis-ci.org/wmde/WikibaseDataModel/builds/22624280 [19:59:01] GerardM: Any idea where the WikidataQuery thing went? [19:59:26] I guess hosting it at http://208.80.153.172/wdq/ isn't a very good idea ;-) [20:01:36] (03PS2) 10Daniel Kinzler: Factor hard constraints out of EntityContent [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124984 [20:07:46] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124984 (owner: 10Daniel Kinzler) [20:26:07] [travis-ci] wmde/WikibaseDataModel/noempty/766cf76 : jeroendedauw The build passed. http://travis-ci.org/wmde/WikibaseDataModel/builds/22628520 [20:29:26] (03CR) 10Thiemo Mättig (WMDE): [C: 04-1] "Fix broken test, fix nitpicks, +2, profit." (032 comments) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124984 (owner: 10Daniel Kinzler) [20:46:32] [travis-ci] wmde/WikibaseDataModel/noempty/93f0ec5 : jeroendedauw The build passed. http://travis-ci.org/wmde/WikibaseDataModel/builds/22630694 [21:05:05] [travis-ci] wmde/WikibaseDataModel/noempty/6017488 : jeroendedauw The build passed. http://travis-ci.org/wmde/WikibaseDataModel/builds/22632018 [21:26:36] [travis-ci] wmde/Diff#17 (testspsr4 - f1d314d : jeroendedauw): The build has errored. [21:26:36] [travis-ci] Change view : https://github.com/wmde/Diff/commit/f1d314d09195 [21:26:36] [travis-ci] Build details : http://travis-ci.org/wmde/Diff/builds/22633111 [21:29:55] [13WikibaseDataModel] 15thiemowmde comment on pull request #47 146017488: According to this a string with spaces is an Alias. According to the constructor it is not. Hm, it's not public, so not a real problem. Still a bit confusing to have such a trivial method (even if I understand it's required for array_filter). 02http://git.io/SsaVlg [21:30:17] [13WikibaseDataModel] 15thiemowmde closed pull request #47: Improvements to AliasGroup and AliasGroupList (06master...06noempty) 02http://git.io/w252SQ [21:46:56] [travis-ci] wmde/Diff#19 (psr4 - d372995 : jeroendedauw): The build has errored. [21:46:56] [travis-ci] Change view : https://github.com/wmde/Diff/commit/d372995a9803 [21:46:56] [travis-ci] Build details : http://travis-ci.org/wmde/Diff/builds/22637471 [21:56:42] [13WikibaseDataModel] 15JeroenDeDauw created 06aliasgroupz (+2 new commits): 02http://git.io/_Ae0Dg [21:56:42] 13WikibaseDataModel/06aliasgroupz 140d7f698 15jeroendedauw: Use anon method [21:56:42] 13WikibaseDataModel/06aliasgroupz 146f7f3d3 15jeroendedauw: Change behaviour of setting empty groups in AliasGroupList [21:56:53] Thiemo_WMDE: https://github.com/wmde/WikibaseDataModel/pull/49 :) [21:56:57] [13WikibaseDataModel] 15JeroenDeDauw opened pull request #49: Thiemo can haz! (06master...06aliasgroupz) 02http://git.io/Uqa3pA [21:59:04] (03PS1) 10Jeroen De Dauw: Use Diff 1.0 [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/125013 [21:59:22] [13WikibaseDataModel] 15thiemowmde comment on pull request #49 146f7f3d3: Isn't PHP great? :-) 02http://git.io/FybFtg [22:00:17] [13WikibaseDataModel] 15thiemowmde pushed 1 new commit to 06aliasgroupz: 02http://git.io/F4Pq5A [22:00:17] 13WikibaseDataModel/06aliasgroupz 1485fc89d 15thiemowmde: Update AliasGroupList.php [22:01:09] [13WikibaseDataModel] 15thiemowmde pushed 1 new commit to 06master: 02http://git.io/ks87DQ [22:01:09] 13WikibaseDataModel/06master 140bf7e82 15thiemowmde: Merge pull request #49 from wmde/aliasgroupz... [22:01:14] [travis-ci] wmde/Diff#21 (psr4 - 6ea2838 : jeroendedauw): The build has errored. [22:01:14] [travis-ci] Change view : https://github.com/wmde/Diff/compare/d372995a9803...6ea2838fb791 [22:01:14] [travis-ci] Build details : http://travis-ci.org/wmde/Diff/builds/22637701 [22:01:19] [13WikibaseDataModel] 15thiemowmde 04deleted 06aliasgroupz at 1485fc89d: 02http://git.io/O5KXfQ [22:01:37] (03CR) 10Jeroen De Dauw: [C: 031] Rename option 'displayBadges' to 'badgeClassNames' [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124650 (owner: 10Thiemo Mättig (WMDE)) [22:01:41] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/125013 (owner: 10Jeroen De Dauw) [22:02:54] (03CR) 10Jeroen De Dauw: [C: 04-1] "Still missing null in many places" (033 comments) [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124816 (owner: 10Thiemo Mättig (WMDE)) [22:04:39] [13WikibaseDataModel] 15JeroenDeDauw created 06diff (+1 new commit): 02http://git.io/ezutxQ [22:04:39] 13WikibaseDataModel/06diff 14e1e5e6d 15jeroendedauw: use Diff 1.0 [22:04:41] Thiemo_WMDE: https://github.com/wmde/WikibaseDataModel/pull/50 [22:06:42] (03PS2) 10Jeroen De Dauw: Use Diff 1.0 and DataModel 0.7.3 [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/125013 [22:08:55] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/125013 (owner: 10Jeroen De Dauw) [22:42:13] [travis-ci] wmde/WikibaseDataModel/aliasgroupz/6f7f3d3 : jeroendedauw The build has errored. http://travis-ci.org/wmde/WikibaseDataModel/builds/22647156 [22:53:51] [travis-ci] wmde/WikibaseDataModel/aliasgroupz/85fc89d : thiemowmde The build has errored. http://travis-ci.org/wmde/WikibaseDataModel/builds/22647432 [23:01:27] evening [23:16:13] (03PS7) 10Addshore: Fix formatting dates in most languages [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 [23:18:39] (03PS2) 10Addshore: Add initial test for MwTimeIsoFormatter::format [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124830 (owner: 10Aude) [23:18:55] (03CR) 10Addshore: "Rebased on top of the chain so it should now pass" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124830 (owner: 10Aude) [23:19:25] (03PS2) 10Addshore: Stop ALWAYS using regex to fix dateFormats [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124833 [23:19:30] (03PS3) 10Addshore: Normalise digits in time formatter [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124868 (owner: 10Aude) [23:19:36] (03PS8) 10Addshore: Fix formatting dates in most languages [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 [23:19:38] (03PS3) 10Addshore: Add initial test for MwTimeIsoFormatter::format [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124830 (owner: 10Aude) [23:19:39] [travis-ci] wmde/WikibaseDataModel/diff/e1e5e6d : jeroendedauw The build passed. http://travis-ci.org/wmde/WikibaseDataModel/builds/22647753 [23:20:59] (03PS3) 10Addshore: Handle exceptions in ClaimDifferenceVisualizer [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124773 (owner: 10Aude) [23:21:48] (03CR) 10Addshore: [C: 032] Normalise digits in time formatter [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124868 (owner: 10Aude) [23:26:06] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 (owner: 10Addshore) [23:28:09] (03PS9) 10Addshore: Fix formatting dates in most languages [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 [23:28:15] (03PS4) 10Addshore: Add initial test for MwTimeIsoFormatter::format [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124830 (owner: 10Aude) [23:34:12] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124830 (owner: 10Aude) [23:39:50] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124833 (owner: 10Addshore) [23:43:53] [13WikibaseDataModel] 15JeroenDeDauw pushed 1 new commit to 06master: 02http://git.io/pYMhrw [23:43:53] 13WikibaseDataModel/06master 14d29f743 15Jeroen De Dauw: Update RELEASE-NOTES.md [23:45:53] (03CR) 10WikidataJenkins: [V: 032] "Build Successful" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124868 (owner: 10Aude) [23:46:03] [13WikibaseDataModel] 15JeroenDeDauw pushed 1 new commit to 06master: 02http://git.io/bT25yw [23:46:03] 13WikibaseDataModel/06master 142c8f49d 15Jeroen De Dauw: Update Description.php [23:46:18] [13WikibaseDataModel] 15JeroenDeDauw pushed 1 new commit to 06master: 02http://git.io/8CJZgA [23:46:18] 13WikibaseDataModel/06master 14070dddb 15Jeroen De Dauw: Update Label.php [23:53:10] (03CR) 10WikidataJenkins: [V: 04-1] "Build Failed" [extensions/Wikibase] - 10https://gerrit.wikimedia.org/r/124836 (owner: 10Addshore)