Yep.

CommentFileSizeAuthor
#1 refactor_to_use_merlins_function.patch2.52 KBgreggles

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Active » Needs review
StatusFileSize
new refactor_to_use_merlins_function.patch2.52 KB

Based on the code cleanup patch that Earl provided, here's a patch that fixes this.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Title: if only one subscription mode is enabled, users should only see one checkbox in their edit page » if only one subscription mode is enabled, users should only see one option in their edit page
greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Needs review » Patch (to be ported)

Now fixed - http://drupal.org/cvs?commit=225176

drupdruppalpal’s picture

drupdruppalpal commented

Hi,
is it possible to apply this patch to the current 6.x-1.2 release?
I have try to apply the patch manually but it doesn’t works for me.
The options are still available.

Thanks in advance

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

No, but you can upgrade to 6.x-1.x-dev and get the same code.

If that works better I could create a 6.x-1.3 official release.

drupdruppalpal’s picture

drupdruppalpal commented

ok,
upgading
thanks

drupdruppalpal’s picture

drupdruppalpal commented

would be great to have a new release with these two patches:

- if only one subscription mode is enabled, users should only see one option in their edit page
- Allow admin to set the node notify default

Regards,

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Patch (to be ported) » Closed (fixed)

This module is no longer actively developed for Drupal 5. If someone wishes to take over as the 5.x maintainer I would consider it, but these days everyone should really upgrade to Drupal 6.x.