Shouldn't the Subscription module send email subscriptions on content when that content is set to "published" and not just on node creation? I think this may be a bug, and if not, it is a feature request. :-)
I'd like to be able to send a weekly newsletter with a number of articles in it. In order to do that, I made a "newsletter" taxonomy that all new users get subscribed to automatically. Then on a weekly day, I create several nodes and assign them the "newsletter" term. Unfortunately, if the cron job runs before I'm done with the content, the digest goes out with partial newsletter content, then they get another one when cron runs again with the rest of the content. It doesn't recognize when I set a node to "published" and doesn't send these nodes, even when "send notifications" is turned on in the node creation settings.
I'd like to be able to create a group of node articles without publishing them, then in the content menager I can "publish" them all at once. That way my subscribers will get their digests intact. :-)
As it stands now, I have to log in to the server and disable my cron runs in the crontab interface, then make my newsletter content, then re-enable the cron runs so they can be delivered.
Just seems like the whole thing would work much better if the subscriptions were set to go out when a node is set to "published" instead of as soon as it was created.
Let me know if there is a better way for me to do this.
Oh... another thing that would be handy is a way to disable the user's capability to have "Digest" or not. Currently I'm disabling their ability to turn it off in CSS.
Comments
Comment #1
vm commentedwouldn't simplenews and the simplenews digest modules be a better fit for this task?
Comment #2
salvisYou are right, it's supposed to send out on publish, not on (EDIT: unpublished) creation, with one exception: users with the administer nodes permission (who can see unpublished nodes) will get notification of unpublished nodes.
So, be sure to do your testing with a non-administrative account!
Comment #3
gregarios commented@VeryMisunderstood:
It would if I could get it to work with Drupal 6.
See this issue for details: http://drupal.org/node/368705
I tried Notifications/Messaging module as well, but couldn't get that to work either.
I have Subscriptions doing what I need, except it can't seem to handle 1600 users all signed up for content.
I've been trying to find a solution for 4 months.
Comment #4
gregarios commented@salvis:
Hmmm... I'll try that. :)
Comment #5
mudd commentedIf this is true then I disagree entirely with this model.
Drupal's access layer is such that any number of different 3rd party modules are applied resulting in a set of nodes shown to the visitor. It sounds like the Subscription module would send notification of new content to users who may NOT be able to view the content due to:
I looked through Subscription's documentation and found nothing about what triggers the subscription notification or whether it observes this common access model. Does anyone know if it follows the access rules?
Comment #6
salvis@mudd: Of course, Subscriptions follows the node access model. I happen to also be the maintainer of the ACL and Forum Access modules.
The OP (gregarios) didn't follow up, so I assume my explanation has cleared up the issue.
Comment #7
gregarios commentedSorry... I haven't had time to test this yet. If anyone else has time to see if emails go out to regular users only after publishing, even though they go out to admin accounts immediately even without publishing, that'd be great.
Comment #8
mudd commented@salvis: Sorry, I should have installed it and tested before I posted.
Here's some quick testing results (I'm glad to post this, because I need exactly this well thought-out module -- Thanks salvis!)
I'm using Workflow, have Subscriptions set to send notice As soon as possible, and test by executing cron through the Devel module.
While subscribing to both content-type being tested AND taxonomy terms AND I'm subscribing to the terms I'm testing with:
While ONLY subscribing to a set taxonomy terms:
While subscribing to content-type at the time node is created BUT unsubscribing before it's made accessible:
--- EDIT: I just want to be clear on my meaning here ---
My application is in a very non-casual setting, so I'm hoping to help by being nitpicky (I hope you don't mind).
Since my users may take weeks to format a node (mine are very large projects, with sub-nodes attached, etc etc) it would make sense for the Subscriptions matching to not take place at publish time (for me, but not necessarily all Workflow users). (and since workflow requires a node to be published all the while it passes through its states). But, I totally understand there wouldn't be much logic having Subscription make its check via the node access hooks because that would happen only when, and every time, a user views the node.
So I'm thinking of making patch for workflow users that provides a check box to disable the matching at publish time, and make an action available for workflow. Of course, there are lots of issues to iron out, and I'm a newbie (yikes!) so I'll try my best.
Does this make sense? If yes then I'll start a new thread and stop cluttering this one. Thanks!
---End edit---
It seems that Subscription records taxonomy matches at node creation time, but content-type matches at some different point. Since I'm using workflow, and I allow users to come back and edit their nodes before they submit for review, I'd like the Subscription check to be made at the time the node become accessible.
Would it be possible to change the responsible function to an Action (or have a configurable option) so I could fire it off by a workflow state-change trigger? This might make it very flexible. If the maintainer is open to this idea then I could take a stab at it.
Comment #9
salvisThank you for your detailed checking and accounting.
Your requirements are very specific — we'll have to work on this...
I see you've created #520230: Proposed Action plugin (e.g. for workflow) in the meantime — let's continue over there, because this goes way beyond the subject of this thread.
Comment #10
gregarios commentedOk, so the title typo was killing me... I fixed the title of this post. lol