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

Comments

chx’s picture

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 commented
Status: Needs work » Needs review
StatusFileSize
new 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 commented
Status: Needs review » Reviewed & tested by the community

Good job, this one.

drumm’s picture

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

Committed to HEAD.

Anonymous’s picture

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