Missing t()-function in many lines.

CommentFileSizeAuthor
#5 archive.inc_.patch1.14 KBSusurrus
#5 archive.inc_.patch1.14 KBSusurrus
#1 archive_0_0.patch5.94 KBSusurrus
archive.inc__0.patch2.94 KBdicreat

Comments

Susurrus’s picture

Susurrus commented
StatusFileSize
new archive_0_0.patch5.94 KB

I've attached a patch that fixes all of the translating issues your patch covered, but I just translated every entry in the month array, versus translating at usage, since this should result in half as many calls to t().

This patch also addresses translation in the calendar block along with start of week settings being ignored by the calendar block (bug #83063). Let me know if this works for you.

Susurrus’s picture

Susurrus commented
Status: Needs review » Fixed

Fixed in 5.x-1.6

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)
dicreat’s picture

dicreat commented
Status: Closed (fixed) » Active

In last release 5.x-1.7 this problem present again.

Susurrus’s picture

Susurrus commented
Version: 5.x-1.5 » 5.x-1.7
StatusFileSize
new archive.inc_.patch1.14 KB
new archive.inc_.patch1.14 KB

Okay, so I think translation wasn't working correctly for the page titles, though I think it's still working for the rest of the module. I've changed around some code substantially, and it seems that the translation files require line numbers to be correct, so many things weren't working correctly. Including files is currently broken, but I've committed my changes to CVS, so you can grab the latest archive.inc as well as the archive-module.pot file that needs to be updated.

dicreat’s picture

dicreat commented
Status: Active » Fixed

Thanks, patch work for me.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

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