HI,

A requirement of my project is for users to be able to subscribe to individual categories of specific content types. I have CCK news and event nodes, which both have taxonomy categories assigned to them.

I made changes to this module so the user can go into "my notify settings" and see a list of checkboxes for each taxonomy term for each cck node.

I also made changes for a user to be able to define when they should receive updates: 1. immediately 2. daily 3. weekly

The only problem is, because CCK don't have teasers, I have yet to come up with a nice solution to send only teasers or the body of CCK nodes through the notification system.

I am new to drupal programming, so I am unable to provide a decent patch, but I can certainly hand over the changes I made so someone else can provide a proper patch, if anyone is interested.

Cheers,
-Justin

CommentFileSizeAuthor
#5 notify.install.patch752 bytesCauliflower
#5 notify.module.patch7.65 KBCauliflower

Comments

RobRoy’s picture

RobRoy commented
Version: 4.7.x-1.x-dev » 5.x-1.x-dev
Category: task » feature
Status: Needs review » Active

Have you looked at http://drupal.org/patch? Using TortoiseCVS you can pretty easily create a patch and this would be very helpful. If you can separate your changes so there is just one feature for each patch/issue.

bill18’s picture

bill18 commented

This can be a very useful feature for users. A Notify module sending notification for everything doesn't make much sense for users. Since I know nothing about php programming, I hope that a new notify module (on version 4.7)with such feature will be ready ASAP.

stella’s picture

stella commented

Marked #137534 as a duplicate of this.

Pushkar Gaikwad’s picture

Pushkar Gaikwad commented

So what is the progress on this as I can still see the status as active ?

I am myself looking for similar feature where ppl can subscribe to taxonomy terms rather than subscribing to everything. Is there anyway i can do this ?

Thanks

Cauliflower’s picture

Cauliflower commented
Assigned: marsrc » Cauliflower
Status: Active » Needs review
StatusFileSize
new notify.module.patch7.65 KB
new notify.install.patch752 bytes

This is my first patch for the drupal system, so please check my code and post your remarks.

After updating the notify module you can:
- on the 'Admin page': check which vocabularies the users can choose from
- on the 'My notification page': the user can select their interests

Some remarks:
- If the users haven't checked any terms of a vocabulary => he/she will be notified of all nodes
- If a node has no vocabularies specified => he/she will always be notified

Thanks.

Jack_Sparrow’s picture

Jack_Sparrow commented

I've applied the patches and am in the process of testing...
So far the output is 100% OK ;-) (There were just 2 '}' that needed to be deleted - unexpected errors...)
I've posted content and will wait to see the results...
I'll update as soon as I can.

THANK YOU FOR THIS!!!

pieterdc’s picture

pieterdc
Primary language Dutch
Location Ghent
commented

(just posting a comment as a way to subscribe to this thread...)

+1 for this feature

Cauliflower’s picture

Cauliflower commented
Assigned: Cauliflower » Unassigned

+1

mdowsett’s picture

mdowsett commented

how about an option to send out notifications on nodes that have been edited...? can that be added?

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented

Somehow applying the patch from the command line resulted in errors:

warning: include_once(./modules/notify/notify.module) [function.include-once]: failed to open stream: Permission denied in D:\myprojectpath\includes\bootstrap.inc on line 512.

The file permissions got munged for the notify.module file...note the missing '+' after ls -al

~/MyProject/modules $ ls -al notify/
total 53K
drwx------+  3 James Ninguno    0 Mar 14 16:17 ./
drwx------+ 77 James Ninguno    0 Feb 27 15:36 ../
-rwx------+  1 James Ninguno  15K Apr 23  2006 LICENSE.txt*
-rwx------+  1 James Ninguno 1.8K Jul 14  2007 README.txt*
-rwx------+  1 James Ninguno  295 Jul 14  2007 notify.info*
-rwx------+  1 James Ninguno 2.7K Mar 14 16:05 notify.install*
-rwx------   1 James Ninguno  25K Mar 14 16:05 notify.module*
drwx------+  2 James Ninguno    0 Feb 27 14:37 po/

not sure why it happened, i just copied the notify.install file (with correct permissions) to notify.module.new and pasted in the contents from the notify.module file, then erased the bad one, and replaced it with the new one.... all is well :/

--edited after i noticed the problem was the missing '+' sign -- extended security

matt2000’s picture

matt2000 commented
Status: Needs review » Closed (won't fix)

outside the scope of this module. Other modules do it better.