IAlex has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/101650


Change subject: Use a better regular expression to find documented hooks in 
findHooks.php
......................................................................

Use a better regular expression to find documented hooks in findHooks.php

Added a ":" after the second quote so that only strings in the form
'Hook': are matched, otherwise the line 1172 of hooks.txt provoques the
the following message:
Documented and not found: value
because the line starts with 'value'.

Change-Id: I1d603e3fa10dcabf25e31f6a71aad233571738b5
---
M maintenance/findHooks.php
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/50/101650/1

diff --git a/maintenance/findHooks.php b/maintenance/findHooks.php
index a936436..24b23d9 100644
--- a/maintenance/findHooks.php
+++ b/maintenance/findHooks.php
@@ -142,7 +142,7 @@
        private function getHooksFromLocalDoc( $doc ) {
                        $m = array();
                        $content = file_get_contents( $doc );
-                       preg_match_all( "/\n'(.*?)'/", $content, $m );
+                       preg_match_all( "/\n'(.*?)':/", $content, $m );
                        return array_unique( $m[1] );
        }
 

-- 
To view, visit https://gerrit.wikimedia.org/r/101650
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1d603e3fa10dcabf25e31f6a71aad233571738b5
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: IAlex <coderev...@emsenhuber.ch>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to