jenkins-bot has submitted this change and it was merged. Change subject: findHooks.php: Work also on .inc and .php5 ......................................................................
findHooks.php: Work also on .inc and .php5 Also fix the use of RecursiveDirectoryIterator::SKIP_DOTS, which was passed to the wrong constructor. Follows I29280dfe629126c71adfb65ac1a59394bc459dc4 Follows I6c01b8768f9f70152e320d254f8025a0ffde8c62 Change-Id: Ib35ca10b809afd5f6da80cd5edd1a71479c917a7 --- M maintenance/findHooks.php 1 file changed, 4 insertions(+), 5 deletions(-) Approvals: Legoktm: Looks good to me, approved jenkins-bot: Verified diff --git a/maintenance/findHooks.php b/maintenance/findHooks.php index 89e0311..bc36141 100644 --- a/maintenance/findHooks.php +++ b/maintenance/findHooks.php @@ -305,17 +305,16 @@ if ( $recurse === self::FIND_RECURSIVE ) { $iterator = new RecursiveIteratorIterator( - new RecursiveDirectoryIterator( $dir ), - RecursiveIteratorIterator::SELF_FIRST | RecursiveDirectoryIterator::SKIP_DOTS + new RecursiveDirectoryIterator( $dir, RecursiveDirectoryIterator::SKIP_DOTS ), + RecursiveIteratorIterator::SELF_FIRST ); } else { $iterator = new DirectoryIterator( $dir ); } foreach ( $iterator as $info ) { - // Ignore directories, ignore json (installer and api i18n), - // ignore extension-less files like HISTORY - if ( $info->isFile() && $info->getExtension() === 'php' + // Ignore directories, work only on php files, + if ( $info->isFile() && in_array( $info->getExtension(), [ 'php', 'php5', 'inc' ] ) // 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/293972 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib35ca10b809afd5f6da80cd5edd1a71479c917a7 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/core Gerrit-Branch: master Gerrit-Owner: Umherirrender <umherirrender_de...@web.de> Gerrit-Reviewer: Chad <ch...@wikimedia.org> Gerrit-Reviewer: Krinkle <krinklem...@gmail.com> Gerrit-Reviewer: Legoktm <legoktm.wikipe...@gmail.com> 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