User: goba Branch: DRUPAL-7--2 Date: Fri, 28 Jan 2011 12:45:16 +0000 Modified files: /modules/potx potx.admin.inc potx.inc potx.module /modules/potx/tests potx.test Log message: #1043406 by myself: major rework of the internal parsing API - remove support for multiple status modes, and only keep structured status collection; now external tools are responsible for making sense of the status reports gathered - removed _potx_write_files(), moved header population to file generation code, the rest is responsibility of the caller now (to save or output the string) - removed support for different build modes, since ftp.drupal.org, localize.drupal.org and l10n_update module standarizes on single .po files; no need to complicate the code here - fixed performance issue where .info files were attempted to be parsed as PHP - fixed performance/memory issue where all string tokens were used to build an index array, while we know the exact set of strings we should look for; now that array is coded into the module, should use less memory/time - unified parser for context-aware and context-less t() to use a single function for both (like format_plural() does) - optimized memory use by cleaning out temporary token data when we don't need it anymore; also provided a memory cleanup callback that was also useful for testing - renamed all functions to use a better naming scheme and reorganized functions to group them by role - added error messages when tokens are not properly identified in t() and format_plural() parsing, so we are aware of the error happening - updated default Drupal 7 region list based on Drupal 7.0 - removed t() and drupal_parse_info_file() implementations since we are going to implement command line operation via Drupal which makes us have these - removed outputScreenContents() from tests, since simpletest now support this via the UI - fixed Drupal 7 coding style issues and phpcode comment headers to be single-line TODOs include: - make error messages appear in the output (now they are not anymore in dsm() but neither in the file yet): http://drupal.org/node/185599 - rework the UI for the module: http://drupal.org/node/624370 - add drush integration: http://drupal.org/node/720986 - backport the whole 7.x-2.x branch to Drupal 6 in a new branch :) Links: http://cvs.drupal.org/diff.php?path=contributions/modules/potx/potx.admin.in... http://cvs.drupal.org/diff.php?path=contributions/modules/potx/potx.inc&old=... http://cvs.drupal.org/diff.php?path=contributions/modules/potx/potx.module&o... http://cvs.drupal.org/diff.php?path=contributions/modules/potx/tests/potx.te...