this is a followup for #357885: multilingual: select a newsletter term before you can send

I applied your recommended fix of updating to the dev version, but now im experiencing another error.

When I edit some properties of a newsletter on the simplenews admin interface (admin/content/simplenews/types/edit/$tid) the language gets set back to the current website's interface language. IE, it loses the original language setting. I have to subsequently go edit the page in taxonomy edit (admin/content/taxonomy/edit/term/$tid) and reset the language again.

Could not the information on the custom simplenews newsletter term edit page be PLUGGED INTO the taxonomy edit page, so everything regarding a simplenews newsletter term is all in one place, and get rid of this custom edit page altogether?

CommentFileSizeAuthor
#2 simplenews.361352.patch834 bytessutharsan

Comments

sutharsan’s picture

Title: newsletter loses international language status after editing » newsletter term loses language after editing

On the contrary, plans are to move away from using (only) taxonomy for issue identification.
I will look into this error.

sutharsan’s picture

Status: Active » Closed (fixed)
StatusFileSize
new834 bytes

Attached patch fixes this. Patch committed, please inform me your results.

jwilson3’s picture

fail against 6.x-1.0-rc5, I couldn't even apply it by hand, apparently there's quite a big difference between the dev branch and the rc5 tag as rc5 doesnt even contain the affected function simplenews_admin_types_form_sub().

yes, i realize this was filed against dev... because when i filed it rc5 didnt exist.

sutharsan’s picture

This has been applied to 6.x-1.x-dev (and HEAD) just before the RC5 release. It is therefore included in this release.
See cvs for details.