Group labels translation

ok, sorry for that ;)

fixed and committed

There's security issue with this fix http://drupal.org/cvs?commit=468534

<span class="field-group-format-toggler">' . t($group->label) . '</span>

User input printed without validation!!!

$group->label now goes to check_plain.

Suppose this should be check_plain(t($group->label)) and another missed place with direct output

Using %group is enough for check_plain()

Patch applied

+++ field_group.field_ui.inc	25 Dec 2010 15:48:25 -0000
@@ -455,7 +455,7 @@ function field_group_delete_form($form, 
   $output = confirm_form($form,
-    t('Are you sure you want to delete the group %group?', array('%group' => check_plain(t($group->label)))),
+    t('Are you sure you want to delete the group %group?', array('%group' => t($group->label))),

This hunk is lost before commit!!!

Also I'd like to point that core does not translate a strings that are user defined - this task for l18n module

Powered by Dreditor.

Second line of patch also committed.

Thx for the Dreditor link. Didn't know that one. Very usefull for small patches.
Hope Chrome support works fast. :)

Nice job ;)

This will be incorporated into the next update?

This is on dev. So it will idd be incorporated into the next update.

an estimated date???

It is not on the 7.x - 2.x-dev yet

subscribe - solution for 7?