Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
configurable by admin, all subscription when not a user of the website.
Also provide the url to the mailman interface if they don't want to use this.
If no-one objects, i'll build this feature myself :-)
Comment | File | Size | Author |
---|---|---|---|
#11 | mailman_manager _1.1.patch | 4.16 KB | ccb621 |
#7 | mailman_manager.patch | 4.92 KB | ccb621 |
#3 | mailman_manager_anon.php_.txt | 3.21 KB | cgfoz |
Comments
Comment #1
Shiny CreditAttribution: Shiny commentedThoughts --- should be on a per list basis.
I've hooked this module further into the forms api - so it uses the _validate and _submit to handle list creation.
Next a database patch to create a boolean column, "allow_anon".
Comment #2
cboshuizen CreditAttribution: cboshuizen commentedA better approach might be to support user roles. Being able to restrict different lists to different users roles is really an essential feature.
Comment #3
cgfoz CreditAttribution: cgfoz commentedHi, I've just implemented a version of this for a site I'm working on. See attached php code
It works so that you can add php into any page which will produce the appropriate subscribe form, or link to archive for a particular list
For an anonymous subscribe form:
For a link to an archive:
Seems to work well for my site, but if I hit anymore bugs I'll update
Comment #4
Shiny CreditAttribution: Shiny commentedawesome-ness of the form api!
Comment #5
dan.blah CreditAttribution: dan.blah commentedvery nice addition. i am having an issue where i have two submit forms on the same page for two different lists and for both submits submit a request to the first mailing list (list with the higest lid).
Comment #6
Shiny CreditAttribution: Shiny commentedthere's some bugs reported to investigate.. but otherwise awesome
can you submit this as a patch? there's some info on creating a patch file here:
http://drupal.org/patch/create
you'll want to change this:
to this
also, run it by the code style script e.g.
drupal/scripts/code-style.pl drupal/modules/all/mailman_manager/mailman_manager.module
this will point out where you've missed some of the coding style standards, which are also here:
http://drupal.org/coding-standards
thanks heaps for your work here, it's a feature that's going to be of use to many folks
-- Brenda
Comment #7
ccb621 CreditAttribution: ccb621 commentedThis is my first time doing any development for Drupal, so forgive me if I messed up somewhere.
I created a patch incorporating the changes to allow anonymous users to subscribe to lists. I made the following changes to the code:
1. Made change mentioned in comment #6.
2. Added spaces/quotes where suggested by code-style.pl.
3. Removed echo of mailman command (e.g. subscribe no-digest email=...) when users click Submit button. This can be re-enabled by commenting out line 735 and un-commenting 736.
Comment #8
daddison CreditAttribution: daddison commentedWorked pretty much as advertised for me -- although the patch would not take on the 5.x-1.1 version of the mailman_manager code, just on the 5.x-1.0 code.
Comment #9
samuelet CreditAttribution: samuelet commentedDo you mean that the patch does not apply or that it does not work as aspected when it's applied?
Comment #10
daddison CreditAttribution: daddison commentedPatch does not apply on 5.x-1.1. Applies and works fine on 5.x-1.0 version.
Comment #11
ccb621 CreditAttribution: ccb621 commentedThe previous patch removed some lines (as far as I can tell; I'm not a regular user of diff). This new one simply adds the appropriate lines. I have tested it with 5.x-1.1 and it works.
Comment #12
ccb621 CreditAttribution: ccb621 commentedIf for some reason your patch fails, try removing the space in the filename above between 'r' and '_'.
Comment #13
yan CreditAttribution: yan commentedIs this patch gonna be commited? I'm looking for that function...
Comment #14
samuelet CreditAttribution: samuelet commentedComment #15
toddgee CreditAttribution: toddgee commentedI'm looking at the patch, looking to port it to the DRUPAL-6--2 branch.... more as it develops.
Comment #16
toddgee CreditAttribution: toddgee commentedThoughts.... this seems to be part of the configuration of the D-6 version (not sure about D-5)... it doesn't work, however. The link that comes up for the anon user, '/user/0/mailman_manager', is backed by code that explicitly checks for uid=0 for the auth check.
I'm interested in helping out, just want to see what the status is first.
thanks, tod
Comment #17
toddgee CreditAttribution: toddgee commentedIs there any updates to this?
Comment #18
yan CreditAttribution: yan commentedThis is possible in 6.x version. I suppose there won't be any more features in 5.x, will there?