X-Git-Url: http://git.cyclocoop.org/?a=blobdiff_plain;f=maintenance%2Ffindhooks.php;h=c7b7a0d2b4036f8ae87c8e0d3feb12612b8cf1bb;hb=b9f1fef0c2f6116db6b252dcc6f274bd7d8cb22b;hp=6c349099567bbef68d58b8178358ce5059cb9a52;hpb=1b6be6ad2e044d02fccd6d16940ee8fd9d399468;p=lhc%2Fweb%2Fwiklou.git diff --git a/maintenance/findhooks.php b/maintenance/findhooks.php index 6c34909956..c7b7a0d2b4 100644 --- a/maintenance/findhooks.php +++ b/maintenance/findhooks.php @@ -12,7 +12,21 @@ * * Any instance of wfRunHooks that doesn't meet these parameters will be noted. * - * @file + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License along + * with this program; if not, write to the Free Software Foundation, Inc., + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. + * http://www.gnu.org/copyleft/gpl.html + * * @ingroup Maintenance * * @author Ashar Voultoiz @@ -20,162 +34,174 @@ * @license http://www.gnu.org/copyleft/gpl.html GNU General Public Licence 2.0 or later */ -/** This is a command line script*/ -require('commandLine.inc'); -# GLOBALS +require_once( dirname(__FILE__) . '/Maintenance.php' ); + +class FindHooks extends Maintenance { + public function __construct() { + parent::__construct(); + $this->mDescription = "Find hooks that are undocumented, missing, or just plain wrong"; + $this->addOption( 'online', 'Check against mediawiki.org hook documentation' ); + } + + protected function getDbType() { + return Maintenance::DB_NONE; + } -$doc = $IP . '/docs/hooks.txt'; -$pathinc = array( - $IP.'/', - $IP.'/includes/', - $IP.'/includes/api/', - $IP.'/includes/db/', - $IP.'/includes/diff/', - $IP.'/includes/filerepo/', - $IP.'/includes/parser/', - $IP.'/includes/search/', - $IP.'/includes/specials/', - $IP.'/includes/upload/', - $IP.'/languages/', - $IP.'/maintenance/', - $IP.'/skins/', -); + public function execute() { + global $IP; -# FUNCTIONS + $documented = $this->getHooksFromDoc( $IP . '/docs/hooks.txt' ); + $potential = array(); + $bad = array(); + $pathinc = array( + $IP.'/', + $IP.'/includes/', + $IP.'/includes/api/', + $IP.'/includes/db/', + $IP.'/includes/diff/', + $IP.'/includes/filerepo/', + $IP.'/includes/parser/', + $IP.'/includes/search/', + $IP.'/includes/specials/', + $IP.'/includes/upload/', + $IP.'/languages/', + $IP.'/maintenance/', + $IP.'/skins/', + ); -/** - * @return array of documented hooks - */ -function getHooksFromDoc() { - global $doc, $options; - if( isset( $options['online'] ) ){ - // All hooks - $allhookdata = Http::get( 'http://www.mediawiki.org/w/api.php?action=query&list=categorymembers&cmtitle=Category:MediaWiki_hooks&cmlimit=500&format=php' ); - $allhookdata = unserialize( $allhookdata ); - $allhooks = array(); - foreach( $allhookdata['query']['categorymembers'] as $page ) { - $found = preg_match( '/Manual\:Hooks\/([a-zA-Z0-9- :]+)/', $page['title'], $matches ); - if( $found ) { - $hook = str_replace( ' ', '_', $matches[1] ); - $allhooks[] = $hook; - } + foreach( $pathinc as $dir ) { + $potential = array_merge( $potential, $this->getHooksFromPath( $dir ) ); + $bad = array_merge( $bad, $this->getBadHooksFromPath( $dir ) ); } - // Removed hooks - $oldhookdata = Http::get( 'http://www.mediawiki.org/w/api.php?action=query&list=categorymembers&cmtitle=Category:Removed_hooks&cmlimit=500&format=php' ); - $oldhookdata = unserialize( $oldhookdata ); - $removed = array(); - foreach( $oldhookdata['query']['categorymembers'] as $page ) { - $found = preg_match( '/Manual\:Hooks\/([a-zA-Z0-9- :]+)/', $page['title'], $matches ); - if( $found ) { - $hook = str_replace( ' ', '_', $matches[1] ); - $removed[] = $hook; + + $potential = array_unique( $potential ); + $bad = array_unique( $bad ); + $todo = array_diff( $potential, $documented ); + $deprecated = array_diff( $documented, $potential ); + + // let's show the results: + $this->printArray('Undocumented', $todo ); + $this->printArray('Documented and not found', $deprecated ); + $this->printArray('Unclear hook calls', $bad ); + + if ( count( $todo ) == 0 && count( $deprecated ) == 0 && count( $bad ) == 0 ) + $this->output( "Looks good!\n" ); + } + + /** + * Get the hook documentation, either locally or from mediawiki.org + * @return array of documented hooks + */ + private function getHooksFromDoc( $doc ) { + if( $this->hasOption( 'online' ) ){ + // All hooks + $allhookdata = Http::get( 'http://www.mediawiki.org/w/api.php?action=query&list=categorymembers&cmtitle=Category:MediaWiki_hooks&cmlimit=500&format=php' ); + $allhookdata = unserialize( $allhookdata ); + $allhooks = array(); + foreach( $allhookdata['query']['categorymembers'] as $page ) { + $found = preg_match( '/Manual\:Hooks\/([a-zA-Z0-9- :]+)/', $page['title'], $matches ); + if( $found ) { + $hook = str_replace( ' ', '_', $matches[1] ); + $allhooks[] = $hook; + } + } + // Removed hooks + $oldhookdata = Http::get( 'http://www.mediawiki.org/w/api.php?action=query&list=categorymembers&cmtitle=Category:Removed_hooks&cmlimit=500&format=php' ); + $oldhookdata = unserialize( $oldhookdata ); + $removed = array(); + foreach( $oldhookdata['query']['categorymembers'] as $page ) { + $found = preg_match( '/Manual\:Hooks\/([a-zA-Z0-9- :]+)/', $page['title'], $matches ); + if( $found ) { + $hook = str_replace( ' ', '_', $matches[1] ); + $removed[] = $hook; + } } + return array_diff( $allhooks, $removed ); + } else { + $m = array(); + $content = file_get_contents( $doc ); + preg_match_all( "/\n'(.*?)'/", $content, $m ); + return array_unique( $m[1] ); } - return array_diff( $allhooks, $removed ); - } else { - $m = array(); - $content = file_get_contents( $doc ); - preg_match_all( "/\n'(.*?)'/", $content, $m ); - return array_unique( $m[1] ); } -} -/** - * Get hooks from a PHP file - * @param $file Full filename to the PHP file. - * @return array of hooks found. - */ -function getHooksFromFile( $file ) { - $content = file_get_contents( $file ); - $m = array(); - preg_match_all( '/wfRunHooks\(\s*([\'"])(.*?)\1/', $content, $m); - return $m[2]; -} + /** + * Get hooks from a PHP file + * @param $file Full filename to the PHP file. + * @return array of hooks found. + */ + private function getHooksFromFile( $file ) { + $content = file_get_contents( $file ); + $m = array(); + preg_match_all( '/wfRunHooks\(\s*([\'"])(.*?)\1/', $content, $m); + return $m[2]; + } -/** - * Get hooks from the source code. - * @param $path Directory where the include files can be found - * @return array of hooks found. - */ -function getHooksFromPath( $path ) { - $hooks = array(); - if( $dh = opendir($path) ) { - while(($file = readdir($dh)) !== false) { - if( filetype($path.$file) == 'file' ) { - $hooks = array_merge( $hooks, getHooksFromFile($path.$file) ); + /** + * Get hooks from the source code. + * @param $path Directory where the include files can be found + * @return array of hooks found. + */ + private function getHooksFromPath( $path ) { + $hooks = array(); + if( $dh = opendir($path) ) { + while(($file = readdir($dh)) !== false) { + if( filetype($path.$file) == 'file' ) { + $hooks = array_merge( $hooks, $this->getHooksFromFile($path.$file) ); + } } + closedir($dh); } - closedir($dh); + return $hooks; } - return $hooks; -} -/** - * Get bad hooks (where the hook name could not be determined) from a PHP file - * @param $file Full filename to the PHP file. - * @return array of bad wfRunHooks() lines - */ -function getBadHooksFromFile( $file ) { - $content = file_get_contents( $file ); - $m = array(); - # We want to skip the "function wfRunHooks()" one. :) - preg_match_all( '/(?getBadHooksFromFile($path.$file) ); + } } + closedir($dh); } - closedir($dh); + return $hooks; } - return $hooks; -} -/** - * Nicely output the array - * @param $msg A message to show before the value - * @param $arr An array - * @param $sort Boolean : wheter to sort the array (Default: true) - */ -function printArray( $msg, $arr, $sort = true ) { - if($sort) asort($arr); - foreach($arr as $v) echo "$msg: $v\n"; -} - -# MAIN - -$documented = getHooksFromDoc($doc); -$potential = array(); -$bad = array(); -foreach( $pathinc as $dir ) { - $potential = array_merge( $potential, getHooksFromPath( $dir ) ); - $bad = array_merge( $bad, getBadHooksFromPath( $dir ) ); + /** + * Nicely output the array + * @param $msg A message to show before the value + * @param $arr An array + * @param $sort Boolean : wheter to sort the array (Default: true) + */ + private function printArray( $msg, $arr, $sort = true ) { + if($sort) asort($arr); + foreach($arr as $v) $this->output( "$msg: $v\n" ); + } } -$potential = array_unique( $potential ); -$bad = array_unique( $bad ); -$todo = array_diff( $potential, $documented ); -$deprecated = array_diff( $documented, $potential ); - -// let's show the results: -printArray('undocumented', $todo ); -printArray('not found', $deprecated ); -printArray('unclear hook calls', $bad ); - -if ( count( $todo ) == 0 && count( $deprecated ) == 0 && count( $bad ) == 0 ) - echo "Looks good!\n"; +$maintClass = "FindHooks"; +require_once( DO_MAINTENANCE );