[contributions(DRUPAL-6--2):goba] /modules/l10n_server/l10n_community l10n_community.module /modules/l10n_server/l10n_community/tests l10n_community.test /modules/l10n_server/l10n_community translate.inc
User: goba Branch: DRUPAL-6--2 Date: Tue, 16 Feb 2010 07:46:50 +0000 Modified files: /modules/l10n_server/l10n_community l10n_community.module /modules/l10n_server/l10n_community/tests l10n_community.test /modules/l10n_server/l10n_community translate.inc Log message: #715606 by myself: Updated tests to reflect the current reality of form elements and tools - /view and /edit now became /translate - changed plural assert to look for both strings in one structure now (more accurate test) - merged submitter checking now that we do not have individual callback paths (slightly less accurate test) - decline and approve actions now done on the same page via checkboxes and radio buttons - messages slightly changed when no source string is found Found issues along the way: - not all feedback messages were printed when submitting translations or .po files - suggestion additions always registered as a translation submission even if not directly approved Before patch: 1004 passes, 184 fails, 0 exceptions. After patch: 1130 passes, 0 fails, 0 exceptions, yay! (Less asserts due to how we check for singular/plural string and user submission info in a more compact fashion). Links: http://cvs.drupal.org/diff.php?path=contributions/modules/l10n_server/l10n_c... http://cvs.drupal.org/diff.php?path=contributions/modules/l10n_server/l10n_c... http://cvs.drupal.org/diff.php?path=contributions/modules/l10n_server/l10n_c...
participants (1)
-
drupal-cvs@drupal.org