jenkins-bot has submitted this change and it was merged. Change subject: Support short array syntax in findHooks.php ......................................................................
Support short array syntax in findHooks.php Bug: T135257 Change-Id: I19d4f073ff730821230ccb28d29df303fbaccbc2 --- M maintenance/findHooks.php 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Legoktm: Looks good to me, approved jenkins-bot: Verified diff --git a/maintenance/findHooks.php b/maintenance/findHooks.php index 7304a2a..c91d824 100644 --- a/maintenance/findHooks.php +++ b/maintenance/findHooks.php @@ -244,11 +244,11 @@ // Comma for second argument '(?:\s*(,))?' . // Second argument must start with array to be processed - '(?:\s*array\s*\(' . + '(?:\s*(?:array\s*\(|\[)' . // Matching inside array - allows one deep of brackets - '((?:[^\(\)]|\([^\(\)]*\))*)' . + '((?:[^\(\)\[\]]|\((?-1)\)|\[(?-1)\])*)' . // End - '\))?/', + '[\)\]])?/', $content, $m, PREG_SET_ORDER -- To view, visit https://gerrit.wikimedia.org/r/293874 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I19d4f073ff730821230ccb28d29df303fbaccbc2 Gerrit-PatchSet: 4 Gerrit-Project: mediawiki/core Gerrit-Branch: master Gerrit-Owner: Chad <ch...@wikimedia.org> Gerrit-Reviewer: Anomie <bjor...@wikimedia.org> Gerrit-Reviewer: Chad <ch...@wikimedia.org> Gerrit-Reviewer: Legoktm <legoktm.wikipe...@gmail.com> Gerrit-Reviewer: Parent5446 <tylerro...@gmail.com> Gerrit-Reviewer: Umherirrender <umherirrender_de...@web.de> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits