fu: Close-up of Fu, bringing a scoop of water to her mouth (Default)
fu ([personal profile] fu) wrote in [site community profile] changelog2010-04-13 10:36 am

[dw-free] Deleted OpenID accounts don't have strikethrough on manage/circle/edit

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

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

Strike through deleted openid user tags in all views.

Patch by [personal profile] noodles.

Files modified:
  • cgi-bin/LJ/User.pm
--------------------------------------------------------------------------------
diff -r dedb01f15a96 -r ba43ac3e0131 cgi-bin/LJ/User.pm
--- a/cgi-bin/LJ/User.pm	Mon Apr 12 19:30:25 2010 -0700
+++ b/cgi-bin/LJ/User.pm	Tue Apr 13 03:40:43 2010 -0700
@@ -2821,11 +2821,16 @@ sub ljuser_display {
     my $id = $u->identity;
     return "<b>????</b>" unless $id;
 
-    my $andfull = $opts->{'full'} ? "&amp;mode=full" : "";
-    my $img = $opts->{'imgroot'} || $LJ::IMGPREFIX;
-    my $strike = $opts->{'del'} ? ' text-decoration: line-through;' : '';
-    my $profile_url = $opts->{'profile_url'} || '';
-    my $journal_url = $opts->{'journal_url'} || '';
+    # Mark accounts as deleted that aren't visible, memorial, locked, or
+    # read-only
+    $opts->{del} = 1 unless $u->is_visible || $u->is_memorial ||
+            $u->is_locked || $u->is_readonly;
+
+    my $andfull = $opts->{full} ? "&amp;mode=full" : "";
+    my $img = $opts->{imgroot} || $LJ::IMGPREFIX;
+    my $strike = $opts->{del} ? ' text-decoration: line-through;' : '';
+    my $profile_url = $opts->{profile_url} || '';
+    my $journal_url = $opts->{journal_url} || '';
     my $display_class = $opts->{no_ljuser_class} ? "" : "class='ljuser'";
     my $type = $u->journaltype_readable;
 
--------------------------------------------------------------------------------