[DW-Changelog] dw-free: http://bugs.dwscoalition.org/show_bug.cgi?id=3097

[Dreamwidth Changelog]

        Filter out userpic mappings with null keyword ids (pic#...) from 
selects. These are handled by if-statements at other points.

        Patch by fu. 

diffs (21 lines):

diff -r b5f5f34d1cc9 -r 5899499b8c7a cgi-bin/LJ/Userpic.pm
--- a/cgi-bin/LJ/Userpic.pm     Sun Oct 03 12:27:46 2010 -0500
+++ b/cgi-bin/LJ/Userpic.pm     Mon Oct 04 08:24:37 2010 +0800
@@ -1043,7 +1043,7 @@ sub set_keywords {
     my $dbh;
 
     if ( $have_mapid ) {
-        $sth = $u->prepare( "SELECT kwid FROM userpicmap3 WHERE userid=? AND 
picid=?" );
+        $sth = $u->prepare( "SELECT kwid FROM userpicmap3 WHERE userid=? AND 
picid=? AND kwid NOT NULL" );
     } else {
         $sth = $u->prepare( "SELECT kwid FROM userpicmap2 WHERE userid=? AND 
picid=?" );
     }
@@ -1056,7 +1056,7 @@ sub set_keywords {
 
     my %kwid_to_mapid;
     if ( $have_mapid ) {
-        $sth = $u->prepare( "SELECT mapid, kwid FROM userpicmap3 WHERE 
userid=?" );
+        $sth = $u->prepare( "SELECT mapid, kwid FROM userpicmap3 WHERE 
userid=? AND kwid NOT NULL" );
         $sth->execute( $u->userid );
 
         while (my ($mapid, $kwid) = $sth->fetchrow_array) {
_______________________________________________
dw-changelog mailing list
dw-changelog@lists.dwscoalition.org
http://lists.dwscoalition.org/mailman/listinfo/dw-changelog