class ImportSiteScripts extends Maintenance {
public function __construct() {
parent::__construct();
- $this->mDescription = 'Import site scripts from a site';
+ $this->addDescription( 'Import site scripts from a site' );
$this->addArg( 'api', 'API base url' );
$this->addArg( 'index', 'index.php base url' );
$this->addOption( 'username', 'User name of the script importer' );
public function execute() {
global $wgUser;
- $user = User::newFromName( $this->getOption( 'username', 'ScriptImporter' ) );
+ $username = $this->getOption( 'username', false );
+ if ( $username === false ) {
+ $user = User::newSystemUser( 'ScriptImporter', [ 'steal' => true ] );
+ } else {
+ $user = User::newFromName( $username );
+ }
$wgUser = $user;
$baseUrl = $this->getArg( 1 );
}
$this->output( "Importing $page\n" );
- $url = wfAppendQuery( $baseUrl, array(
+ $url = wfAppendQuery( $baseUrl, [
'action' => 'raw',
- 'title' => "MediaWiki:{$page}" ) );
- $text = Http::get( $url );
+ 'title' => "MediaWiki:{$page}" ] );
+ $text = Http::get( $url, [], __METHOD__ );
$wikiPage = WikiPage::factory( $title );
$content = ContentHandler::makeContent( $text, $wikiPage->getTitle() );
}
protected function fetchScriptList() {
- $data = array(
+ $data = [
'action' => 'query',
- 'format' => 'php', //'json',
+ 'format' => 'json',
'list' => 'allpages',
'apnamespace' => '8',
'aplimit' => '500',
- );
+ 'continue' => '',
+ ];
$baseUrl = $this->getArg( 0 );
- $pages = array();
+ $pages = [];
- do {
+ while ( true ) {
$url = wfAppendQuery( $baseUrl, $data );
- $strResult = Http::get( $url );
- //$result = FormatJson::decode( $strResult ); // Still broken
- $result = unserialize( $strResult );
+ $strResult = Http::get( $url, [], __METHOD__ );
+ $result = FormatJson::decode( $strResult, true );
- if ( !empty( $result['query']['allpages'] ) ) {
- foreach ( $result['query']['allpages'] as $page ) {
- if ( substr( $page['title'], -3 ) === '.js' ) {
- strtok( $page['title'], ':' );
- $pages[] = strtok( '' );
- }
+ $page = null;
+ foreach ( $result['query']['allpages'] as $page ) {
+ if ( substr( $page['title'], -3 ) === '.js' ) {
+ strtok( $page['title'], ':' );
+ $pages[] = strtok( '' );
}
}
- if ( !empty( $result['query-continue'] ) ) {
- $data['apfrom'] = $result['query-continue']['allpages']['apfrom'];
- $this->output( "Fetching new batch from {$data['apfrom']}\n" );
+
+ if ( $page !== null ) {
+ $this->output( "Fetched list up to {$page['title']}\n" );
}
- } while ( isset( $result['query-continue'] ) );
+
+ if ( isset( $result['continue'] ) ) { // >= 1.21
+ $data = array_replace( $data, $result['continue'] );
+ } elseif ( isset( $result['query-continue']['allpages'] ) ) { // <= 1.20
+ $data = array_replace( $data, $result['query-continue']['allpages'] );
+ } else {
+ break;
+ }
+ }
return $pages;
}