kareila: (Default)
kareila ([personal profile] kareila) wrote in [site community profile] changelog2010-01-12 08:07 pm

[dw-free] Interests of one user replacing interests of another user

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

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

Fix scoping problem with conditional variable assignment.

Patch by [personal profile] alierak.

Files modified:
  • cgi-bin/LJ/User.pm
--------------------------------------------------------------------------------
diff -r 43c370a701e8 -r ea171f7eb151 cgi-bin/LJ/User.pm
--- a/cgi-bin/LJ/User.pm	Mon Jan 11 18:59:36 2010 +0000
+++ b/cgi-bin/LJ/User.pm	Tue Jan 12 14:07:27 2010 -0600
@@ -4220,7 +4220,8 @@ sub get_interests {
 
     # load the ids
     my $mk_ids = [$uid, "intids:$uid"];
-    my $ids = LJ::MemCache::get($mk_ids) unless $opts->{forceids};
+    my $ids;
+    $ids = LJ::MemCache::get($mk_ids) unless $opts->{forceids};
     unless ( $ids && ref $ids eq "ARRAY" ) {
         $ids = [];
         my $dbh = LJ::get_db_writer();
--------------------------------------------------------------------------------