[02:17:09] $mapName = "ImageMap_" . ++self::$id . '_' . mt_rand( 0, 0x7fffffff ); [02:17:09] $mapHTML = "\n$mapHTML\n"; [02:17:45] I figured out why no one ever tried to write parser tests for ImageMap [02:59:40] legoktm: then there's birthday paradox that makes collisions way more likely than anybody expected [04:20:34] i want DPL fixed for my birthday [13:54:32] legoktm: Looks similar to TMH tests, which means there might be a fix. [13:55:08] wait, why does it also need a random iD? [17:24:01] Krinkle: yeah, Tim and I discussed yesterday how to fix it. It uses a random ID so people could put in their sitenotice or something like that and not have it conflict with other image maps already on the page [19:36:41] tgr: did you have any other comments for the ShellCommandFactory patch? [19:40:12] legoktm: no, I would merge it as it is, I just haven't gotten around to it [19:41:09] tgr: do you mind if I do? [19:41:30] not at all, thanks [19:53:12] MaxSem: if Shell\Command supported providing stdin we could get rid of symfony/process in SyntaxHighlight [19:55:26] sounds like a plan [20:03:26] Warning: Invalid argument supplied for foreach() in /srv/mediawiki/php-1.31.0-wmf.3/extensions/MassMessage/includes/content/MassMessageListContent.php on line 186 [20:03:26] Warning: Invalid argument supplied for foreach() in /srv/mediawiki/php-1.31.0-wmf.3/extensions/MassMessage/includes/content/MassMessageListContent.php on line 288 [20:03:26] Warning: Invalid argument supplied for foreach() in /srv/mediawiki/php-1.31.0-wmf.3/extensions/MassMessage/includes/content/MassMessageListContent.php on line 90 [20:03:32] All new in wmf.4 [20:08:41] hmm. generateLocalAutoload.php is picking up classes from LocalSettings.php. That seems wrong. [20:18:29] * anomie hacks around it for now with eval.