The latest cvs version does not allow to edit and delete locale strings. The attached patch fixes locale.module and locale.inc

CommentFileSizeAuthor
#2 locale_edit_strings.patch1.86 KBasimmonds
locale.patch_4.txt1.6 KBstefano73
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chx’s picture

chx CreditAttribution: chx commented
Status: Active » Needs work 
-        'callback arguments' => arg(4),
+        'callback arguments' => arg(5),

this should read array(arg(5)) . Also, I do not find it a particularly good idea to set #base to something that starts with an underscore, that can lead to functions like theme__

asimmonds’s picture

asimmonds CreditAttribution: asimmonds commented
Status: Needs work » Needs review
FileSize
locale_edit_strings.patch1.86 KB

Updated the patch for chx's comments. Added a helper function locale_admin_string_edit_submit() instead of inserting a '#base' tag into the form.

chx’s picture

chx CreditAttribution: chx commented
Status: Needs review » Reviewed & tested by the community

Good job, this one.

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD.

Anonymous’s picture

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