fu: Close-up of Fu, bringing a scoop of water to her mouth (Default)
fu ([personal profile] fu) wrote in [site community profile] changelog2011-01-13 05:45 am

[dw-free] Renaming tags when the new name already exists error message wrong

[commit: http://hg.dwscoalition.org/dw-free/rev/7158b7960bfe]

http://bugs.dwscoalition.org/show_bug.cgi?id=3388

Use correct variable.

Patch by [personal profile] foxfirefey.

Files modified:
  • cgi-bin/LJ/Tags.pm
--------------------------------------------------------------------------------
diff -r 7c72e84a2a8e -r 7158b7960bfe cgi-bin/LJ/Tags.pm
--- a/cgi-bin/LJ/Tags.pm	Thu Jan 13 12:38:54 2011 +0800
+++ b/cgi-bin/LJ/Tags.pm	Thu Jan 13 13:45:17 2011 +0800
@@ -1279,7 +1279,7 @@ sub rename_usertag {
     # if so, error and suggest merging the tags
     # FIXME: ask user to merge and then merge
     my $tags = LJ::Tags::get_usertags( $u );
-    return $err->( LJ::Lang::ml( 'taglib.error.exists', { tagname => LJ::ehtml( $oldkw ) } ) )
+    return $err->( LJ::Lang::ml( 'taglib.error.exists', { tagname => LJ::ehtml( $newname ) } ) )
         if $tags->{$newkwid};
 
     # escape sub
--------------------------------------------------------------------------------

Post a comment in response:

This account has disabled anonymous posting.
If you don't have an account you can create one now.
HTML doesn't work in the subject.
More info about formatting

If you are unable to use this captcha for any reason, please contact us by email at support@dreamwidth.org