This patch simply wraps the field label with t() when declaring the cck widget to make it consistent with other cck modules.

CommentFileSizeAuthor
fivestar_field_label_i18n.patch844 bytesfloretan

Comments

quicksketch’s picture

Status: Needs review » Fixed

Thanks. Normally translating dynamic strings like this is a bad practice, but it's what CCK does itself for field labels, so we'll have to assume this is what most translators will expect. Committed to both 5.x and 6.x branches.

Status: Fixed » Closed (fixed)

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

joostvdl’s picture

Status: Closed (fixed) » Reviewed & tested by the community

I tested this patch and it works. Please change it in the core.

ezra-g’s picture

Status: Reviewed & tested by the community » Closed (fixed)

This is already committed as per the comment in #1.