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

Comments

JirkaRybka’s picture

JirkaRybka commented
Status: Active » Needs review
StatusFileSize
new 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 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.