afuna: Cat under a blanket. Text: "Cats are just little people with Fur and Fangs" (Default)
afuna ([personal profile] afuna) wrote in [site community profile] changelog2009-05-09 10:25 am

[dw-free] Importing entries fail in certain cases if user posts a new tag between tags and entry imp

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

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

Further prevent checking for the tags limit

Patch by [personal profile] exor674.

Files modified:
  • cgi-bin/taglib.pl
--------------------------------------------------------------------------------
diff -r e8a3218f40d6 -r fd96649d7d3e cgi-bin/taglib.pl
--- a/cgi-bin/taglib.pl	Sat May 09 18:04:39 2009 +0800
+++ b/cgi-bin/taglib.pl	Sat May 09 10:24:34 2009 +0000
@@ -788,7 +788,8 @@ sub update_logtags {
     }
 
     # now we can create the new tags, since we know we're safe
-    LJ::Tags::create_usertag($u, $_, { display => 1 }) foreach @to_create;
+    # We still need to propagate ignore_max, as create_usertag does some checks of it's own.
+    LJ::Tags::create_usertag( $u, $_, { display => 1, ignore_max => $opts->{ignore_max} } ) foreach @to_create;
 
     # %add and %delete are accurate, but we need to track necessary
     # security updates; this is a hash of keyword ids and a modification
--------------------------------------------------------------------------------

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