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-06-28 01:38 am

[dw-free] Visiting a deleted journal shows blank error message

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

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

Error message on deleted and suspended journals.

Patch by [personal profile] kareila.

Files modified:
  • cgi-bin/LJ/User.pm
--------------------------------------------------------------------------------
diff -r 25508e22121e -r 1372d04b4ce0 cgi-bin/LJ/User.pm
--- a/cgi-bin/LJ/User.pm	Sun Jun 28 01:28:39 2009 +0000
+++ b/cgi-bin/LJ/User.pm	Sun Jun 28 01:37:28 2009 +0000
@@ -8421,10 +8421,11 @@ sub make_journal
     my $error = sub {
         my $msg = shift;
         my $status = shift;
+        my $header = shift || 'Error';
         $opts->{'status'} = $status if $status;
 
         return qq{
-            <h1>Error</h1>
+            <h1>$header</h1>
             <p>$msg</p>
         }.("<!-- xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx -->\n" x 50);
     };
@@ -8570,15 +8571,14 @@ sub make_journal
 
     unless ($geta->{'viewall'} && LJ::check_priv($remote, "canview", "suspended") ||
             $opts->{'pathextra'} =~ m!/(\d+)/stylesheet$!) { # don't check style sheets
-        if ($u->is_deleted){
-            my $warning = LJ::Lang::get_text(LJ::Lang::get_effective_lang(),
-                                    'journal.deleted', undef, {username => $u->username})
-                       || LJ::Lang::get_text($LJ::DEFAULT_LANG,
-                                    'journal.deleted', undef, {username => $u->username});
-            return $error->($warning, "404 Not Found");
-
-        }
-        return $error->("This journal has been suspended.", "403 Forbidden") if ($u->is_suspended);
+        if ( $u->is_deleted ) {
+            my $warning = BML::ml( 'error.deleted.text', { user => $u->display_name } );
+            return $error->( $warning, "404 Not Found", BML::ml( 'error.deleted.name' ) );
+        }
+        if ( $u->is_suspended ) {
+        	my $warning = BML::ml( 'error.suspended.text', { user => $u->ljuser_display, sitename => $LJ::SITENAME } );
+            return $error->($warning, "403 Forbidden", BML::ml( 'error.suspended.name' ) );
+        }
 
         my $entry = $opts->{ljentry};
         return $error->("This entry has been suspended. You can visit the journal <a href='" . $u->journal_base . "/'>here</a>.", "403 Forbidden")
--------------------------------------------------------------------------------

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