The text of some popup boxes like
Loading
Are you sure you want to cancel your ...
Are you sure you want to approve the ...
Are you sure you want to disapprove the ...
can not be translated.

Comments

jaydub’s picture

Assigned: Unassigned » jaydub

Ok I've taken a look at this and in trying to dump translations I am seeing a lot of warnings from the Drupal translation layer. Going to have to take a closer look here...A quick glance shows that things like remove t() calls from hook_menu() implementation will be on the list as the change to d6 means that t() calls should be removed from menu items.

jaydub’s picture

Version: 6.x-1.0-beta9 » 6.x-1.x-dev
Category: bug » task
Status: Active » Needs review
StatusFileSize
new20.05 KB
new1.49 KB

Ok posting some patches here to improve translation. Most of the changes are from errors returned by the potx module which help to point out places where translation best practices are not being followed.

These probably should be reviewed before committing as a sanity check.

Posting one patch per module/sub-module

jaydub’s picture

UR-Implications and UR-Defaults

jaydub’s picture

UR-Node Access and UR-Blocks

alex.k’s picture

Status: Needs review » Reviewed & tested by the community

@jaydub great stuff, thank you. Apologies for not looking at this earlier. Everything looks good - please apply when you have a chance. Menu items already had t() removed, actually, in a another issue, so that will probably give you conflicts with these patches.

jaydub’s picture

I'll go over these again since some have been addressed and others might have different copy now.

andrenoronha’s picture

alex.k’s picture

Status: Reviewed & tested by the community » Fixed

Reviewed and committed these patches. Thx again.

Status: Fixed » Closed (fixed)

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