Hello,

Is there a way to add the functionality provided by Comment Notify in Subscriptions module? If it can be coded I would like to give it a try. In particular, for a given node, I am trying to give users the option of subscribing only to replies to their comment, not just any comment to that node (this is provided by Comment Notify)

My background is not a hard-core developer, but with some guidance and pointers I can maybe try and get it to work, like for example, sending notifications based on cck node reference, thanks to http://drupal.org/node/286789.

I am on Drupal 5. Any pointers would be appreciated.

Thanks,
Lakshmi

Comments

salvis’s picture

Status: Active » Closed (won't fix)

What is the "functionality provided by Comment Notify" that you'd like to add to Subscriptions?

If it's anonymous subscriptions, then no.

subscribing only to replies to their comment

This could be done, but it would have to follow the pattern of subscriptions_blog rather than subscriptions_cck.

However, new functionality would have to go into D6 first (soon D7) and only afterwards to D5. We cannot have users who upgrade to D6 lose functionality.

Please reopen and change the version to D6 if you want to pursue or further discuss this.

lakshmi_drupal2’s picture

Title: Ading Comment Notify functionality » Adding Comment Notify functionality
Version: 5.x-2.5 » 6.x-1.3
Status: Closed (won't fix) » Active

Hi Salvis,

Reopened for D6. I was looking for code related to "subscriptions_blog", where might I find it? I have downloaded Subscriptions module for D6.

Lakshmi

salvis’s picture

Sorry, I meant to say subscriptions_blog_ui.module. This shows you how to add a new Subscribe option and a new Subscriptions list page.

Like subscriptions_blog_ui you'll essentially create node/nid subscriptions. The challenge that I see right away will be how to distinguish them from the normal node/nid subscriptions ("subscribe to this page")
-- for execution (i.e. how to know when to check for the parent comment), and
-- for listing, because they are different from the normal node/nid subscriptions on the "Pages: list.

lakshmi_drupal2’s picture

How about having a separate subscriptions_comment.module where we would create comment/cid subscriptions, with these values: module='comment', type='cid', value=[cid of the comment being subscribed to], send_updates=0, send_comments=0 (actually I don't know the significance of the module column unless I read some more and understand the code better)

And in subscriptions_comment_comment() we look up an ancestor of the comment being inserted and if it is in subscriptions table then add it to subscriptions_queue? So users can subscribe to replies to their comment.

I imagine in hook_comment(&$a1, $op) $a1 will have the whole comment structure including pid?

The subscriptions list for comments could be displayed in a separate tab like pages and we list subscribed comments with their links.

salvis’s picture

Very nice plan, go for it!

damienmckenna’s picture

Has anyone worked on this? Right now our client's previous developer hacked the functionality into the module - after pulling it out into a separate module it works but is obviously a little hacky as we don't (currently) have a way to let people control it.

damienmckenna’s picture

I'd break this down into several specific user stories:

  • Option to subscribe users to a given node when they post a comment.
  • Option to subscribe users to any node's comment.

What else is needed?

salvis’s picture

Has anyone worked on this?

No.

Option to subscribe users to a given node when they post a comment.

Don't we already have that?

Option to subscribe users to any node's comment.

I don't understand what this is.

salvis’s picture

Issue summary: View changes
Status: Active » Closed (outdated)