This patch adds a service to subscribe/unsubscribe an email to a newsletter (requires simplenews module : http://drupal.org/project/simplenews )

CommentFileSizeAuthor
services_simplenews_service.patch4.43 KBg10

Comments

snelson’s picture

Category: bug » feature
Status: Fixed » Needs review

Awesome, and perfect timing. I need this right now.

But, I'm not sure whether this should go in Services core. I think services core should really only contain services for core modules (I make an exception for views), and contrib modules would have additional modules for services. I realize this is a bit of work to open up a whole new project for such a module as this, but ideally, it should become part of the simplenews module, IMO.

I'm open to discussion on this...

When coding services, I imagined that someday it might replace core xmlrpc. Then, with services part of core, each core module and contrib module would be in charge of providing service hooks, through separate "_service" modules.

Thanks for your work on this.

Scott

g10’s picture

noprob for the work, in fact its simply a wrapper that calls the methods of the simplenews api…

as for this not belonging in the core, I agree with you…
what would be the best approach for this? as I think it would be overkill to add a new module for every service that is written that exposes some contrib module it's API for the services module…
maybe a project that groups all kinds of services that exposes APIs of contrib module?

snelson’s picture

I totally agree about it being overkill. Personally I'd like to see existing modules start to include service hooks, in the same way that they might hook into other contrib modules like views or cck. I don't really know if the community cares enough about services yet to do that though. As for a single contrib module with a bunch of random services, don't know about that either, seems like it would be bloat.

I think the best solution for now would be either to get simplenews to include the module in their project, or to just create a new project that you will probably never have to touch after setup.

Scott

robloach’s picture

Project: Services » Simplenews

This should be added to the Simplenews module as opposed to being added to the Services module. Could actually be part of simplenews.module instead of its own thing.

sutharsan’s picture

This can be a candidate for Simplenews for Drupal-6. I will look into it.

sutharsan’s picture

Status: Needs review » Postponed

Flagged as reminder for simplenews-6.x developement

sutharsan’s picture

Version: 5.x-1.x-dev » 6.x-2.x-dev
Status: Postponed » Needs work

This probably needs to be ported.

simon georges’s picture

Apparently, there already is an issue with a patch for D6 : #221521: SimpleNews Service.

miro_dietiker’s picture

Status: Needs work » Closed (duplicate)

Setting this to duplicate although it was before..

Please update the referenced patch to 6.x-2.x and add unsubscribe support.
But most importantly - what i also miss here - is a permission check.