The newsletter tid (taxonomy term ID) isn't set when newsletter is created through SCS.

This creates the unwanted situation where a newsletter exists but one of it's mandatory fields is empty.

The attached patch fixes this issue providing a way to set the type of newsletter being created at the initial node selection page.

Please consider it for inclusion.

CommentFileSizeAuthor
scs-set_newsletter_tid.patch3.31 KBrsevero

Comments

dtrdewaele’s picture

dtrdewaele commented
Assigned: Unassigned » dtrdewaele
Status: Needs review » Needs work

This is a nice addiction to the module.
I'm not going to apply the patch to the core of the module, but we are going to create an addon api so all these things can go in an addon module.
We want the core of the module to be clean and simple.

dtrdewaele’s picture

dtrdewaele commented
Status: Needs work » Fixed

The newly created addon for the module has the possibilitity to set a default taxonomy term for every created newsletter for the module.

This is not completely what your patch is doing, but we are working further on this.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

lucuhb’s picture

lucuhb commented

I have the same needs, i.e. set taxonomy before the newsletter was created because my url aliases are based on a vocabulary (I use a view to create newsletter).
Where can I get the addon module indicated in #2 ?