Views integration for comment links

Any chance of making this work for Drupal 6.x/Views 2?

Additionally is there any other methods of fixing this issue and rewriting the links from the Recent Comments block to the correct comment location in the Talk tab?

-backdrifting

True, but what to do with forum topics? I would not want to have a talk page for it.

Also, having added a [nid]/talk link to the proper field, how to specify the comment's anchor?

Hmmm ... I added this [nid]/talk to the comment title field in the text rewrite box but it's not working. Can anyone send instructions on how to rewrite the comments URLs to the Talk enabled ones using Views 2? What do I put into the Text box (under Rewrite output of this URL) and then what goes into the Link Path field?

-backdrifting

The anchor is #comment-[cid]

vasudev - can you give a specific example of how you did this in Views? Can you send me a screenshot of your exact configuration in Views - how you did the actual rewrite using Tokens?

Thanks in advance.

-backdrifting

Maybe this would best be done in custom_url_rewrite_outbound() instead? We could include a sample function in the README.txt, or define the function on a conditional basis / in a separate file.

Docs would be good.

I made it work. Be sure that the order of the fields puts the nid/cid before the subject.

Updating fields... This problem is in 6.x as well as 5.x, it is now a documentation issue not a code issue, and "needs work" rather than review because someone has to actually write the documentation :-( Maybe my last posting can be the basis for that. I am willing to write the text if someone can tell me where it should go and how to get it there. Currently there is no "documentation" link on either the module project page or on the admin page when installed.

OK, I volunteer to write a new README.txt, and perhaps the project page can link to it for documentation.

There are several new features under active development (which I'm depending on for my own sites). Maybe those will converge pretty soon and there will be a new release that includes them? If so, I'll hold off until a release is near, so the documentation can cover them.

#375272: alternate path integration for D6 version
#751334: Option to use string different from "talk" in path
#444894: Add option to remove "View" and "Talk" tabs under node title

I think you can go ahead with the changes without worrying about those three issues. They won't affect the nature of the task very much so it will still be good documentation even if those three issues get implemented.

Also, rather than linking to a new README.txt from the project page you can just add your text to the current README.txt and file a patch.

The last submitted patch, TalkViewsDoc.patch, failed testing.

Revised patch using relative path.

I have the same issue as the OP. So what I gather from this is that the solution is a workaround using the views field rewrite feature, correct? I assume I can do this with View 3.x in use on my site using "node/[nid]/talk/#comment-[cid]" as the rewrite link. But won't this create a problem since my views include several node types, some of which use talk and some that do not use talk?

I've really been wanting to fix this problem on my site, because I love using this module, so any help that intrinsically fixes this views issue will make things operate much better.

@a1tsal

Your last comment applies to my situation, I have several single views that have links to comments of various node types (some using talk and some not). The approach in #12 doesn't appear to be something I am capable of right now, a code rewrite (if I am understanding that solution correctly). Since the "rewrite field output" proposed here doesn't solve my views problem, should I open a separate issue or keep following here?

Hmm. As an alternative to the (somewhat ridiculously long) series of operations needed to create the view, one could do it in one step via the Views "import" functionality.

Maybe it would be better to document that?

The gibberish to import, for the record, is:

$view = new view;
$view->name = 'recent_comments';
$view->description = 'Recent comments';
$view->tag = 'undefined';
$view->view_php = '';
$view->base_table = 'comments';
$view->is_cacheable = FALSE;
$view->api_version = 2;
$view->disabled = FALSE; /* Edit this to true to make a default view disabled initially */
$handler = $view->new_display('default', 'Defaults', 'default');
$handler->override_option('relationships', array(
'nid' => array(
'label' => 'Node',
'required' => 0,
'id' => 'nid',
'table' => 'comments',
'field' => 'nid',
'relationship' => 'none',
),
));
$handler->override_option('fields', array(
'nid' => array(
'label' => 'Nid',
'alter' => array(
'alter_text' => 0,
'text' => '',
'make_link' => 0,
'path' => '',
'link_class' => '',
'alt' => '',
'prefix' => '',
'suffix' => '',
'target' => '',
'help' => '',
'trim' => 0,
'max_length' => '',
'word_boundary' => 1,
'ellipsis' => 1,
'html' => 0,
'strip_tags' => 0,
),
'empty' => '',
'hide_empty' => 0,
'empty_zero' => 0,
'link_to_node' => 0,
'exclude' => 1,
'id' => 'nid',
'table' => 'node',
'field' => 'nid',
'relationship' => 'nid',
),
'cid' => array(
'label' => 'ID',
'alter' => array(
'alter_text' => 0,
'text' => '',
'make_link' => 0,
'path' => '',
'link_class' => '',
'alt' => '',
'prefix' => '',
'suffix' => '',
'target' => '',
'help' => '',
'trim' => 0,
'max_length' => '',
'word_boundary' => 1,
'ellipsis' => 1,
'html' => 0,
'strip_tags' => 0,
),
'empty' => '',
'hide_empty' => 0,
'empty_zero' => 0,
'link_to_comment' => 1,
'exclude' => 1,
'id' => 'cid',
'table' => 'comments',
'field' => 'cid',
'relationship' => 'none',
),
'subject' => array(
'label' => '',
'alter' => array(
'alter_text' => 1,
'text' => '[subject]',
'make_link' => 1,
'path' => 'node/[nid]/talk/#comment-[cid]',
'link_class' => '',
'alt' => '',
'prefix' => '',
'suffix' => '',
'target' => '',
'help' => '',
'trim' => 0,
'max_length' => '',
'word_boundary' => 1,
'ellipsis' => 1,
'html' => 0,
'strip_tags' => 0,
),
'empty' => '',
'hide_empty' => 0,
'empty_zero' => 0,
'link_to_comment' => 0,
'exclude' => 0,
'id' => 'subject',
'table' => 'comments',
'field' => 'subject',
'relationship' => 'none',
),
'timestamp' => array(
'label' => '',
'alter' => array(
'alter_text' => 0,
'text' => '',
'make_link' => 0,
'path' => '',
'link_class' => '',
'alt' => '',
'prefix' => '',
'suffix' => '',
'target' => '',
'help' => '',
'trim' => 0,
'max_length' => '',
'word_boundary' => 1,
'ellipsis' => 1,
'html' => 0,
'strip_tags' => 0,
),
'empty' => '',
'hide_empty' => 0,
'empty_zero' => 0,
'date_format' => 'time ago',
'custom_date_format' => '',
'exclude' => 0,
'id' => 'timestamp',
'table' => 'comments',
'field' => 'timestamp',
'relationship' => 'none',
),
));
$handler->override_option('sorts', array(
'timestamp' => array(
'order' => 'DESC',
'granularity' => 'second',
'id' => 'timestamp',
'table' => 'comments',
'field' => 'timestamp',
'relationship' => 'none',
),
));
$handler->override_option('filters', array(
'status_extra' => array(
'operator' => '=',
'value' => '',
'group' => '0',
'exposed' => FALSE,
'expose' => array(
'operator' => FALSE,
'label' => '',
),
'id' => 'status_extra',
'table' => 'node',
'field' => 'status_extra',
'relationship' => 'nid',
),
'status' => array(
'operator' => '=',
'value' => '0',
'group' => '0',
'exposed' => FALSE,
'expose' => array(
'operator' => FALSE,
'label' => '',
),
'id' => 'status',
'table' => 'comments',
'field' => 'status',
'relationship' => 'none',
),
));
$handler->override_option('access', array(
'type' => 'none',
));
$handler->override_option('cache', array(
'type' => 'none',
));
$handler->override_option('title', 'Recent comments');
$handler->override_option('use_more', 1);
$handler->override_option('use_more_always', 0);
$handler->override_option('use_more_text', 'more comments . . .');
$handler->override_option('style_plugin', 'list');
$handler->override_option('style_options', array(
'grouping' => '',
'type' => 'ul',
));
$handler = $view->new_display('block', 'Block', 'block_1');
$handler->override_option('block_description', '');
$handler->override_option('block_caching', -1);