jenkins-bot has submitted this change and it was merged.

Change subject: findHooks.php: Only look at PHP files, not silly things
......................................................................


findHooks.php: Only look at PHP files, not silly things

Change-Id: I29280dfe629126c71adfb65ac1a59394bc459dc4
(cherry picked from commit 4b4ace0e003463fef467ce0812b676cff5382411)
---
M maintenance/findHooks.php
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Chad: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/maintenance/findHooks.php b/maintenance/findHooks.php
index d44581c..52818d7 100644
--- a/maintenance/findHooks.php
+++ b/maintenance/findHooks.php
@@ -315,7 +315,7 @@
                foreach ( $iterator as $info ) {
                        // Ignore directories, ignore json (installer and api 
i18n),
                        // ignore extension-less files like HISTORY
-                       if ( $info->isFile() && $info->getExtension() !== 
'json' && $info->getExtension()
+                       if ( $info->isFile() && $info->getExtension() === 'php'
                                // Skip this file as it contains text that 
looks like a bad wfRunHooks() call
                                && $info->getRealPath() !== __FILE__
                        ) {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I29280dfe629126c71adfb65ac1a59394bc459dc4
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: REL1_27
Gerrit-Owner: Chad <ch...@wikimedia.org>
Gerrit-Reviewer: Chad <ch...@wikimedia.org>
Gerrit-Reviewer: Parent5446 <tylerro...@gmail.com>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to