Posted by hunmonk on October 23, 2007 at 2:52pm
4 followers
| Project: | Drupal core |
| Version: | 6.x-dev |
| Component: | comment.module |
| Category: | feature request |
| Priority: | critical |
| Assigned: | hunmonk |
| Status: | closed (fixed) |
Issue Summary
attached patch adds a 'Posted in' column to the table at admin/content/comment.
this is especially important now that issue followups are going to be comments, because the issue comment subjects themselves provide no contextual information (#1, #2, etc).
tested the display for published and unpublished comments, and the table sorting by node title -- all seems to be working perfectly.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| comment_admin_post_title.patch | 2.71 KB | Ignored: Check issue status. | None | None |
Comments
#1
This is definitely valuable information. Patch applies, code looks sane to me and works as expected. Being brave here and setting as RTBC.
#2
I would rather alias "title" as "node_title" or something along these lines in the query and use it under that name later. The "subject" of comment and the "title" of the node can be quite confusing this way. Otherwise the functionality and implementation looks good to me.
#3
attached has been adjusted according to your request.
#4
Very welcome improvement. Patch applies, works great and addresses Gabor's comments, so RTBC.
#5
Great, thanks, committed.
#6
Automatically closed -- issue fixed for two weeks with no activity.