This is a placeholder issue to remind me about this task. Core (both 6.x and 7.x) already moved away from having t() in schema descriptions, as it makes little sense to translate them (never really shown to the user), and it's unneeded extra weight to the localization system. Having descriptions without t() is now the policy at Drupal AFAICT.

CommentFileSizeAuthor
#1 cat-schema-t.patch18.79 KBJirkaRybka
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JirkaRybka’s picture

JirkaRybka CreditAttribution: JirkaRybka commented
Status: Active » Needs review
FileSize
cat-schema-t.patch18.79 KB

This is really simple and safe one: Just removed a couple of now unwanted t()'s, added a few missing descriptions, and fixed one mention of a primary key, which isn't actually defined.

I tested that it parses with no error (by running update.php), so it should be ready to go.

Jaza’s picture

Jaza CreditAttribution: Jaza commented
Status: Needs review » Fixed

Committed to HEAD. Thanks! See:

#158598: Category port for Drupal 6.x

Status: Fixed » Closed (fixed)

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