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-08-27 05:59 pm

[dw-free] Comment headers should indicate if a comment is screened

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

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

Add (screened) in front of screened comments subjects.

Patch by [personal profile] yvi.

Files modified:
  • bin/upgrading/s2layers/core2.s2
  • htdocs/talkread.bml
  • htdocs/talkread.bml.text
--------------------------------------------------------------------------------
diff -r c264a63ae857 -r 624cbb39f2ca bin/upgrading/s2layers/core2.s2
--- a/bin/upgrading/s2layers/core2.s2	Thu Aug 27 16:30:55 2009 +0000
+++ b/bin/upgrading/s2layers/core2.s2	Thu Aug 27 17:59:12 2009 +0000
@@ -2332,7 +2332,7 @@ set text_comment_edittime = "Edited at";
 
 property string text_screened {
    des = "The text used when a comment is screened";
-   example = "(screened)";
+   example = "(screened) ";
 }
 
 property string text_poster_anonymous {
@@ -2366,7 +2366,7 @@ property string text_reply_nocomments {
 
 property string text_website_default_name { noui = 1; des = "If an account's website is specified, but there's no website name, use this text instead"; }
 
-set text_screened = "screened";
+set text_screened = "(screened) ";
 set text_poster_anonymous = "(Anonymous)";
 set text_openid_from = "from";
 set text_reply_back = "Read Comments";
@@ -3928,7 +3928,11 @@ function EntryLite::print_subject() [fix
 
 function EntryLite::print_subject( string{} opts ) [fixed] {
     if ($this isa Comment) {
+        var Comment c = $this as Comment;
         """<div class="comment-title">""";
+        if ($c.screened) {
+            print $*text_screened;
+        }
         print $this->formatted_subject( $opts );
         "</div>";
     }
diff -r c264a63ae857 -r 624cbb39f2ca htdocs/talkread.bml
--- a/htdocs/talkread.bml	Thu Aug 27 16:30:55 2009 +0000
+++ b/htdocs/talkread.bml	Thu Aug 27 17:59:12 2009 +0000
@@ -572,7 +572,11 @@ body<=
                 }
 
                 my $cleansubject = LJ::ehtml($post->{'subject'});
-                $ret .= "<span class='commentsubject'><b>$cleansubject</b></span> $icon";
+                if ( $post->{state} eq 'S' ) {
+                    $ret .= "<span class='commentsubject'>" . BML::ml( '.comment.screened.subject' ) . "<b>$cleansubject</b></span> $icon";
+                } else {
+                    $ret .= "<span class='commentsubject'><b>$cleansubject</b></span> $icon";
+                }
                 $ret .= "<br />$user\n";
                 $ret .= " " . BML::ml( '.from_external', {site=>$post->{props}->{imported_from}} ) if $post->{props}->{imported_from};
                 $ret .= "<br /><span class='datepost'>$datepost</span>\n";
diff -r c264a63ae857 -r 624cbb39f2ca htdocs/talkread.bml.text
--- a/htdocs/talkread.bml.text	Thu Aug 27 16:30:55 2009 +0000
+++ b/htdocs/talkread.bml.text	Thu Aug 27 17:59:12 2009 +0000
@@ -6,6 +6,8 @@
 .button.post=Post Comment
 
 .confirm.action=Are you sure you wish to delete the selected comments?
+
+.comment.screened.subject=(screened) 
 
 .deletedpost=<b>(Deleted post)</b>
 
--------------------------------------------------------------------------------

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