From b972ea9c59483a9a2c0be19450846ae7efc34dea Mon Sep 17 00:00:00 2001 From: Antoine Musso Date: Tue, 9 Jan 2007 06:09:39 +0000 Subject: [PATCH] - move stuff around - let us batch test i18n files from the extensions repository --- maintenance/language/checkExtensioni18n.php | 176 ++++++++++++++++---- 1 file changed, 143 insertions(+), 33 deletions(-) diff --git a/maintenance/language/checkExtensioni18n.php b/maintenance/language/checkExtensioni18n.php index 695927fb12..b0511f8e32 100644 --- a/maintenance/language/checkExtensioni18n.php +++ b/maintenance/language/checkExtensioni18n.php @@ -27,14 +27,29 @@ */ # -# Lacking documentation. An example is: +# Lacking documentation. Examples: # php checkExtensioni18n.php /opt/mw/extensions/CentralAuth/CentralAuth.i18n.php wgCentralAuthMessages +# php checkExtensioni18n.php --extdir /opt/mw/extensions/ # +# BUGS: cant guess registered extensions :) +# + +// Filename for the extension i18n files database: +define( 'EXT_I18N_DB', 'i18n.db' ); + +// Global parameters for checkLanguage.inc +$wgDisplayLevel = 2; +$wgChecks = array( 'untranslated', 'obsolete', 'variables', 'empty', 'whitespace', 'xhtml', 'chars' ); + + +$optionsWithArgs = array( 'extdir' ); + require_once( dirname(__FILE__).'/../commandLine.inc' ); require_once( 'languages.inc' ); require_once( 'checkLanguage.inc' ); + class extensionLanguages extends languages { private $mExt18nFilename, $mExtArrayName ; private $mExtArray; @@ -49,7 +64,43 @@ class extensionLanguages extends languages { if ( file_exists( $this->mExt18nFilename ) ) { require_once( $this->mExt18nFilename ); - $this->mExtArray = ${$this->mExtArrayName} ; + + $foundarray = false ; + + if( isset( ${$this->mExtArrayName} ) ) { + // File provided in the db file + $foundarray = ${$this->mExtArrayName} ; + } else { + // Provided array could not be found we try to guess it. + + # Using the extension path ($m[1]) and filename ($m[2]): + preg_match( '%.*/(.*)/(.*).i18n\.php%', $this->mExt18nFilename, $m); + $arPathCandidate = 'wg' . $m[1].'Messages'; + $arFileCandidate = 'wg' . $m[2].'Messages'; + $funcCandidate = "ef{$m[2]}Messages"; + + // Try them: + if( isset($$arPathCandidate) && is_array( $$arPathCandidate ) ) { + print "warning> messages from guessed path array \$$arPathCandidate.\n"; + $foundarray = $$arPathCandidate; + } elseif( isset($$arFileCandidate) && is_array( $$arFileCandidate ) ) { + print "warning> messages from guessed file array \$$arFileCandidate.\n"; + $foundarray = $$arFileCandidate; + } elseif( function_exists( $funcCandidate ) ) { + print "warning> messages build from guessed function {$funcCandidate}().\n"; + $foundarray = $funcCandidate(); + } + + # We are unlucky, return empty stuff + if(!$foundarray) { + print "ERROR> failed to guess an array to use.\n"; + $this->mExtArray = null; + $this->mLanguages = null; + return; + } + } + + $this->mExtArray = $foundarray ; $this->mLanguages = array_keys( $this->mExtArray ); } else { wfDie( "File $this->mExt18nFilename not found\n" ); @@ -73,55 +124,114 @@ class extensionLanguages extends languages { } +function checkExtensionLanguage( $filename, $arrayname ) { + global $wgGeneralMessages, $wgRequiredMessagesNumber; + + $extLanguages = new extensionLanguages($filename, $arrayname); + + // Stuff needed by the checkLanguage routine (globals) + $wgGeneralMessages = $extLanguages->getGeneralMessages(); + $wgRequiredMessagesNumber = count( $wgGeneralMessages['required'] ); + + $langs = $extLanguages->getLanguages(); + if( !$langs ) { + print "ERROR> \$$arrayname array does not exist.\n"; + return false; + } + + print "Found ". count($langs) . " languages : " . implode(' ', $langs) ."\n"; + foreach( $langs as $lang ) { + if( $lang == 'en' ) { + #print "Skipped english language\n"; + continue; + } + + checkLanguage( $extLanguages, $lang ); + } +} + +function checkExtensionRepository( $extdir, $db ) { + $fh = fopen( $extdir. '/' . $db, 'r' ); + + $line_number = 0; + while( $line = fgets( $fh ) ) { + $line_number++; + + // Ignore comments + if( preg_match( '/^#/', $line ) ) { + continue; + } + + // Load data from i18n database + $data = split( ' ', chop($line) ); + $i18n_file = @$data[0]; + $arrayname = @$data[1]; + + print "------------------------------------------------------\n"; + print "Checking $i18n_file (\$$arrayname).\n"; + + // Check data + if( !file_exists( $extdir . '/' . $i18n_file ) ) { + print "ERROR> $i18n_file not found ($db:$line_number).\n"; + continue; + } +# if( $arrayname == '' ) { +# print "warning> no array name for $i18n_file ($db:$line_number).\n"; +# } + + $i18n_file = $extdir . '/' . $i18n_file ; + + checkExtensionLanguage( $i18n_file, $arrayname ); + + print "\n"; + } +} + + function usage() { // Usage print << +Usage: + php checkExtensioni18n.php + php checkExtensioni18n.php --extdir END; die; } -// Check arguments -if ( isset( $argv[0] ) ) { +// Play with options and arguments +if( isset( $options['extdir'] ) ) { + $extdb = $options['extdir'] . '/' . EXT_I18N_DB ; - if (file_exists( $argv[0] ) ) { - $filename = $argv[0]; + if( file_exists( $extdb ) ) { + checkExtensionRepository( $options['extdir'], EXT_I18N_DB ); } else { - print "Unable to open file '{$argv[0]}'\n"; - usage(); + print "$extdb does not exist\n"; } - if ( isset( $argv[1] ) ) { - $arrayname = $argv[1]; - } else { - print "You must give an array name to be checked\n"; - usage(); - } } else { - usage(); -} + // Check arguments + if ( isset( $argv[0] ) ) { -$extLanguages = new extensionLanguages($filename, $arrayname); + if (file_exists( $argv[0] ) ) { + $filename = $argv[0]; + } else { + print "Unable to open file '{$argv[0]}'\n"; + usage(); + } -// Stuff needed by the checkLanguage routine (globals) -$wgGeneralMessages = $extLanguages->getGeneralMessages(); -$wgRequiredMessagesNumber = count( $wgGeneralMessages['required'] ); -$wgDisplayLevel = 2; -$wgChecks = array( 'untranslated', 'obsolete', 'variables', 'empty', 'whitespace', 'xhtml', 'chars' ); + if ( isset( $argv[1] ) ) { + $arrayname = $argv[1]; + } else { + print "You must give an array name to be checked\n"; + usage(); + } -foreach( $extLanguages->getLanguages() as $lang ) { - if( $lang == 'en' ) { - print "Skipped english language\n"; - continue; + checkExtensionLanguage( $filename, $arrayname ); + } else { + usage(); } - checkLanguage( $extLanguages, $lang ); -/* - print "== $lang ==\n"; - print count($ext->getUntranslatedMessages( $lang ) ) . "\n"; - print count($ext->getEmptyMessages( $lang ) ) . "\n"; -*/ } ?> -- 2.20.1