Patch attached

CommentFileSizeAuthor
webform_un_t_ify_schema-D6.patch13.24 KBhass

Comments

quicksketch’s picture

quicksketch
he/him
commented

As far as I know, t() is supposed to remain in hook_schema: http://api.drupal.org/api/function/hook_schema

hass’s picture

hass commented

No... it has been removed from core and all module maintainers should do, too. See #332123: Remove t() from all schema descriptions

zoltán balogh’s picture

zoltán balogh commented

Patch is works fine.

pasqualle’s picture

pasqualle
Location 🇭🇺 Budapest
commented
Status: Needs review » Reviewed & tested by the community

yes, the t() should be removed from the .install files. Makes translators life much more easier..

quicksketch’s picture

quicksketch
he/him
commented

Sorry I was keeping open to remind myself to fix the documentation in #332123: Remove t() from all schema descriptions. Now that's fixed I can commit this patch. :-)

quicksketch’s picture

quicksketch
he/him
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD and DRUPAL-6--2. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.