Changeset 8277

Timestamp:

07/07/08 22:49:52 (5 months ago)

Author:

mdawaffe

Message:

crazyhorse: let's try a spam list in edit-comments.php too

Files:

• branches/crazyhorse/wp-admin/admin-ajax.php (modified) (1 diff)
• branches/crazyhorse/wp-admin/css/global.css (modified) (1 diff)
• branches/crazyhorse/wp-admin/edit-comments.php (modified) (3 diffs)
• branches/crazyhorse/wp-admin/includes/template.php (modified) (3 diffs)
• branches/crazyhorse/wp-admin/js/edit-comments.js (modified) (1 diff)
• branches/crazyhorse/wp-admin/wp-admin.css (modified) (1 diff)

branches/crazyhorse/wp-admin/admin-ajax.php

@@ -136 +136 @@
-        die('-1');
-
-'unapproved' == wp_get_comment_status($comment->comment_ID
+in_array( wp_get_comment_status($comment->comment_ID), array( 'unapproved', 'spam' ) 
-        check_ajax_referer( "approve-comment_$id" );
-        if ( wp_set_comment_status( $comment->comment_ID, 'approve' ) )

branches/crazyhorse/wp-admin/css/global.css

@@ -111 +111 @@
-
-.subsubsub li { display: inline; margin: 0; padding: 0; }
+
+/* make it small? */
+.subsubsub li.spam {
+    font-size: .9em;
+}
-
-.widefat {

branches/crazyhorse/wp-admin/edit-comments.php

@@ -96 +96 @@
-$status_links = array();
-$num_comments = wp_count_comments();
-
+
+
+
+
+
-$class = ( '' === $comment_status ) ? ' class="current"' : '';
-$status_links[] = "<li><a href=\"edit-comments.php\"$class>".__('Show All Comments')."</a>";
@@ -105 +109 @@
-        $class = ' class="current"';
-
-
+
+
-}
-
@@ -167 +172 @@
-<select name="action">
-<option value="" selected>Actions</option>
-
+
+
+
-<option value="approve"><?php _e('Approve'); ?>
-<?php endif; ?>
+<?php if ( 'spam' != $comment_status ): ?>
-<option value="markspam"><?php _e('Mark as Spam'); ?></option>
-<?php if ( 'moderated' != $comment_status ): ?>
-<option value="unapprove"><?php _e('Unapprove'); ?></option>
-<?php endif; ?>
-<option value="delete"><?php _e('Delete'); ?></option>

branches/crazyhorse/wp-admin/includes/template.php

@@ -710 +710 @@
-    $authordata = get_userdata($post->post_author);
-    $the_comment_status = wp_get_comment_status($comment->comment_ID);
-    $class = ('unapproved' == $the_comment_status) ? 'unapproved' : '';
-
-    if ( current_user_can( 'edit_post', $post->ID ) ) {
@@ -738 +737 @@
-
-?>
-clas
+the_comment_statu
-<?php if ( $checkbox ) : ?>
-    <td class="check-column"><?php if ( current_user_can('edit_post', $comment->comment_post_ID) ) { ?><input type="checkbox" name="delete_comments[]" value="<?php echo $comment->comment_ID; ?>" /><?php } ?></td>
@@ -761 +760 @@
-        $actions['approve']   = "<a href='$approve_url' class='dim:the-comment-list:comment-$comment->comment_ID:unapproved:e7e7d3:e7e7d3' title='" . __( 'Approve this comment' ) . "'>" . __( 'Approve' ) . '</a> | ';
-        $actions['unapprove'] = "<a href='$unapprove_url' class='dim:the-comment-list:comment-$comment->comment_ID:unapproved:e7e7d3:e7e7d3' title='" . __( 'Unapprove this comment' ) . "'>" . __( 'Unapprove' ) . '</a> | ';
-
-        // we're looking at list of only approved or only unapproved comments
-        if ( 'moderated' == $comment_status ) {
-            $actions['approve'] = "<a href='$approve_url' class='delete:the-comment-list:comment-$comment->comment_ID:e7e7d3:action=dim-comment' title='" . __( 'Approve this comment' ) . "'>" . __( 'Approve' ) . '</a> | ';
-            unset($actions['unapprove']);
-        } elseif ( 'approved' == $comment_status ) {
-            $actions['unapprove'] = "<a href='$unapprove_url' class='delete:the-comment-list:comment-$comment->comment_ID:e7e7d3:action=dim-comment' title='" . __( 'Unapprove this comment' ) . "'>" . __( 'Unapprove' ) . '</a> | ';
-            unset($actions['approve']);
-        }
-
-        $actions['edit']      = "<a href='comment.php?action=editcomment&amp;c={$comment->comment_ID}' title='" . __('Edit comment') . "'>". __('Edit') . '</a> | ';
-        $actions['spam']      = "<a href='$spam_url' class='delete:the-comment-list:comment-$comment->comment_ID::spam=1' title='" . __( 'Mark this comment as spam' ) . "'>" . __( 'Spam' ) . '</a> | ';
-        $actions['delete']    = "<a href='$delete_url' class='delete:the-comment-list:comment-$comment->comment_ID delete'>" . __('Delete') . '</a>';
+
+        if ( $comment_status ) { // not looking at all comments
+            if ( 'approved' == $the_comment_status ) {
+                $actions['unapprove'] = "<a href='$unapprove_url' class='delete:the-comment-list:comment-$comment->comment_ID:e7e7d3:action=dim-comment' title='" . __( 'Unapprove this comment' ) . "'>" . __( 'Unapprove' ) . '</a> | ';
+                unset($actions['approve']);
+            } else {
+                $actions['approve'] = "<a href='$approve_url' class='delete:the-comment-list:comment-$comment->comment_ID:e7e7d3:action=dim-comment' title='" . __( 'Approve this comment' ) . "'>" . __( 'Approve' ) . '</a> | ';
+                unset($actions['unapprove']);
+            }
+        }
+
+        if ( 'spam' == $the_comment_status )
+            unset($actions['spam']);
+
-        $actions = apply_filters( 'comment_row_actions', $actions, $comment );
+
-        foreach ( $actions as $action => $link )
-            echo "<span class='$action'>$link</span>";

branches/crazyhorse/wp-admin/js/edit-comments.js

@@ -56 +56 @@
-        a.html( n.toString() );
-    });
+    $('li span.spam-comment-count' ).each( function() {
+        var a = $(this);
+        var n = parseInt(a.html(),10);
+        if ( $(settings.target).parents( 'span.spam' ).size() ) { // we marked a comment as spam
+            n = n + 1;
+        } else if ( $('#' + settings.element).is('.spam') ) { // we approved or deleted a comment marked as spam
+            n = n - 1;
+        }
+        if ( n < 0 ) { n = 0; }
+        a.html( n.toString() );
+    });
-
-    if ( theExtraList.size() == 0 || theExtraList.children().size() == 0 ) {

branches/crazyhorse/wp-admin/wp-admin.css

@@ -247 +247 @@
-}
-
-
+, .spam .approve
-    display: inline;
-}