Skip to content

Conversation

@lacatoire
Copy link
Contributor

Summary

  • Add --lang=XX parameter to all qaxml-*.php scripts, allowing direct language specification without requiring temp/lang from configure.php
  • Add simple file listing support: pass file paths as positional arguments to check specific files instead of the full translation tree
  • Existing behavior without parameters is fully preserved (backward compatible)

Usage examples

# Existing usage (unchanged)
php scripts/translation/qaxml-attributes.php

# New: specify language directly
php scripts/translation/qaxml-attributes.php --lang=fr

# New: check specific files only
php scripts/translation/qaxml-tags.php --lang=de reference/strings/functions/strlen.xml

# New: combine with existing options
php scripts/translation/qaxml-entities.php --urgent --lang=ja -trim

Files changed

  • scripts/translation/libqa/SyncFileList.php — accept optional $lang and $files parameters
  • scripts/translation/qaxml-attributes.php — parse --lang= and file args
  • scripts/translation/qaxml-entities.php — parse --lang= and file args
  • scripts/translation/qaxml-pi.php — parse --lang= and file args
  • scripts/translation/qaxml-tags.php — parse --lang= and file args
  • scripts/translation/qaxml-revtag.php — parse --lang= and file args
  • scripts/translation/qaxml-ws.php — parse --lang= and file args

Relates to #199 (Sync XML tools to accept language parameter and simple file listing)

@lacatoire lacatoire requested a review from alfsb as a code owner February 10, 2026 09:11
Allow all qaxml-*.php scripts to accept --lang=XX to specify the
target language directly, without requiring temp/lang from
configure.php. Also accept file paths as positional arguments
for checking specific files instead of the full translation tree.

Existing behavior without parameters is preserved. Relates to php#199.
@lacatoire lacatoire force-pushed the sync-tools-lang-param branch from e236d3f to 00d86bc Compare February 10, 2026 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant